ai-collab/bulk-image-organizer

Fork 0

Fix Dupe Review Multi view restoring wrong group after Single navigation (fixes #68) #71

Merged

ValleyGeek merged 2 commits from 0.1.26/issue-68-dupe-multi-return into main

2026-06-13 02:00:02 +00:00

Grok commented

2026-06-13 01:59:42 +00:00

Member

Summary

Fixes #68 — when navigating between duplicate groups in Single view, returning to Multi view (back button, Esc, or Multi button) now shows the group that was active in Single view instead of the group last open in Multi view.

Cause

Exiting Single view only synced tag metadata on the existing Multi comparison grid. Group navigation in Single view updates the active duplicate tag but intentionally skips reloading the multi grid while the viewer is open.

Fix

Track _dupe_multi_displayed_tag when the multi grid is loaded.
On exit from Single view: refresh the multi grid when the active group differs from the displayed group; otherwise keep the lightweight tag-only sync.
Apply the same displayed-tag check in _on_dupe_multi_requested.

Testing

test_dupe_single_exit_returns_active_group_multi_view (new regression test)
test_dupe_viewer_exit_skips_full_dupe_list_rebuild (unchanged — same-group exit still avoids full dupe list rebuild)
ruff check / ruff format --check

Release

Version 0.1.27
CHANGELOG updated

## Summary Fixes #68 — when navigating between duplicate groups in **Single** view, returning to **Multi** view (back button, Esc, or Multi button) now shows the group that was active in Single view instead of the group last open in Multi view. ## Cause Exiting Single view only synced tag metadata on the existing Multi comparison grid. Group navigation in Single view updates the active duplicate tag but intentionally skips reloading the multi grid while the viewer is open. ## Fix - Track `_dupe_multi_displayed_tag` when the multi grid is loaded. - On exit from Single view: refresh the multi grid when the active group differs from the displayed group; otherwise keep the lightweight tag-only sync. - Apply the same displayed-tag check in `_on_dupe_multi_requested`. ## Testing - `test_dupe_single_exit_returns_active_group_multi_view` (new regression test) - `test_dupe_viewer_exit_skips_full_dupe_list_rebuild` (unchanged — same-group exit still avoids full dupe list rebuild) - `ruff check` / `ruff format --check` ## Release - Version **0.1.27** - CHANGELOG updated

Grok added 2 commits

2026-06-13 01:59:42 +00:00

fix: restore correct dupe group when exiting Single to Multi view

Windows Release / prune-dev-release (push) Has been skipped

Details

Windows Release / build-release (push) Successful in 8m44s

Details

63c18ea17e

Track which duplicate group the Multi comparison grid last displayed.
When returning from Single view (back, Esc, or Multi button), refresh
the grid if the active group changed while navigating in Single view.

Fixes issue #68.

docs: PR-prep for issue 68 — release 0.1.27

Windows Release / prune-dev-release (push) Has been skipped

Details

CI / lint (pull_request) Successful in 7s

Details

CI / nuitka-smoke (pull_request) Successful in 1m36s

Details

CI / test (pull_request) Successful in 1m53s

Details

Windows Release / build-release (push) Successful in 8m49s

Details

6b90359a3b

Bump VERSION and document the Dupe Review Single-to-Multi group restore fix.

ValleyGeek merged commit f6fc340916 into main

2026-06-13 02:00:02 +00:00

ValleyGeek deleted branch 0.1.26/issue-68-dupe-multi-return

2026-06-13 02:00:03 +00:00

ValleyGeek referenced this pull request from a commit

2026-06-13 02:00:05 +00:00

Fix Dupe Review Multi view restoring wrong group after Single navigation (fixes #68) (#71)

No reviewers

No milestone

No project

No assignees

1 participant

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!71

No description provided.

Rows
Columns