Skip to content

ARXIVCE-4340: Accessibility remediations for search templates#369

Open
SBBCornell wants to merge 1 commit into
developfrom
ARXIVCE-4340-accessibility-audit
Open

ARXIVCE-4340: Accessibility remediations for search templates#369
SBBCornell wants to merge 1 commit into
developfrom
ARXIVCE-4340-accessibility-audit

Conversation

@SBBCornell

@SBBCornell SBBCornell commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix invalid aria-role attribute to role="search" on simple search form
  • Add role="search" to advanced search form
  • Convert abstract expand/collapse from inaccessible <a onclick> to keyboard-accessible <button> elements with aria-expanded state
  • Fix aria-current="page" to only apply to the actual current page in pagination
  • Fix invalid disabled attribute on pagination "Next" link (use aria-disabled + tabindex="-1")
  • Add aria-label to category tag tooltips so screen readers announce full category names
  • Update fieldset.js to maintain <label for> associations when cloning search term rows
  • Add role="status" live regions so screen readers announce results count
  • Add role="alert" to form validation error notifications

WCAG criteria addressed

  • 1.3.1 Info and Relationships (A)
  • 2.1.1 Keyboard (A)
  • 3.3.1 Error Identification (A)
  • 4.1.2 Name, Role, Value (A)
  • 4.1.3 Status Messages (AA)

🤖 Generated with Claude Code

Fix invalid aria-role attribute, add role="search" to forms,
convert abstract expand/collapse to keyboard-accessible buttons,
fix pagination aria-current to only mark current page, fix invalid
disabled attribute on pagination links, add aria-labels to category
tooltips, update fieldset.js to maintain label associations on
cloned rows, add role="status" live regions for results count,
and add role="alert" to form error notifications.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants