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

[Bug] Menu key does not work on mods list (accessibility) #3445

Closed
Xetaxheb opened this issue Sep 4, 2021 · 7 comments · Fixed by #3446
Closed

[Bug] Menu key does not work on mods list (accessibility) #3445

Xetaxheb opened this issue Sep 4, 2021 · 7 comments · Fixed by #3446
Labels
Bug Something is not working as intended Easy This is easy to fix GUI Issues affecting the interactive GUI

Comments

@Xetaxheb
Copy link

Xetaxheb commented Sep 4, 2021

Background

  • Operating System: Windows 10
  • CKAN Version: 1.30.4
  • KSP Version: 1.12.2.3167

Have you made any manual changes to your GameData folder (i.e., not via CKAN)?
No

Problem

Describe the bug
Pressing the "menu key" (between windows and right control) does not bring up "Labels/Reinstall/Download Contents/Purge Contents"

Steps to reproduce
Click on any random mod in list
Press "menu key" (https://en.wikipedia.org/wiki/Menu_key)

Expected behavior
The right-click menu pops up for highlighted mod after releasing menu key, navigable with arrows+enter (which it already is)

@HebaruSan HebaruSan added Bug Something is not working as intended Easy This is easy to fix GUI Issues affecting the interactive GUI labels Sep 4, 2021
@HebaruSan
Copy link
Member

HebaruSan commented Sep 4, 2021

Thanks for the report. #3446 should have a test build soon if you'd like to try it, in the Checks tab under the Artifacts dropdown.

@Xetaxheb
Copy link
Author

Xetaxheb commented Sep 4, 2021

Yep works fine. One nuance that may warrant fixing if there's a simple solution:

  • Usually the box is supposed to open on/next to the selected control (at the moment it's opening at cursor position, which is totally fine too; it's just better for clarity; i would recommend the very center of "Name" field or the right edge of that same field at column border)

One other note: right clicking hyperlinks in metadata shows "copy link address" and doesn't function with menu key either, but enter opens it as a http/s handler anyway so it's still browser accessible and a non-issue

@HebaruSan
Copy link
Member

HebaruSan commented Sep 5, 2021

Thanks for testing!

  • Usually the box is supposed to open on/next to the selected control (at the moment it's opening at cursor position, which is totally fine too; it's just better for clarity; i would recommend the very center of "Name" field or the right edge of that same field at column border)

Yeah, I had it like that initially (and I noticed that Chrome does it that way), but then I thought that if the user did want to navigate the menu with the mouse, it would be annoying to have to travel the mouse across the screen to do that. Then I tried pressing the menu key in the Atom editor and it opened at the cursor, so I thought I had found precedent for doing that.

EDIT:

  • Notepad puts the top left corner in the geometric center of the window??
  • Windows Explorer positions the top left corner of the menu approximately on top of the selected item
  • Steam ignores the key
  • Chrome's menu moves to cover the selection if I select some text

So there appears to be no consistency, even among Microsoft's own applications. As expected when a common feature like this has to be implemented in applications rather than in system libraries.

One other note: right clicking hyperlinks in metadata shows "copy link address" and doesn't function with menu key either, but enter opens it as a http/s handler anyway so it's still browser accessible and a non-issue

Good catch, thanks, we ought to be able to add this there, too.

Note to self: We also have a right click menu in the manage game instances dialog, containing the option to open the game folder. And there are several link labels in the About dialog.

@HebaruSan
Copy link
Member

Any ideas for keyboard accessibility of the right click menu of the grid headers? There isn't really a way to focus the headers with the keyboard, and I don't know of an example of this in other applications.

@HebaruSan
Copy link
Member

There's a new test build at the same place that should cover all of those additional context menus.

@Xetaxheb
Copy link
Author

Finally had time to look at this (8a10141, 6 days ago), and the only things I can think of for accessibility left are:

  • adding a new menu at the top (file/settings/help) or maybe as a submenu under the settings menu: to enable/disable columns i.e. grid headers. just calling up the right-click menu itself would be fine.
  • an additionally good thing would be a second submenu, 'Sort By ->" to likewise sort per column instead of needing to click the headers; could just call the same event as clicking them. hacky? maybe, but if it's stupid and it works, it's not stupid.
  • making the launch/refresh/update/apply/filters/arrows accessible through tab cycling; or though a new menu at the top, with a separate menu for filters then
  • Toggling installed/auto-installed/update boxes with spacebar is unclear because the dotted line "selection" box is missing; you can see it used to be there Clean up mod list spacebar handling in GUI #2543 but I'm not seeing it anymore and I don't know if it's CKAN or my windows

@HebaruSan
Copy link
Member

  • adding a new menu at the top (file/settings/help) or maybe as a submenu under the settings menu: to enable/disable columns i.e. grid headers. just calling up the right-click menu itself would be fine.
  • an additionally good thing would be a second submenu, 'Sort By ->" to likewise sort per column instead of needing to click the headers; could just call the same event as clicking them. hacky? maybe, but if it's stupid and it works, it's not stupid.

That makes sense! Sounds like the kind of thing I'd look for a in a "View" menu. It could have two submenus, View → Columns with checkboxes and View → Sort by with radio buttons, as you suggest. I might want to do that in a future pull request to keep this one from expanding too much.

  • making the launch/refresh/update/apply/filters/arrows accessible through tab cycling; or though a new menu at the top, with a separate menu for filters then

That I think we can do simply by setting TabStop=true for the toolbar control, will look into adding that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is not working as intended Easy This is easy to fix GUI Issues affecting the interactive GUI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants