Fix: Add accessible labels to filter select elements in Abilities Exp…#642
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Unlinked AccountsThe following contributors have not linked their GitHub and WordPress.org accounts: @trushiv@example.com. Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases. If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #642 +/- ##
==========================================
Coverage 74.50% 74.50%
Complexity 1740 1740
==========================================
Files 85 85
Lines 7522 7523 +1
==========================================
+ Hits 5604 5605 +1
Misses 1918 1918
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…lorer (#642) Fixed - Added accessible labels to the provider and category filter dropdowns in the Abilities Explorer page. Co-authored-by: Trushiv04 <trushiv@git.wordpress.org> Co-authored-by: dkotter <dkotter@git.wordpress.org>
What?
Closes #641
Adds accessible labels to the provider and category filter
<select>elements in the Abilities Explorer screen.Why?
The two filter dropdowns ("All Providers" and "All Categories") had no accessible name,
aria-label, oraria-labelledby. Screen readers could not announce what each dropdown controls, which is a WCAG 2.1 AA violation (SC 1.3.1 and SC 4.1.2).How?
Added a visually hidden
<label class="screen-reader-text">before each<select>, linked via theforattribute matching the existingid. This uses WordPress's standardscreen-reader-textpattern, so there is no visual change — the labels are only exposed to assistive technology.Use of AI Tools
AI assistance: Yes
Tool(s): Claude
Used for: Identifying the issue location, drafting the label markup, and verifying coding standards. Final implementation was reviewed and tested by me.
Testing Instructions
/wp-admin/tools.php?page=ai-abilities-explorer)<select>now has an associated<label>and that a screen reader announces its purposeChangelog Entry