Repository navigation
feat(gtd): show GTD states first in the move picker - #658
Open
salmonumbrella wants to merge 1 commit into
Open
salmonumbrella wants to merge 1 commit into
salmonumbrella wants to merge 1 commit into
Conversation
salmonumbrella
force-pushed
the
feat/gtd-move-picker
branch
from
October 20, 2026 05:37
34bc906 to
6463770
Compare
With GTD on, every move picker now leads with a GTD section (Todo, Delegated, Watch, Reference, Someday) above Recent. Picking a state tags the message the way the t/d/w/v/b keys do: the Inbox copy stays, any other state is dropped and Undo works. The GTD folders no longer appear a second time as plain folders, where clicking one did an IMAP move that skipped Done and the one-state-per-thread rule. Core gains a move-picker-sections collector and buildMovePickerSections, which replaces the Recent/Favorites/All block copied across the reader dropdown and mobile sheet, the context menu and both bulk pickers. GTD contributes through the collector; core imports no GTD code. Bulk selections tag one message at a time through classifyManyWithUndo with one toast and one Undo. With GTD off the pickers render as before. If the folder list fails to load, the GTD section still shows above the "No folders" message. Bulk tagging tags each message once, and its Undo is dismissed when one of its threads later switches state. The mobile reader sheet draws one line between sections instead of two. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
salmonumbrella
force-pushed
the
feat/gtd-move-picker
branch
from
October 20, 2026 14:34
6463770 to
d7b0cb1
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@maathimself, with GTD on, the move picker now starts with the GTD states.
What changed
Why
The GTD folders were listed as ordinary folders, so clicking one moved the message out of the Inbox without going through Done, and a thread could end up in two states. They now appear once, as tags.
Usage
No usage change.
Closes #657
Frontend only. Tests, lint and build pass, and I checked all five pickers with GTD on and off against a mocked API.
Contributor License Agreement
By submitting this pull request I confirm that:
third-party material and confirmed it is compatible with the CLA).