[BBD] Viewer multi-image-selection shows wrong images

Started by ben, October 11, 2023, 10:27:56 AM

Previous topic - Next topic

ben

Bug
When selecting multiple images in the viewer that are not next to each other in the filmstrip, then the wrong images are shown in the viewer

Steps to reproduce
  • Load 6 images into the viewer: img1 img2 img3 img4 img5 img6
  • Focus img3 -> img3 is shown
  • Scroll to img5 with the mouse wheel (not clicking) -> img5 is shown
  • Hold down CTRL and click img6 -> img3 and img6 are shown, but since img5 was shown before i wanted to see img5 and img6

Btw: The new feature being able to select multiple images that are not next to each other is a huge improvement, thanks for it

Ben

axel.hennig

I can reproduce this, but not sure if this is a bug or as intended.

Mario

Quote from: axel.hennig on October 11, 2023, 10:32:24 AMI can reproduce this, but not sure if this is a bug or as intended.
I've thought about this too, for a while.
Currently, "selected" is independent from "viewed".

If you click on file 3, you select it.
Now you navigate to another file, but without clicking it in the thumbnail strip. The file is shown, but does not become selected and neither is the selection removed from file 3.
Now you Ctrl+click on file 6 and thus select it. Now there are two selected files (file 3 and 6) and these are shown.

If I would have implemented this differently, scrolling would have to reset the selection, making the file scrolled into view the selected file. Now Ctrl+clicking file 6 would show file 5 and 6.
But it now would be impossible to select files in a larger number of files by scrolling to the files you want to see, then Ctrl or Shift-clicking them as needed, then scrolling to another file etc.

I think the current implementation works very well, except perhaps in this particular case. If you want to see 5 and 6, click 5 and then Shift-click or Ctrl-click 6. This also removes file 3 from the selection automatically.

I guess I rate this a BBD (behavior by design).
-- Mario
IMatch Developer
Forum Administrator
http://www.photools.com  -  Contact & Support - Follow me on 𝕏 - Like photools.com on Facebook

ben

Thanks for you explanations.
Let me think about it one more time (as soon as i find some time for it)