Skip to content

UI polish: CompactToggleButtons#8344

Open
LuniMoon wants to merge 3 commits intomasterfrom
CompactToggleButtonsFix
Open

UI polish: CompactToggleButtons#8344
LuniMoon wants to merge 3 commits intomasterfrom
CompactToggleButtonsFix

Conversation

@LuniMoon
Copy link
Collaborator

@LuniMoon LuniMoon commented Mar 2, 2026

Use padding instead of margin for button spacing, make separator stretch to full height, and apply rounded corners only to the first and last buttons for a cleaner grouped appearance.

Use padding instead of margin for button spacing, make separator
stretch to full height, and apply rounded corners only to the
first and last buttons for a cleaner grouped appearance.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@LuniMoon LuniMoon requested a review from 4ian as a code owner March 2, 2026 12:09
Adjusting behavior to clearer UI patterns.
@4ian
Copy link
Owner

4ian commented Mar 2, 2026

Are you happy with this border?
image

It seems less "polished" or a bit awkward compared to what we had before:
image

Possibly because of this extra pixel between the gray separator and the colored button, leaving 1px of black background:
image

@LuniMoon
Copy link
Collaborator Author

LuniMoon commented Mar 2, 2026

Good point.

I'll tweak that so it gets closer to the original design:
Screenshot 2026-03-02 at 14 32 52

Micro space fixing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants