Add session note history to the Capture input#49
Merged
ScottMorris merged 3 commits intomainfrom Apr 2, 2026
Merged
Conversation
**What:** - add session-only plain-text note history for the TUI Capture input - let `Ctrl+Up` and `Ctrl+Down` move through past notes without changing the existing Up and Down selection behaviour - preserve the current in-flight draft as the bottom of the history stack so it can be restored - add tests for the history state and shortcut recognition **Why:** - issue #37 asks for a way to revisit earlier Capture text without giving up the current queue navigation bindings - session-scoped note history keeps the implementation focused and predictable without adding persistence
**What:** - add `Ctrl+Up` and `Ctrl+Down` note-history guidance to the in-app help and shortcut hints - update the README and spec to describe session note history and draft restore behaviour **Why:** - the new history flow needs to be visible where users discover Insert mode shortcuts - matching docs and help text reduce the chance that the feature feels hidden or surprising
**What:** - let `Ctrl+Up` and `Ctrl+Down` move within multiline draft text before switching history entries at the top or bottom - truncate command palette rows to the popup width so descriptions do not spill past the right border - render multi-line note bodies as separate lines in the Status pane - add focused tests for the cursor-boundary, palette-width, and note-rendering cases **Why:** - the history navigation should feel like text editing first and history browsing second - the command palette overflow and flattened note bodies are visible TUI defects - these fixes make issue #37 more polished in day-to-day use
ScottMorris
added
bug
CI check summary
|
Contributor
Author
|
@codex review |
|
Codex Review: Didn't find any major issues. 🚀 ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
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.
Summary
User-facing changes
Ctrl+UpandCtrl+Downnow browse plain-text note history from the current TUI session without taking over the existing Up and Down queue navigation.Ctrl+UpandCtrl+Downmove within the current draft first and only switch history entries at the top or bottom.Documentation
Test plan
cargo fmt --checkcargo buildcargo clippy --workspace -- -D warningscargo testCloses #37.