-
Notifications
You must be signed in to change notification settings - Fork 29.4k
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
Test: Settings Editor Navigation #103908
Comments
From the two options presented here, I vote for the |
I like the first option For the second option, I didn't understand if I should use tab or arrow keys to go through settings. When focus is on the TOC, pressing tab puts focus on the whole container and I am able to tab forward through the settings once. Then pressing shift tab loses focus and I start using my arrow keys to go through the settings. Also, it felt like there's an extra step to change the setting values, once I arrow key to the setting I want, I have to tab to focus on the setting's value and then press enter to change it. |
@lszomoru can you expand on what you like or dislike? |
@roblourens, apologies for not getting back to you as this has gotten lost among my notifications. Personally, here is my flow when I try to look for a setting:
I feel that with pagination there is just too many navigational elements on the page that you can juggle to limit the list of settings. |
Refs: #97567
Complexity: 4
Create Issue
I have been exploring the setting editor this month. I have been mostly focused on two different approaches and I would like to get feedback on both of these. Keep in mind that these aren't polished from a design perspective, so you should focus on providing feedback on the functional differences in both of them.
Issues that we are trying to address
Also copy these keybindings before testing:
1. Non-virtual list with pagination
Settings UI is made up of multiple non-virtual lists. Each category in the table of contents maps to a single non-virtual list. Example: Only the settings in the "Commonly Used" section are rendered when the user presses enter while selecting the "Commonly Used" section in the table of contents. Users navigate settings as a form and tab through the entire page. As each category can have a lot of settings (100+) we decided to paginate this list. It also makes navigating using a tab easier.
Builds
Windows x64: https://az764295.vo.msecnd.net/insider/c9a2f78283b6e5ef708fb8869e2a5adaa476e42f/VSCode-win32-x64-1.48.0-insider.zip
macOS: https://az764295.vo.msecnd.net/insider/c9a2f78283b6e5ef708fb8869e2a5adaa476e42f/VSCode-darwin-insider.zip
Linux x64: https://az764295.vo.msecnd.net/insider/c9a2f78283b6e5ef708fb8869e2a5adaa476e42f/code-insider-1596174438.tar.gz
2. Virtual list with up/down arrow navigation
Settings UI is made up of multiple virtual lists. Each category in the table of contents maps to a single virtual list. Users navigate settings as a normal list and use up and down arrow keys to navigate from setting to setting. Page-up and page-down keys also work as expected.
Builds
Windows x64: https://az764295.vo.msecnd.net/insider/c39d543796fde371e3e1855293b54703070cb2b5/VSCode-win32-x64-1.48.0-insider.zip
macOS: https://az764295.vo.msecnd.net/insider/c39d543796fde371e3e1855293b54703070cb2b5/VSCode-darwin-insider.zip
Linux x64: https://az764295.vo.msecnd.net/insider/c39d543796fde371e3e1855293b54703070cb2b5/code-insider-1596533482.tar.gz
The text was updated successfully, but these errors were encountered: