Skip to content

Conversation

@DiljohnSingh
Copy link
Contributor

Pull request type

Bug fix (non-breaking change which fixes an issue)

Screenshot 2025-11-14 at 12 27 36

Description

  • Team discovered an accessibility issue with slider. Slider handle is not linked to the label
  • Added the missing aria-labelledby using the id provided
  • Updated the snapshots

What should be covered while testing?

  • You should see aria-labelledby with the same value as id of associated label component.
  • Test cases should be passing.
  • Turn on Screen reader and navigate to handle on any slider. It should announce the label text.

@DiljohnSingh DiljohnSingh requested a review from a team as a code owner November 14, 2025 11:33
@DiljohnSingh DiljohnSingh force-pushed the fix/Accessibility-issue-slider-ariaLabelledBy branch 2 times, most recently from 2526891 to fb9aeb7 Compare November 28, 2025 08:45
@DiljohnSingh DiljohnSingh force-pushed the fix/Accessibility-issue-slider-ariaLabelledBy branch from fb9aeb7 to 655b5e6 Compare November 28, 2025 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant