Skip to content
Snippets Groups Projects

Accessibility: Update FilteredConnection to support better usage of labels

Merged Warren Gifford requested to merge tr/filtered-connection-placeholder into main

Created by: umpox

Description

We typically rely on the placeholder for accessible labels in a lot of places, e.g.: image

For further reading, see placeholders are not accessible. Tl;dr - placeholders aren't accessible to screen readers and are only useful before the user has entered text

Note We likely want this to enforce this for all <Input /> elements. There's not a clear solution here yet (as there's so many ways to label an element). The simplest solution is to always enforce that we have some label on these elements, issue here for that: https://github.com/sourcegraph/sourcegraph/issues/36645

Test plan

Tested locally with a screen reader

App preview:

Check out the client app preview documentation to learn more.

closes https://github.com/sourcegraph/sourcegraph/issues/35463 closes https://github.com/sourcegraph/sourcegraph/issues/35468

Merge request reports

Merged by avatar (May 7, 2025 1:06am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading