Repository navigation
Show open decisions as a deck and route blank documents to Decisions - #483
Merged
Merged
Conversation
MaggieAppleton
enabled auto-merge (rebase)
October 11, 2026 12:30
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Choosing the tab focuses the deck's current top; an explicit reveal brings the named card forward. Phones keep the list and save bar. The routing announcement now speaks only once the forced view has applied, and a window blur no longer clears the editing state. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Cards behind the top are inert, so tests bring a decision forward before answering it. A card answered or discarded in this visit now waits at the back of the deck instead of joining resolved history until the next visit. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The label now reads "1 open decision" or "N open decisions" while answered cards wait behind, and the resolved toggle reads as one phrase. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e suites Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…sions Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
MaggieAppleton
force-pushed
the
decisions-tab-deck
branch
from
October 11, 2026 12:45
5a10e44 to
3f3658c
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
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.
What changes
On desktop, the Decisions tab shows open decisions as a deck, and a blank document sends the Planner's questions there.
Screenshots
Desktop Decisions deck
A blank document routed to Decisions
Phone, unchanged list
flowchart LR Q[Planner asks a decision] --> B{Document has prose?} B -- yes --> D[Stay on Document] B -- no --> E{Reader editing or picked a tab?} E -- yes --> D E -- no --> R[Switch to Decisions once and announce it] R --> K{2+ open decisions?} K -- yes --> Deck[Deck, arrow keys shuffle] K -- no --> Single[Single card]Testing
decisionsDeck().harness-decisions-deck.e2e.ts, run with the fake Planner: routing, an editing reader, an explicit tab pick, arrow-key shuffling and resuming at the top card.ready()ine2e/room.tsuses main's attached-editor check, which also counts the editor that stays mounted behind Decisions.sidebar-decision-countsis kept and checks the hidden editor.bun run types, the unit tests and the design-contract check pass. The initial JS bundle is unchanged at 80306 B gzip.Slice 4 of the nested-documents work.
🤖 Generated with Claude Code