Hi again,

I found a bug that I think was not reported before.

Steps to reproduce:

  • Go to peoples tab
  • Open image from person X
  • Remove tag for person X (Ignore or assign to other person)
  • Use keyboard right arrow to go to next image

Expected behaviour:
App goes to next image from person X

Actual behaviour:
Goes to blank screen, UI disappears
Neither ESC nor Enter do work to leave
(Workaround: Leaving requires right mouse click > back )

Best Regards
Adrian

    adrianklein Hi Ardian, thank you for reporting the issue.

    However, I think there is more to it than you described. I just tried the procedure you listed, but I do not see any error. After I reject person and try to go to next image, it just refuses to go anywhere, and when I return - it just returns to the list. No blank screen, nothing odd. Probably there are some other conditions that need to be met in order it to fail as you described.

    adrianklein Expected behaviour:
    App goes to next image from person X

    Well, that part is debatable. You are now viewing the image which is not in the list anymore. So trying to go to the next image in the list is kind of undefined behaviour here. Though that does not explain blank screen of course. But refusing to go to the next image is more less expected, I think.

      Hey, just some clarifications:

      Andrey After I reject person and try to go to next image

      Did you reject a face which was confirmed before, or was it one that was still a suggestion?

      I was in the process of removing wrongly tagged images from people, so they were already confirmed to be Person X, when in fact they were Person Y. I can record a video tomorrow to show you.

      I did encounter the state where it “correctly” refuses to go to the next image. That works in a lot of cases, usually when I remove a suggested face which was not confirmed before.

      Andrey Well, that part is debatable.

      I already expected that 😃

      The discussion about this should probably be a new thread (maybe I’ll create that tomorrow), but in short:
      I think if you’re on the peoples tab viewing images from a single person, it would make sense to always continue with the next image.
      That would speed up going through suggestions (as well as cleanup), since right now if you reject a suggestion, you need to go back to the overview, find the image which you reviewed before the one you just viewed, then open the next one.

      (There is actually a kind of workaround which allows to go to the next image anyway, so as long as that works, I’m fine with the supposed behavior being to prevent navigation.)

        adrianklein Did you reject a face which was confirmed before, or was it one that was still a suggestion?

        Thank you, ignording already confirmed face dit the trick. Now I can reproduce the issus and will add it the plan for next release. Thank you for help!

        adrianklein I think if you’re on the peoples tab viewing images from a single person, it would make sense to always continue with the next image.

        I see your point and you are right, that is separate issue.

        7 days later

        I also have this problem time to time but now it’s been logged as a bug, so that’s good.

          Write a Reply...