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

Keyboard-only user focus on Jupyter logo is just a vertical line not bounding box around the element they are focused on. #6927

Closed
Tracked by #6800
tonyfast opened this issue Jun 16, 2023 · 12 comments · Fixed by #6938

Comments

@tonyfast
Copy link
Collaborator

tonyfast commented Jun 16, 2023

Issue Area # 2

  • Expectation: All supplementary elements that a user needs to interact with receive keyboard focus and be reached using only the keyboard. (WCAG Criteria 2.1.1 (A))
  • Observed:
    • On both light and dark theme, when focused on the Jupyter logo in header, the focus indicator is just a vertical line on the left side. This is not sufficient and needs to either surround the logo or the logo needs to be visually and noticably altered.
@tonyfast
Copy link
Collaborator Author

a#jp-NotebookLogo does not take on the width of the svg contained. adding margin: auto wraps the focus around the logo. the z-index needs to be elevated to see the right side of the focus.

@jtpio jtpio added this to the 7.0.x milestone Jun 19, 2023
@andrii-i andrii-i removed the status:Needs Triage Applied to issues that need triage label Jun 20, 2023
@Cheukting
Copy link
Contributor

Cheukting commented Jun 20, 2023

Screenshot 2023-06-20 at 14 53 46

While investigating, I also realized the tab stops are a bit messy, the most problematic of all, IMO is that I cannot swap between "FIles" and "Running"... need to be fixed as well (It seems more complicated)

I think it is reported as #6928

@Cheukting
Copy link
Contributor

The logo does not select with tab at all in Firefox

@tonyfast
Copy link
Collaborator Author

@Cheukting thanks for opening #6938!

im not seeing the same tabbing pattern you are. what version are you testing? im on Version: 7.0.0rc0.

image

The logo does not select with tab at all in Firefox

to be honest, i feel like the logo should be removed from the tabs in general. it doesn't really hold a meaningful action.

@Cheukting
Copy link
Contributor

Screenshot 2023-06-20 at 14 53 46 While investigating, I also realized the tab stops are a bit messy, the most problematic of all, IMO is that I cannot swap between "FIles" and "Running"... need to be fixed as well (It seems more complicated)

I think it is reported as #6928

This one is Chrome:
Version 114.0.5735.133 (Official Build) (arm64)

@tonyfast
Copy link
Collaborator Author

which version of notebook is this happening on?

@Cheukting
Copy link
Contributor

Cheukting commented Jun 20, 2023

Screenshot 2023-06-20 at 18 26 22

This is what my Firefox does, using:
114.0.2 (64-bit)

As you see, no tab for logo

@Cheukting
Copy link
Contributor

which version of notebook is this happening on?

7.0.0rc0

@tonyfast
Copy link
Collaborator Author

tonyfast commented Jun 20, 2023

The logo does not select with tab at all in Firefox

the tab order is a calculation from the last place of focus. the logo will not appear in the tab order if focus has been set to an element after it. the logo does appear in the tab index if the tab order is measured for the url location. you'll have to start with focus before the logo to measure its tab stop.

@Cheukting
Copy link
Contributor

Screen.Recording.2023-06-20.at.18.35.36.mov

The logo still does not appear even after selecting the URL first

@tonyfast
Copy link
Collaborator Author

oh wow. yea im still getting the focus line to the left of the logo on firefox. either way i prefer the behavior i'm seeing on your machine haha. i don't think the logo will often get clicked or needs focus. even when you click it, it opens another tree which is not what i would have expected.

@Cheukting
Copy link
Contributor

Cheukting commented Jun 20, 2023

@tonyfast Unfortunately one of the ui-test on Firefox failed for PR #6938 That's why I was checking it manually in the first place and realized that it happened even on the 7.0rc - I don't know if I know why it failed. If you have experience maybe you can give me a hint over there? Thank you for looking at this with me.

jtpio pushed a commit that referenced this issue Jun 21, 2023
* Make jupyter logo select properly (#6927)

* auto formation
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants