[A11y] Make screen readers start with packages list on Search Page. by dannyjdev · Pull Request #9465 · NuGet/NuGetGallery · GitHub
Skip to content

[A11y] Make screen readers start with packages list on Search Page.#9465

Merged
dannyjdev merged 2 commits into
devfrom
dev-dj-a11y-focussearchresults
Apr 19, 2023
Merged

[A11y] Make screen readers start with packages list on Search Page.#9465
dannyjdev merged 2 commits into
devfrom
dev-dj-a11y-focussearchresults

Conversation

@dannyjdev

Copy link
Copy Markdown
Contributor

Changes

  • Gives focus to results-column element to tell screen readers were to start if the Apply button or the Search button were clicked.
  • uses localStorage to track if those buttons were clicked before the page refresh.

Addresses: #9422

@dannyjdev dannyjdev requested a review from a team as a code owner April 12, 2023 02:43
Comment thread src/NuGetGallery/Scripts/gallery/page-list-packages.js
joelverhagen
joelverhagen previously approved these changes Apr 18, 2023
@dannyjdev dannyjdev dismissed stale reviews from joelverhagen and keylime-unicorn via c0dd1ad April 19, 2023 19:32
@dannyjdev dannyjdev merged commit 13a10ba into dev Apr 19, 2023
@dannyjdev dannyjdev deleted the dev-dj-a11y-focussearchresults branch April 19, 2023 19:57
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.

3 participants