[Bug]: Pressing 'x' in Image Viewer Doesn't Change Active Image When 'Hide Deleted' Active #93

Closed
opened 2026-06-15 00:07:35 +00:00 by ValleyGeek · 2 comments
Owner

App version

0.1.31

Platform

Both

Steps to reproduce

  1. Enable 'hide deleted' from the tag filter
  2. Open a duplicate group
  3. Switch to single view
  4. Press X

Expected behavior

All images in current group are marked for deletion and the next group is made active.

Actual behavior

Everything appears to happen correctly except for the image viewer not updating the active image to the group that replaces the removed one

Additional context

No response

### App version 0.1.31 ### Platform Both ### Steps to reproduce 1. Enable 'hide deleted' from the tag filter 2. Open a duplicate group 3. Switch to single view 4. Press X ### Expected behavior All images in current group are marked for deletion and the next group is made active. ### Actual behavior Everything appears to happen correctly except for the image viewer not updating the active image to the group that replaces the removed one ### Additional context _No response_
Member

Root cause: _on_dupe_delete_group (the X shortcut) tagged the group DELETE and advanced to the next group's data, but never told the image viewer to resync — unlike _on_dupe_group_prev/_on_dupe_group_next, which already call _resync_dupe_viewer_session() after switching groups. _on_dupe_keep_best had the same gap (also fixed).

Fix implemented on branch 0.1.35/issue-99-dupe-viewer-resync (commit 87289cb), with regression tests covering the "hide deleted" scenario from this report. Awaiting PR to main.

Root cause: `_on_dupe_delete_group` (the X shortcut) tagged the group DELETE and advanced to the next group's *data*, but never told the image viewer to resync — unlike `_on_dupe_group_prev`/`_on_dupe_group_next`, which already call `_resync_dupe_viewer_session()` after switching groups. `_on_dupe_keep_best` had the same gap (also fixed). Fix implemented on branch `0.1.35/issue-99-dupe-viewer-resync` (commit 87289cb), with regression tests covering the "hide deleted" scenario from this report. Awaiting PR to `main`.
Member

Fixed on branch 0.1.35/issue-99-dupe-viewer-resync (now rebased onto current main, including the CI fix from #106/#107 so this branch gets a real dev build). VERSION bumped to 0.1.36 and CHANGELOG updated. Opening a PR to main.

Fixed on branch `0.1.35/issue-99-dupe-viewer-resync` (now rebased onto current `main`, including the CI fix from #106/#107 so this branch gets a real dev build). VERSION bumped to 0.1.36 and CHANGELOG updated. Opening a PR to `main`.
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ai-collab/bulk-image-organizer#93
No description provided.