Tree: double the disclosure chevron size#22
Merged
Merged
Conversation
The ▸/▾ markers were text-xs in a w-4 box — hard to see and visually too small a target. Now text-lg (2x) centered in a w-6 box. (The whole row is the actual click/toggle target; this is a visibility fix.) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011TrhtF1SiJghJASMWECzBh
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.
The ▸/▾ disclosure markers rendered at
text-xsinside aw-4box — visually tiny and hard to aim at. Nowtext-lg(≈2×) centered in aw-6box. Note the whole row is the actual click/toggle target since the headless-tree migration, so this is a visibility/affordance fix rather than a hit-target one. Tests unaffected (8/8).🤖 Generated with Claude Code
https://claude.ai/code/session_011TrhtF1SiJghJASMWECzBh
Greptile Summary
This PR enlarges the tree disclosure chevron (
▸/▾) fromtext-xsin aw-4container totext-lgin aw-6flexbox container, improving visual visibility of the expand/collapse indicator. The fix also addsleading-noneto prevent the larger font size from inflating row height, andflex items-center justify-centerto center the Unicode triangle character within its now-wider box.truncate, so this is handled gracefully on narrow layouts.Confidence Score: 5/5
A focused visual tweak to one CSS class string — no logic, data, or API changes. Safe to merge.
The change touches a single span's Tailwind classes: the chevron container grows from 16 px to 24 px wide and the character scales from text-xs to text-lg, with flex centering and leading-none added. The label already uses truncate, so the slightly narrower text area is handled gracefully. No state, props, or behaviour is altered.
No files require special attention.
Important Files Changed
Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A[Tree Row div\nflex items-center gap-1] --> B[Chevron span\nw-6 flex items-center justify-center\ntext-lg leading-none] A --> C[Label span\ntruncate text-sm] A --> D[Badge optional\nshowCounts && isFolder] B --> E{isFolder?} E -- yes --> F{isOpen?} E -- no --> G[empty string] F -- yes --> H[▾ expanded] F -- no --> I[▸ collapsed]%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%% flowchart TD A[Tree Row div\nflex items-center gap-1] --> B[Chevron span\nw-6 flex items-center justify-center\ntext-lg leading-none] A --> C[Label span\ntruncate text-sm] A --> D[Badge optional\nshowCounts && isFolder] B --> E{isFolder?} E -- yes --> F{isOpen?} E -- no --> G[empty string] F -- yes --> H[▾ expanded] F -- no --> I[▸ collapsed]Reviews (1): Last reviewed commit: "Tree: double the disclosure chevron size" | Re-trigger Greptile