-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Labels
FrontendRelates to the clientRelates to the clientTechnical DebtCleaning up after ourselvesCleaning up after ourselvesa11yIssues with Accessibility or UsabiltiyIssues with Accessibility or Usabiltiy
Description
Our Button and Borderless Button components both accept a "disabled" prop that gets passed to the underlying <button> element, but that only prevents the onClick handler from being fired -- it doesn't change the styling to signal to the user that the can't click the button, and the :hover and :active pseudo-classes still get applied so it just looks like something is broken.
We should, at a minimum, set cursor: not-allowed on :hover. We can also switch to a "ghosted" background/text for those buttons to provide a passive visual indicator, and one that will apply on mobile.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
FrontendRelates to the clientRelates to the clientTechnical DebtCleaning up after ourselvesCleaning up after ourselvesa11yIssues with Accessibility or UsabiltiyIssues with Accessibility or Usabiltiy