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

Improve, expand, and document keyboard shortcuts #1202

Open
4 tasks
klemay opened this issue Apr 9, 2021 · 8 comments
Open
4 tasks

Improve, expand, and document keyboard shortcuts #1202

klemay opened this issue Apr 9, 2021 · 8 comments

Comments

@klemay
Copy link

klemay commented Apr 9, 2021

Currently, users can annotate selected text by typing a to create an annotation or h to create a highlight. This meets WCAG guidelines but could use some improvement and new additions. For example:

  • Enabling/disabling the browser extension
  • Opening and closing the sidebar
  • Moving from document to sidebar & vice versa
  • Interacting with an annotation: Editing, Replying, Flagging, and Deleting
  • Editing an annotation: toggle between preview and edit mode

Notes

  • All of these commands should have some kind of modifier (CTRL or CMD). We should also ensure that these do not interfere with commonly used keyboard shortcuts in the browser or other assistive technology.
  • Keyboard shortcuts should be documented in the KB

To Do:

  • Make a list of "actions" someone would need to do within the app
  • Propose a set of keyboard shortcuts for each of these options (might be helpful to research commonly used shortcuts to avoid)
  • Meet with our contacts at Benetech to validate those suggestions
  • Get this issue on our product backlog to be prioritized among our other work
@mattdricker
Copy link

Example of potential for conflict between shortcut keys: Using keyboard shortcut 'h' with PDF changes pointer to Hand Tool

@mattdricker
Copy link

mattdricker commented May 26, 2021

Proposed list of actions to trigger with a keyboard shortcut:

  • Activate Hypothesis extension
  • Open/close sidebar
  • Navigate to next/previous annotation card
  • Navigate to next/previous selected document text
  • Search

In Editor:

  • Toggle between write & preview
  • Post to group
  • Post to Only Me
  • Cancel

[DONE]

  • Annotate a
  • Highlight h

@mattdricker
Copy link

mattdricker commented May 26, 2021

Some additional notes:

  • Navigate to next/previous annotation card
  • Navigate to next/previous selected document text

It may make sense for these to use same KB shortcut, with the difference being whether sidebar is open or shut.

Also, if a user has navigated to a selected text in the document, opening the sidebar should transfer that focus to the related annotation card. Vice versa with an annotation card selected when the sidebar is closed.

@jeremydean
Copy link

Adding a detail I noted during the OSU call.

The "h" shortcut for "Highlight" conflicts with the common use of "h" as a shortcut to navigate "Headers."

While there is a "pass through mode" in screen readers to avoid this conflict, not all screen reader users will be that sophisticated.

@mattdricker you had mentioned something about work we had planned on "modifier keys" that might address this.

@mattdricker
Copy link

@jeremydean yes, we will plan on using modifier keys (e.g. ctrl + alt) with all of our kb shortcuts to avoid these conflicts. Modifier keys currently TBA.

@mkdir-washington-edu
Copy link

Discussion on this topic will likely solve https://github.com/hypothesis/support/issues/201.

@chrisshaw
Copy link

Issue was linked to Harvestr Discovery: Use only my keyboard

@tripluoffthoranlz
Copy link

Some additional notes:

  • Navigate to next/previous annotation card
  • Navigate to next/previous selected document text

It may make sense for these to use same KB shortcut, with the difference being whether sidebar is open or shut.

Also, if a user has navigated to a selected text in the document, opening the sidebar should transfer that focus to the related annotation card. Vice versa with an annotation card selected when the sidebar is closed.

Hi @jeremydean ,

Do you know if they've gotten anywhere on "Navigate to next/previous annotation card?"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants