[Enhancement]: Keep Selected Dupe Group Button On Screen #113

New issue

Closed

opened 2026-06-21 23:10:48 +00:00 by ValleyGeek · 2 comments

ValleyGeek commented

2026-06-21 23:10:48 +00:00

Owner

Affected area

Duplicate Group Toolbox

Current behavior

As duplicate groups are reviewed the duplicate group toolbox stays in a fixed location

Desired improvement

When a duplicate group is reviewed scroll the duplicate group toolbar down to keep the newly active group button 40% from the top.

References

No response

### Affected area Duplicate Group Toolbox ### Current behavior As duplicate groups are reviewed the duplicate group toolbox stays in a fixed location ### Desired improvement When a duplicate group is reviewed scroll the duplicate group toolbar down to keep the newly active group button 40% from the top. ### References _No response_

ValleyGeek added the

Kind/Enhancement

Priority/Medium

labels

2026-06-21 23:10:48 +00:00

Claude referenced this issue from a commit

2026-06-24 13:51:14 +00:00

feat(dupe): keep active group button on screen (#113)

Claude referenced this issue from a commit

2026-06-24 13:54:06 +00:00

docs: PR-prep — bump to 0.3.2 for issue #113 toolbox auto-scroll

Claude commented

2026-06-24 13:54:14 +00:00

Member

Implemented on branch 0.2.4/issue-113-keep-group-onscreen (pushed, stacked on 0.2.4/issue-136-keep-dropdown, not yet merged to main — awaiting PR):

The duplicate-group list (DupeContainer) now scrolls automatically whenever the active group changes, keeping the active group's button ~40% from the top of the visible list, clamped at the ends so it never scrolls past the first/last group.
Every existing way the active group changes — clicking a button, W/S keyboard prev/next, Keep/AoK auto-advance, programmatic selection — funnels through one shared method (_scroll_active_button_into_view()), so future navigation entry points (e.g. issue #131's upcoming "next unreviewed" button) inherit this behavior automatically.
The scroll-target math is a pure, unit-tested classmethod (_compute_scroll_target()).

Added tests/test_dupe_active_button_scroll.py (6 tests: pure-function math + click/keyboard/programmatic integration via qtbot). Full suite (593 passed), ruff, and the UI-exit smoke all pass. This is the third branch in the v0.3.0 tranche (VERSION 0.3.1 → 0.3.2).

Implemented on branch `0.2.4/issue-113-keep-group-onscreen` (pushed, stacked on `0.2.4/issue-136-keep-dropdown`, not yet merged to `main` — awaiting PR): - The duplicate-group list (`DupeContainer`) now scrolls automatically whenever the active group changes, keeping the active group's button ~40% from the top of the visible list, clamped at the ends so it never scrolls past the first/last group. - Every existing way the active group changes — clicking a button, `W`/`S` keyboard prev/next, Keep/AoK auto-advance, programmatic selection — funnels through one shared method (`_scroll_active_button_into_view()`), so future navigation entry points (e.g. issue #131's upcoming "next unreviewed" button) inherit this behavior automatically. - The scroll-target math is a pure, unit-tested classmethod (`_compute_scroll_target()`). Added `tests/test_dupe_active_button_scroll.py` (6 tests: pure-function math + click/keyboard/programmatic integration via `qtbot`). Full suite (593 passed), ruff, and the UI-exit smoke all pass. This is the third branch in the v0.3.0 tranche (`VERSION` 0.3.1 → 0.3.2).

Claude added the

Reviewed/Confirmed

label

2026-06-24 13:55:07 +00:00

Claude referenced this issue from a commit

2026-06-24 14:01:45 +00:00

feat(dupe): add Next Unreviewed button to group toolbox (#131)

Claude referenced this issue

2026-06-24 14:03:44 +00:00

[Enhancement]: GoTo Next Unreviewed Dupe Group Button #131

Claude referenced this issue from a commit

2026-06-25 00:38:01 +00:00