Conversation
Documentation build overview
|
|
Nice little shortcut, but I'm concerned this will interfere with keyboard handling for people using assistive tech such as screen readers. https://www.w3.org/WAI/ARIA/apg/practices/keyboard-interface/ includes:
https://webaim.org/techniques/keyboard/ has examples where |
|
@hugovk now this gets tricky - DevGuide actually uses collapsible navigation tree for menu, but.. it doesn't conform to the guidelines itself. I tried several way to expand it on https://devguide.python.org/getting-started/ - right/left/space don't work, tab focus doesn't jump to down arrow. Enter just navigates to the section index page. So the menu tree is not ARIA compatible, and enabling keyboard navigation won't break it. If somebody decides to make the devguide tree element ARIA compatible, the setting doesn't prevent more specific key handler to do the ARIA thing and prevent default popping to more generic document handler, where the shortcuts are defined. |
|
Thanks @abitrolly and @hugovk. I've opened an issue about accessibility #1792. I am closing this PR as not planned. |
|
@willingc please reopen. The accessibility was an argument that this PR would break it, but now that we've determined that there is nothing to break, I see no reason to keep keep these keyboard shortcuts disabled. They come from Spinx anyways. |
|
We haven't determined there's nothing to break. As I said at python/cpython#148751 (comment), some screen readers use left and right keys, and I think any interception of them could break accessibility. This isn't proven (I only tested VoiceOver on macOS, which isn't affected), but it's not proven the other way either. |
This makes it easy to browse the doc without mouse.
https://pradyunsg.me/furo/customisation/#navigation-with-keys