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

Graph navigation hotkeys #223

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Jongy
Copy link

@Jongy Jongy commented Jan 8, 2020

This PR adds 3 navigation hotkeys that can be used to navigate the graph quickly:

  1. Ctrl+U resets the zoom (like the existing "Reset Zoom" button)
  2. Ctrl+[ goes back to the previous zoom.
  3. Ctrl+] goes forward to the next zoom (can be used after going back from existing zoom).

I'm not sure the backward & forward logic is entirely correct, but what's "correct" here anyway... I tried to imitate Chrome and it feels quite intuitive to me, so I find it useful.

Basically, I tried to imitate how back & forward works in Chrome.
This feels quite right now.
@versable
Copy link
Contributor

versable commented Mar 3, 2020

If PR #198 gets merged, the previous/next zoom states can easily be implemented using the native back/forward Browser navigation (Alt+:arrow_left:, Alt+:arrow_right:) by simply replacing history.replaceState with history.pushState.

@Jongy
Copy link
Author

Jongy commented Mar 3, 2020

Yeah, seems so. I'll follow

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

Successfully merging this pull request may close these issues.

2 participants