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

"Top of Page" link in template footer bug #11

Closed
3 tasks
isabela-pf opened this issue Sep 12, 2022 · 0 comments · Fixed by #49
Closed
3 tasks

"Top of Page" link in template footer bug #11

isabela-pf opened this issue Sep 12, 2022 · 0 comments · Fixed by #49
Labels
test 1: navigation Related to the first round of user testing with navigation emphasized

Comments

@isabela-pf
Copy link
Contributor

Problem and context

This issue comes from our user testing round 1: navigation. We ran this round of tests with a notebook that uses the Space Telescope style guides' example tutorial which includes elements to be used in every public STScI notebook. One of the elements this template provides is a footer, which includes a "Top of Page" link for easy navigation.

While few participants used this link to navigate, some who did noted that the link was "broken." In trying to reproduce it myself, I found that while the link is accurate, clickable (participants who tried were able to interact with it), and will visually scroll a user back to the top, it doesn't seem to move keyboard focus. For users relying on focus to give them feedback on where they are on the page, this means the link does not work because their point of reference has not moved.

Possible solutions

There is only one solution for this from a user experience perspective: the link needs to change the keyboard focus to an element at the top of the page. In terms of implementation, I would imagine this is linking to the top cell. There may be a more standard approach that I am not aware of at the time of writing this.

Acceptance criteria

  • Merge a PR that allows the "Top of page" link to move keyboard focus to the top of the page. This fix should be tested with a screen reader.

Tasks to complete

  • Explore for standard methods of changing focus in top of page links
  • Make code changes where needed (probably to the template)
  • Get work merged
@isabela-pf isabela-pf added the test 1: navigation Related to the first round of user testing with navigation emphasized label Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test 1: navigation Related to the first round of user testing with navigation emphasized
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant