lsst-epo / lsst-epo/epo-react-lib

Missing aria-disabled property on button components

Open
#406 0 comments 0 reactions 0 assignees View on GitHub
a11y bug
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.