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

'Zoom in' not working as keybind #28

Closed
hariseldon137 opened this issue May 20, 2023 · 5 comments
Closed

'Zoom in' not working as keybind #28

hariseldon137 opened this issue May 20, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@hariseldon137
Copy link

No description provided.

@csparker247
Copy link
Contributor

csparker247 commented May 20, 2023

Are you on a Mac or a keyboard where you have to hold shift to get the +?

From the PR:

Depending on your keyboard layout, you may have to somewhat modify the above commands. For example, my keyboard has the + key as a super to the = key, changing the Zoom In shortcut to Ctrl + Shift + =. Modifying the shortcuts for all keyboard layouts is not convenient or easy, so consider this a limitation.

@csparker247 csparker247 added the bug Something isn't working label May 20, 2023
@hariseldon137
Copy link
Author

I'm on a PC running linux, and yes I have to hold shift to get the +. (is there a keyboard that isn't like that??)

@hariseldon137
Copy link
Author

Ah ok yes ctrl- to zoom out and ctrl+shift= to zoom in. Thanks.

@csparker247
Copy link
Contributor

Yes, there are many different keyboard layouts and it can be in different places. As it currently is coded, it should work if you hold Shift as well or use the + on a numpad (which I know is dumb).

Anyway, this is an issue with how Qt has chosen to implement the standard keyboard action for the ZoomIn concept. I personally think there's a bug, but they don't seem to agree or at least don't think it's a priority.

The obvious solution for VC is hard code it to Ctrl + =, but that's no necessarily portable across keyboard layouts, so I chose to do it the official Qt way for the time being in lieu of implementing #23. Holding Shift for a little while didn't seem like that big of a deal, but let me if that doesn't work or if it's a major inconvenience.

@hariseldon137
Copy link
Author

hariseldon137 commented May 21, 2023

Well whatever position the various commands are in, a segmenter's hand sits in that position for the better of 8 hours a day. So anything awkward is quickly compounded. Imagine a gamer who's WASD is mapped to CTRL-SHIFT-=, and ALT, and so on. Additionally since we move the dots with the mouse, it makes the most sense to have all the commands over on the left side of the keyboard, otherwise the keyboard is hanging off the edge of your desk. I see you are going to make it user-mappable, so I'll wait for that (or reprogram my keyboard with QMK).

The only requirements are that the hand needs to be able to rest in a neutral position on the left side of the keyboard for the zoom, movement (what the alt key does), and impact range. (zoom is least important)

Another helpful command would be to have another bind to move up or down 100 slices, in addition to the current 1x and 10x. Would really help people find good pages to segment.

One last low priority thing would be to make the enter key work as 'start' in the Ending Slice box, the way enter works in the choose slice box.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants