-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Comments
|
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 |
The logo does not select with tab at all in Firefox |
@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.
to be honest, i feel like the logo should be removed from the tabs in general. it doesn't really hold a meaningful action. |
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) This one is Chrome: |
which version of notebook is this happening on? |
7.0.0rc0 |
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. |
Screen.Recording.2023-06-20.at.18.35.36.movThe logo still does not appear even after selecting the URL first |
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. |
@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. |
Issue Area # 2
The text was updated successfully, but these errors were encountered: