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 4 times, most recently from 655b5e6 to b46e536 Compare December 2, 2025 09:05
@DiljohnSingh DiljohnSingh force-pushed the fix/Accessibility-issue-slider-ariaLabelledBy branch from b46e536 to 18cf572 Compare December 31, 2025 12:25
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