feat(differ): add keyboard shortcuts for file navigation and font sizing#502
Conversation
rnc505
force-pushed
the
robbyc/03-26-feat_differ_add_keyboard_shortcuts_for_file_navigation_and_font_sizing
branch
from
b8f7aca to
800f525
Compare
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 800f52578c
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
apps/differ/src/App.svelte
Outdated
| if (si && !args.mode) { | ||
| setMode('stack'); | ||
| } |
There was a problem hiding this comment.
After await loadRepoInfo(), this block immediately calls setMode('stack'), which triggers a second loadDiff() while the first loadDiff() (started inside loadRepoInfo) is still in flight. Because loadDiff() has no request token/cancellation, whichever response finishes last wins; on slower repos the earlier "All Changes" request can overwrite the stack results, leaving diffMode === 'stack' but showing the wrong file list. The same pattern is also used in handleFolderSelect, so the mismatch can occur both on startup and when switching repos.
Useful? React with 👍 / 👎.
rnc505
force-pushed
the
robbyc/03-26-feat_differ_add_keyboard_shortcuts_for_file_navigation_and_font_sizing
branch
3 times, most recently
from
7fc1449 to
c0d93bf
Compare
Fix several UI density and overflow issues: - Replace hardcoded 9px font sizes with var(--size-xs) for section labels, dividers, and count capsules - Bump mode-seg buttons from --size-xs to --size-sm for readability - Fix titlebar overflow: add ellipsis truncation on branch name and repo button, allow titlebar-center to shrink - Increase stack dropdown padding for better touch targets - Add UI Font Size preference (--size-base) with stepper control in Settings panel, persisted to Tauri store Amp-Thread-ID: https://ampcode.com/threads/T-019d28f5-71eb-7256-af52-1b4c406e53a7 Co-authored-by: Amp <amp@ampcode.com> Amp-Thread-ID: https://ampcode.com/threads/T-019d28f5-71eb-7256-af52-1b4c406e53a7 Co-authored-by: Amp <amp@ampcode.com>
Add app-level keyboard shortcuts for faster review workflow: - ] / [ to navigate to next/previous file in the diff list - Cmd+= to increase both UI and code font size by 1 - Cmd+- to decrease both UI and code font size by 1 - Font size changes are reflected live in Settings panel - File nav shortcuts are suppressed when focus is in inputs
rnc505
force-pushed
the
robbyc/03-26-feat_differ_add_keyboard_shortcuts_for_file_navigation_and_font_sizing
branch
from
c0d93bf to
57300b6
Compare
2 participants
Stack
This is PR 4/4 in a stack of Differ improvements:
Background
Differ lacked keyboard shortcuts for common review actions, requiring mouse interaction to navigate between files or adjust font size.
Overview of Changes
]/[to navigate to next/previous file in the diff listCmd+=/Cmd+-to increase/decrease both UI and code font size by 1pxTesting Performed
Video testing (with all 4 from stack)
gtinstalledgtinstalledScreen.Recording.2026-03-26.at.10.38.26.AM_compressed.mp4
Screen.Recording.2026-03-26.at.10.41.16.AM_compressed.mp4
Fixed the text overflow artifacts that you can see in the video
I just didn't want to rerecord
✨🤖 heavily assisted by AmpCode