lsst-epo / lsst-epo/epo-react-lib
Missing aria-disabled property on button components
- Dominant language
- TypeScript
- Stars
- 4
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
## Context
In the [`OrbitalDetails` component](https://github.com/lsst-epo/epo-react-lib/blob/develop-v3/packages/epo-widget-lib/src/widgets/OrbitalSim/Orbitals/OrbitalDetails.jsx) we are passing a boolean to the `isInactive` prop on the `Styled.ButtonWrapper` that seems to be intended to set the `aria-disabled` value on the rendered button.
## Issue
The `aria-disabled` value of this prop is not being set on the "Show Details" button. It is possible that the issue lies with the underlying button component, but some discovery is required.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in packages/epo-widget-lib/src/widgets/OrbitalSim/Orbitals/OrbitalDetails.jsx and trace the Styled.ButtonWrapper used by the “Show Details” button. Inspect the underlying button component to determine why isInactive does not produce aria-disabled, then verify that the rendered button exposes the expected aria-disabled value.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- accessibility, frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100