Skip to content

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

Open
marcalexiei wants to merge 1 commit into
stats-organization:masterfrom
marcalexiei:wizard-a11y
Open

feat(frontend): Wizard - improve and use more daisyUI components#539
marcalexiei wants to merge 1 commit 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.

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