Skip to content

Design polish: radius scale, tabular numbers, shared icons, busy states - #207

Merged
nobrega8 merged 2 commits into
mainfrom
design-polish-checklist
Nov 8, 2026
Merged

nobrega8 merged 2 commits into
mainfrom
design-polish-checklist

Conversation

@nobrega8

@nobrega8 nobrega8 commented Nov 8, 2026

Copy link
Copy Markdown
Owner

Fixes from a 20-point UI checklist audit.

  • Radii (14): 17 ad-hoc values collapse to `--r-xs/sm/md/lg/xl` (4/6/8/10/14) so nested corners follow inner = outer − padding.
  • Tabular numbers (18): `font-variant-numeric: tabular-nums` on the root, so every column, counter and label lines up.
  • Icons (12): the 6 inline SVGs with different viewBox/stroke (play, stop, reset, refresh, external link, heart) now use `Icon.svelte`. Select chevron/check and the library chevrons are left as they are.
  • Headings (16): `text-wrap: balance` on h1–h4, `pretty` on paragraphs.
  • Layout animation: the scan progress bar scales with `transform` instead of animating `width`. Toaster and `smooth-height` still animate height on purpose.
  • Loading (19): buttons with `aria-busy` show a small ring (Import, Install and restart).
  • Italics (3): removed from the log note rows and the custom-register group label.

Not changed: badges, text contrast on low-opacity text and the accent colour are design calls, and several checklist points need the app running to judge.

`npm run check` and `npm test` pass (247 tests). I did not run the app, so the visual changes are unchecked on screen.

…p, busy buttons

Radii snap to a five-step scale (--r-xs/sm/md/lg/xl) so nested corners follow inner = outer - padding. Digits are tabular everywhere. Inline SVGs become Icon.svelte glyphs (one viewBox and stroke). Headings balance and paragraphs avoid orphans. The scan progress bar animates transform instead of width. Buttons show a ring while busy. Decorative italics removed.
@nobrega8
nobrega8 merged commit 4b9a9c5 into main Nov 8, 2026
7 checks passed
@nobrega8
nobrega8 deleted the design-polish-checklist branch November 8, 2026 20:29
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.

1 participant