Skip to content

Review: a newer list starts at its first job until you've moved - #59

Merged
voidReq merged 3 commits into
mainfrom
review-fresh-list
Oct 24, 2026
Merged

voidReq merged 3 commits into
mainfrom
review-fresh-list

Conversation

@voidReq

@voidReq voidReq commented Oct 24, 2026

Copy link
Copy Markdown
Owner

Fixes the macOS CI failure in tests/test_gui.py (Review after an undo didn't show the undone job first).

Review shows the list it made last at once, then swaps in the fresh one; the swap kept the card on screen, so on a slow machine the old list's first job stayed. Now, until you decide / skip / go back, the fresh list starts at its first job; after that it keeps your place as before.

🤖 Generated with Claude Code

voidReq and others added 3 commits October 24, 2026 16:01
The quick list's first card stayed when the fresh list came in, so a job that now comes first
(e.g. one just undone back to Proposed) showed up later. On slow machines the fresh list lags,
which is why test_gui failed on macOS CI.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A sheet check asked for while another read was running was dropped, and that read may have
started before the change; the rows stayed old until the next scheduled read (test_gui on macOS).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@voidReq
voidReq merged commit 4e470e4 into main Oct 24, 2026
6 checks passed
@voidReq
voidReq deleted the review-fresh-list branch October 24, 2026 20:40
@voidReq voidReq mentioned this pull request Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant