Skip to content

fix(ui): disabled primary chip, transaction badge, MCP sizes, ERD toolbar row - #1482

Merged
ZhuchkaTriplesix merged 2 commits into
devfrom
issue/1480-ui-review-fixes
Oct 11, 2026
Merged

ZhuchkaTriplesix merged 2 commits into
devfrom
issue/1480-ui-review-fixes

Conversation

@ZhuchkaTriplesix

Copy link
Copy Markdown
Member

Part of #1480 (the issue stays open for the items below).

Fixed

  • 8: a disabled primary / destructive button is a grey chip (muted surface, mutedForeground label) instead of a half-transparent accent whose label was hard to read. Other disabled variants keep the fade.
  • 9: the SQL toolbar's transaction badge reads Auto-commit / Manual commit before the first run, not Transaction —.
  • 3: the MCP section buttons are md, like the rest of Preferences.
  • 4 (partly): switch rows no longer inset their label by 4 px.
  • 1: below 1100 px the ERD toolbar's Note, Auto layout and the hidden-tables / group buttons are icon-only (tooltips keep the names), so Export no longer wraps alone.

Not done

  • 2 (lighter canvas rectangle in the light theme), 7 (menu ~12 px narrower than its trigger), 6 (Browse… is 1.5 px shorter than the path field), the slider label alignment in 4, and 5 (the ghost Close of Preferences stays: the standard says ghost). Each needs a look at a build rather than a guess.

Test: a disabled primary has no Opacity, a disabled secondary keeps it.

… ERD toolbar, switch row inset

- A disabled filled button uses the muted surface and label instead of a
  half-transparent accent.
- The SQL toolbar badge reads Auto-commit / Manual commit before the first
  run instead of 'Transaction —'.
- MCP section buttons are md like the rest of Preferences.
- Switch rows no longer inset their label by 4 px.
- Below 1100 px the ERD toolbar's Note, Auto layout and hidden-tables
  buttons are icon-only, so Export does not wrap alone.

Refs #1480
@github-actions github-actions Bot added bug Something isn't working frontend Theme parser epic label: frontend ui User interface components and widgets ux User experience, keyboard shortcuts and interactions P3 Low priority / Polish & Enhancements labels Oct 11, 2026
@ZhuchkaTriplesix
ZhuchkaTriplesix merged commit ea74feb into dev Oct 11, 2026
15 checks passed
@ZhuchkaTriplesix
ZhuchkaTriplesix deleted the issue/1480-ui-review-fixes branch October 11, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working frontend Theme parser epic label: frontend P3 Low priority / Polish & Enhancements ui User interface components and widgets ux User experience, keyboard shortcuts and interactions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant