Skip to content

style(ui): unify button/link hover treatments and fix disabled states - #49

Merged
yungsamd17 merged 2 commits into
mainfrom
style/ui-button-link-polish
Oct 26, 2026
Merged

yungsamd17 merged 2 commits into
mainfrom
style/ui-button-link-polish

Conversation

@yungsamd17

@yungsamd17 yungsamd17 commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Replaces accent-fill + dark-text hover on all accent buttons with the
    Option C treatment (red wash + accent border + bright text), keeping
    WCAG AA contrast (no dark text anywhere).
  • Button SVGs now inherit the label color and are flex-aligned; .btn-fmt
    format labels no longer render in accent.
  • Disabled buttons no longer flash the accent border/bg on hover.
  • Convert/resize buttons no longer use the .ready accent state (only
    export/download buttons do), and image-resizer's resize button matches
    svg-to-png's convert button.
  • Privacy/terms links use a new .link-secondary-permanent variant
    (grey → bright on hover, always underlined).
  • Home search FAB repositioned to align with the topbar hamburger across
    breakpoints.

Touched areas

  • src/styles/global.css, tool.css, tools/icon-maker.css
  • src/pages/index.astro, privacy.astro, terms.astro, design.astro
  • src/pages/tools/{icon-maker,image-resizer,svg-to-png}.astro
  • src/components/ToolLayout.astro
  • src/scripts/tools/{image-resizer,svg-to-png}.ts, lib.ts

Testing

  • npx astro check (0 errors, 2 pre-existing hints)
  • npm run build (12 pages, clean)
  • Preview verified in browser

Built with beam-free in the OpenCode harness.

@yungsamd17
yungsamd17 merged commit 200bb68 into main Oct 26, 2026
2 checks passed
@github-actions

Copy link
Copy Markdown

✅ PR Checks passed

Typecheck (npx astro check) and production build both green.

@yungsamd17
yungsamd17 deleted the style/ui-button-link-polish branch October 26, 2026 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant