-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Notebook 7 prerelease keyboard navigation review #6595
Comments
Hello everyone, many thanks for @isabela-pf for providing the keyboard navigation review of Notebook 7! I am happy to announce that we have received a development grant from @InseeFr for 8 weeks of work to address the keyboard navigation issues of the Jupyter notebook! Hopefully, we will be able to resolve most (or all) of the issues that were discovered by Isabela as part of this review. On a separate note, we are aware of the work that @gabalafou has already done on fixing the tab traps in the JupyterLab repository at jupyterlab/jupyterlab#14115. If this PR has not been merged when we start working on this assignment, it will definitely be a priority to help get this work integrated in JupyterLab (review, debugging, etc). Upon finishing this assignment, we would love to connect with you on the possibility of writing a joint blog post showing progress on these subjects, the multi-stakeholder effort on accessibility, and crediting the audit done by Isabela. |
@SylvainCorlay wonderful news! Does the 8 weeks mean that there's a specific start date or deadline on the work or a specific timeframe in which it has to be done? Just wondering in terms of planning. |
We have to spend this budget before the end of December. EDIT: but I think we will start earlier. |
|
Thanks @isabela-pf for reporting this.
Is this still the case ? On my side the focus can be seen every time, in the file browser or the Notebook view.
Do you know if there are suggestions about this when using inline editors ? I think the standard navigation keys are
Does it also concern Notebook 7 or only Jupyterlab ? Don't know if we can split the main area in Notebook, and if it is expected.
I can't reproduce it, on my side there is no "unknow" tab, even when the header is hidden. |
the tab order doesn't take me to any hidden elements either. maybe something got fixed?
there is a splitter region in the current notebook main area for extensions. the ARIA authoring practice guide has recommendations for implementing a window splitter. the one splitter in notebook is probably requires different considerations than window splitters in lab.
lots of scope creep can happen at the intersection of authoring (ATAG) and WCAG. some inline widgets with features like autocomplete recommend esc implementations. the closest APG pattern would be to think about the notebook or file editor interfaces as part of a complex grid layout interface. in this pattern,
i was able to find focus each time, too. maybe its all good at the moment.
the inconsistent focus is confusing when navigating around the headers. this is likely a big meta issue. theres a lot of different focus treatments across the interface. testing for focus is likely something we could do though. this would be a good piece of automation. sara soueidan's focus indicators post is always worth mentioning when discussing focus |
As a part of reviewing the prerelease of Notebook 7 for accessibility issues, we teamed up to run some manual keyboard navigation tests on this Notebook 7 binder.
Many thanks to @ohrely, @gabalafou, @krassowski, @vidartf, @steff456, and @afshin for authoring this review!
Please note that this format of reporting the full review via issue is new and possibly a little messy. Please let me know if there's anything that would make it easier to understand or manage. Thanks!
Takeaways
Based on the full review, I'm going to briefly issues that stood out.
Content order
In the file browser
Areas to navigate
lm-mod-active
class on last active item"esc
key. This isn't standard keyboard navigation behavior.Keyboard/tab traps
esc
key. This isn't standard keyboard navigation behavior.Skip links
Focus
Mixed input
In the file browser
Keyboard shortcuts
Full review
This is the full list of prompts and responses from our review session.
When accessed via keyboard, the content order is logical. (WCAG 2.2 Focus order)
Does the content order follow reviewer expectations based on reading the content?
Result: It goes from top to bottom and left to right, which makes sense for an English keyboard. I am able to loop from the end of the document back to the top, so that meets my expectations as well.
I cannot identify the content order in its entirety because of the lack of visible focus indicator (see below). The content order may make sense, but I cannot tell.
Does the content order make sense for interacting with the content?
Result: The content order on the file browser goes as follows: 5 unknowns, File, View, Settings, Help, Share, 3 unknowns, Upload, Refresh the file browser, Filter, File browser (can navigate list of files with arrow keys). This is a reading order that makes sense for the page. The main issues are the "unknowns" (where I had to use the tab key that number of times with no visible focus state in order to find myself on an interactive area again) and areas that are missed entirely.
Are any major content areas missed when navigating via keyboard?
Result: While the whole content order is not entirely clear based on this evaluation, we did notice areas missing in the file browser. Missing areas included
Are any interactive content areas missed when navigating via keyboard?
Result: See above.
Other notes or recommendations:
Areas to navigate
All areas of the tool can be accessed using keyboard navigation. (WCAG 2.2 Keyboard)
lm-mod-active
class on last active itemOther notes by area:
Keyboard/tab traps
When navigating via keyboard, there are no areas where keyboard focus can enter but not exit. The focus never gets "trapped." (WCAG 2.2 No keyboard trap)
From the above section (Areas to navigate), please list all areas where the reviewer could not navigate out of using the keyboard. Notes are welcome!
Keyboard/tab traps:
Other notes or recommendations:
Skip links
When using keyboard navigation, there is a link to switch keyboard focus directly to the tool's main content and skip header navigation or repeated content. (WCAG 2.2 Bypass Blocks)
Is there a skip link?
Result: There does not appear to be one in either the file tree page or the notebook page. I inspected html source in browser, but quick searches found nothing.
Is the skip link prompt visible?
Result: No visible prompt on pages tested.
What does the skip link prompt skip?
Result: N/A
Where does the skip link prompt move user focus to?
Result: N/A
Does the skip link behavior meet reviewer expectations?
Result: No; there should be skip link(s) to skip over menu bar(s), straight to main page content.
Other notes or recommendations: https://github.com/ offers a basic working example of skip links for comparison.
Interactive Areas
All buttons, links, form fields, or similar interactive areas can be both accessed and activated using only the keyboard. (WCAG 2.2 Keyboard)
Other notes by area:
Focus
There is a visible focus state for all content. It appears by default when navigating via keyboard. (WCAG 2.2 Focus visible)
Result: pressing the Tab key repeatedly moves the focus around the interface, but it doesn't give focus to all the elements of the UI. There's no focus to the sidebars items, though items inside the opened sidebar are getting focus.
2. If end of content cannot be reached, the reviewer may report on what they could interact with.
Result: The end content can be reached, and it will circle back to the starting point of the document.
3. If the reviewer is comfortable, they may also use the browser inspector to select an area and toggle focus per area.
Is there visible focus styling? If so, please briefly describe it.
Result:
Is the focus styling consistent throughout?
Result:
Are there any areas without visible focus styling?
Result: The change from sidebar to the notebook and afterwards is tricky because there's no visible focus styling.
Was the reviewer ever unclear about where their keyboard focus was during the review? If so, please note where or why.
Result:
Some items are not focused because the user will not be able to make any action, but it makes the interaction weird. For example, the only item that you are able to get focus in the status bar is the first one (which is the only one that has an action associated). Also the focus changes in the different type of documents that you can open in the main content area.
Other notes or recommendations:
Mixed input
It is possible to complete tasks while switching between input mechanisms, for example using a keyboard and mouse and touch screen. (WCAG 2.2 Concurrent input mechanisms)
Files
andRunning
tabs.Task name + region
Does the task have multiple steps? If yes, please list them.
Result:
Can the task be completed with only keyboard controls?
Result:
Can the task be completed with only mouse controls?
Result:
Can the task be completed with only touch screen contorls?
Result:
Keyboard shortcuts
Keyboard shortcuts--key combinations that initiate an action-- may not inhibit keyboard navigation. Keyboard shortcuts also need to be configurable. (WCAG 2.2 Character key shortcuts)
List of shortcuts for this tool.
Can shortcuts be turned off?
Result: Not in the file browser or notebook UI, we should add this feature
Can shortcuts be configured (remapped to different keys than the default)?
Result: Not in the file browser or notebook UI, we should add this feature
Do any shortcuts conflict with browser or operating system controls?
Result: Yes, but in relevant situations (e.g., for copy and paste), this seems okay
Other notes or recommendations:
Shortcut name
Shortcut keys:
How many keys are needed?
Result:
How close together are the keys?
Result:
Does the shortcut only activate when focus is on the related part of the interface?
Result:
Other notes or recommendations:
The text was updated successfully, but these errors were encountered: