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

NewHotkeys #29

Merged
merged 4 commits into from
Sep 4, 2022
Merged

NewHotkeys #29

merged 4 commits into from
Sep 4, 2022

Conversation

OloRatuj
Copy link
Contributor

Add 3 new hotkeys for the prev and next page switch, and for the insert coordinates function

New hotkeys were added for prev and next pages as well as insert coordinates.
@rkusa
Copy link
Owner

rkusa commented Jul 28, 2022

Thanks for the PR!

Do you need the next/prev page shortcuts for VoiceAttack, or are they something you actually reach for when interacting with the keyboard yourself?

I wonder whether it'd be worth making the following adjustments to the insert coordinate shortcut:

  • only work while the insert coodinate mode is active
  • if it only works when the insert coordinate mode is active, use a simpler and easier to reach shortcut (like a simple Space)

Wdyt?

@OloRatuj
Copy link
Contributor Author

Hi,
I use the next/prev when in VR and use these keys with the throttle or switch box under my hand. So, the shortcuts are beneficial for reading pages that I have pretty many (around 10). This way I use the Scratchpad as a regular kneeboard with a lot of static information available under my hand. Inserting coordinates is a second priority function for me.

So, it is not only for insert coordinate mode and I would like to have them active all the time.

@rkusa rkusa merged commit a2cf651 into rkusa:main Sep 4, 2022
@rkusa
Copy link
Owner

rkusa commented Sep 4, 2022

Thanks for the PR! I've made the following changes:

  • renamed the config keys (added hotkey prefix),
  • made them optional (they are not set by default)

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