Skip to content

feat(frontend): Wizard - improve and use more daisyUI components - #539

Merged
martin-mfg merged 3 commits into
stats-organization:masterfrom
marcalexiei:wizard-a11y
Sep 14, 2026
Merged

martin-mfg merged 3 commits into
stats-organization:masterfrom
marcalexiei:wizard-a11y

Conversation

@marcalexiei

Copy link
Copy Markdown
  • Text and number fields now use daisyUI's .input instead of hand-rolled Tailwind,
    and NumericSection adds validator so its existing min/max show a hint.
  • Fixed two contrast bugs in index.css:
    1. an override was suppressing daisyUI's focus ring.
    2. disabled text was at 2.5:1 contrast in the light theme (now 5.3:1 / 7.4:1 dark).
  • Replaced two hand-rolled pieces with what daisyUI already ships: the delete-account
    modal is a native <dialog> plus .modal, gaining Esc, focus trap and focus restore
    while dropping
    • useOutsideAlerter hook
    • createPortal
    • LoginBox's isOpaque prop
  • The card placeholders use .skeleton, removing react-loading-skeleton.
  • The sticky card panel on steps 3 and 5 was pinned at a fixed top-32, so the progress bar covered up to 57px of it.
    ProgressBar now exposes its measured height for the panels to stack against.
  • Section controls had no accessible name:
    Section takes a titleId for its heading and the five sections point at it with aria-labelledby.

Screenshots

Card position on step 3

Before After
before after

Disabled field

Before After
before after

Elsewhere

Dark theme Delete-account dialog
dark dialog

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

@marcalexiei is attempting to deploy a commit to the martin-mfg's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel

vercel Bot commented Sep 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
github-stats-extended-backend Skipped Skipped Sep 14, 2026 4:03pm UTC

Request Review

@vercel
vercel Bot temporarily deployed to Preview – github-stats-extended-backend September 14, 2026 16:03 Inactive
@martin-mfg
martin-mfg merged commit 3ca1ba7 into stats-organization:master Sep 14, 2026
7 of 8 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