Skip to content

fix(web): clarify settings visual hierarchy - #1177

Merged
tiann merged 1 commit into
tiann:mainfrom
techotaku39:fix/settings-heading-hierarchy
Jul 26, 2026
Merged

fix(web): clarify settings visual hierarchy#1177
tiann merged 1 commit into
tiann:mainfrom
techotaku39:fix/settings-heading-hierarchy

Conversation

@techotaku39

Copy link
Copy Markdown
Contributor

Summary

  • establish a consistent typography hierarchy across settings sections, rows, choice groups, custom controls, and voice controls
  • remove redundant desktop category headings while preserving responsive page titles and a semantic page heading
  • normalize settings-card spacing, composer-toolbar insets, and the duplicated appearance label

Responsive behavior

  • mobile keeps an 18px page title and divider, with 16px section headings and 14px setting labels
  • desktop uses the persistent category navigation instead of repeating each category title and reserves symmetric scrollbar gutters to prevent horizontal shifts between pages

Compatibility

Testing

  • bun typecheck
  • bun run test:web
  • bun run build:web
  • bun run test:cli
  • bun run test:shared
  • git diff --check origin/main...HEAD

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Findings
None.

Summary
Review mode: initial
No issues found in the added/modified lines. Residual risk: visual hierarchy changes were reviewed from diff/source only; no browser screenshot pass was run here.

Testing
Ran git diff --check origin/main...HEAD. Not run: typecheck or test suites (automation review).

HAPI Bot

@tiann
tiann merged commit 8432349 into tiann:main Jul 26, 2026
2 checks passed
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