Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Facility Locator pagination Prev/Next links do not scroll to top #14895

Closed
3 tasks
laflannery opened this issue Aug 22, 2023 · 2 comments
Closed
3 tasks

Facility Locator pagination Prev/Next links do not scroll to top #14895

laflannery opened this issue Aug 22, 2023 · 2 comments
Labels
a11y-defect-2 Serious accessibility issue that should be fixed in the next 1-2 sprints accessibility Issues related to accessibility Defect Something isn't working (issue type) Facilities Facilities products (VAMC, Vet Center, etc) Facility Locator product owned by Facilities team sitewide VA.gov frontend CMS team practice area

Comments

@laflannery
Copy link
Contributor

laflannery commented Aug 22, 2023

Description

When using the Prev/Next links on the pagination on Facility Locator, the screen should load with the top of the results in view so the veteran can find their place at the top of the list on the next or previous page. Focus should also be at the "Showing X - XX of XX results" so folks using assistive technology can easily orient themselves to the new list of results.

Currently, the behavior scrolls to the bottom of the page and the pagination numbers are at the top of the view.

See this Slack comment for video example.

The pagination & focus behavior should match Find a Form paging.

User story

AS A Veteran
I WANT the page to scroll to the top when I use Prev/Next links in paginated lists
SO THAT I can easily find my place in the list on the next page

Slack thread for context on related issue in VAMC Events: https://dsva.slack.com/archives/C52CL1PKQ/p1691683090666189

Acceptance criteria

  • When the user activates Prev/Next links, the page should load with the top of the results list in view (as Find a Form currently does)
  • Focus should go to "Showing XX - XX of XX results" when the user activates Prev/Next links (as Find a Form currently does)
  • Accessibility review
@laflannery laflannery added the Needs refining Issue status label Aug 22, 2023
@laflannery laflannery changed the title <Insert summary of task> Facility Locator pagination Prev/Next links do not scroll to top Aug 22, 2023
@laflannery laflannery added Defect Something isn't working (issue type) VA.gov frontend CMS team practice area Facilities Facilities products (VAMC, Vet Center, etc) Facility Locator product owned by Facilities team accessibility Issues related to accessibility a11y-defect-2 Serious accessibility issue that should be fixed in the next 1-2 sprints UX labels Aug 22, 2023
@xiongjaneg
Copy link
Contributor

xiongjaneg commented Sep 19, 2023

Currently, it's an odd interaction. Laura is noticing a quick jump to the footer and then back to the top of the page. This is intermittent. Could be a page loading issue. Potentially timebox an exploration of this inconsistent behavior. And will watch in accessibility testing later.

@davidmpickett davidmpickett removed the UX label Sep 21, 2023
@xiongjaneg xiongjaneg removed the Needs refining Issue status label Sep 25, 2023
@laflannery
Copy link
Contributor Author

Retested this and this is now resolved, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-defect-2 Serious accessibility issue that should be fixed in the next 1-2 sprints accessibility Issues related to accessibility Defect Something isn't working (issue type) Facilities Facilities products (VAMC, Vet Center, etc) Facility Locator product owned by Facilities team sitewide VA.gov frontend CMS team practice area
Projects
None yet
Development

No branches or pull requests

4 participants