[Bug]: Image Viewer Mishandling Sequencing when Hiding Tags/Deleted is Active and a Tag is Added #84

New issue

Closed

opened 2026-06-14 16:11:17 +00:00 by ValleyGeek · 1 comment

ValleyGeek commented

2026-06-14 16:11:17 +00:00

Owner

App version

0.1.30

Platform

Both

Steps to reproduce

Open an image in the image viewer
Press delete or tag the image

Expected behavior

Active image is tagged or flagged as deleted
Active image is hidden from the image viewer scene, carousel, & filmstrip
What was the next image is now the active image in the view scene and filmstrip

Actual behavior

Tagging an image

Updates the carousel and filmstrip
The image that was tagged remains in the active image viewer scene
Navigating to the next skips what was previously the next image

Pressing Delete in Image Viewer is Skipping Next Image When 'Hide Deleted' Selected

Image is flagged as deleted
Image set is updated to hide the deleted image, altering what image the user is pointing at
A redundant next image is selected automatically

Additional context

No response

### App version 0.1.30 ### Platform Both ### Steps to reproduce 1. Open an image in the image viewer 2. Press delete or tag the image ### Expected behavior 1. Active image is tagged or flagged as deleted 2. Active image is hidden from the image viewer scene, carousel, & filmstrip 3. What was the next image is now the active image in the view scene and filmstrip ### Actual behavior ## Tagging an image 1. Updates the carousel and filmstrip 2. The image that was tagged remains in the active image viewer scene 3. Navigating to the next skips what was previously the next image ## Pressing Delete in Image Viewer is Skipping Next Image When 'Hide Deleted' Selected 1. Image is flagged as deleted 2. Image set is updated to hide the deleted image, altering what image the user is pointing at 3. A redundant next image is selected automatically ### Additional context _No response_

ValleyGeek added the

Kind/Bug

Priority/Medium

labels

2026-06-14 16:11:17 +00:00

Grok referenced this issue from a commit

2026-06-14 16:23:48 +00:00

ui: viewer/grid UX batch (#54, #60, #77, #78, #82, #84)

Grok commented

2026-06-14 16:23:52 +00:00

Member

Implemented on branch 0.1.31/issue-54-60-77-78-82-84-ux.

When tag/filter changes remove the viewer's active image (Hide Deleted, untagged-only, active tag filter):

_refresh_tag_filter_views() captures pre-invalidate session state and calls resync_after_scan() (forward-then-backward index pick) instead of keeping a stale index.
_on_viewer_delete_requested() skips advance_after_delete_marked() when Hide Deleted will prune the current image, preventing double-advance/skipped images.

Implemented on branch `0.1.31/issue-54-60-77-78-82-84-ux`. When tag/filter changes remove the viewer's active image (Hide Deleted, untagged-only, active tag filter): - `_refresh_tag_filter_views()` captures pre-invalidate session state and calls `resync_after_scan()` (forward-then-backward index pick) instead of keeping a stale index. - `_on_viewer_delete_requested()` skips `advance_after_delete_marked()` when Hide Deleted will prune the current image, preventing double-advance/skipped images.

Grok referenced this issue from a commit

2026-06-14 16:49:05 +00:00

docs: PR-prep for viewer/grid UX batch (#54, #60, #77, #78, #82, #84)

Grok referenced this issue

2026-06-14 16:49:13 +00:00

Viewer/grid UX batch (#54, #60, #77, #78, #82, #84) — v0.1.32 #85

Grok closed this issue

2026-06-14 16:49:13 +00:00

ValleyGeek referenced this issue from a commit

2026-06-14 17:00:07 +00:00

Viewer/grid UX batch (#54, #60, #77, #78, #82, #84) — v0.1.32 (#85)