feat(differ): fix font sizing, padding, and titlebar responsiveness#501
Conversation
rnc505
force-pushed
the
robbyc/03-26-feat_differ_fix_font_sizing_padding_and_titlebar_responsiveness
branch
from
5c0d882 to
03c0b79
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: 03c0b79539
ℹ️ 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-tauri/src/lib.rs
Outdated
| let output = Command::new("gt") | ||
| .args(["state"]) | ||
| .current_dir(repo) |
There was a problem hiding this comment.
The new stack detection calls gt state and assumes JSON on stdout, but the current Graphite command reference states it “documents every command” and does not include a state subcommand (it lists gt log, gt parent, etc. instead). On current CLI versions this call will fail or return non-JSON, causing get_stack_info to return None and the stack mode UI to never appear even when users are in a Graphite-managed repo.
Useful? React with 👍 / 👎.
rnc505
force-pushed
the
robbyc/03-26-feat_differ_fix_font_sizing_padding_and_titlebar_responsiveness
branch
3 times, most recently
from
033518a to
6d2ef73
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>
rnc505
force-pushed
the
robbyc/03-26-feat_differ_fix_font_sizing_padding_and_titlebar_responsiveness
branch
from
6d2ef73 to
2b84547
Compare
2 participants
Stack
This is PR 3/4 in a stack of Differ improvements:
Background
Several UI elements in Differ used hardcoded font sizes and lacked overflow handling, causing visual inconsistency and layout breakage with long branch names or repo paths.
Overview of Changes
9pxfont sizes withvar(--size-xs)for section labels, dividers, and count capsules--size-xsto--size-smfor readabilitytitlebar-centerto shrink--size-base) with stepper control in Settings panel, persisted to Tauri storeTesting 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