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/out doesn't work if plasma mouse gestures are set to RMB #155

Closed
alekksander opened this issue Jan 3, 2020 · 7 comments
Closed

Comments

@alekksander
Copy link

alekksander commented Jan 3, 2020

to be honest i'm not sure if it's something to report. RMB hold and move up/down is great way to control zoom of images, but it collides with plasma gestures (if also set to RMB) — making it impossible to use. i know there are plenty of options which include excluding gestures for specific applications, yet i decided to inform You (@easymodo) that this takes place, and perhaps it's wise in this situation to let user control which button to hold for zoom, just as other keys are controllable. thanks!

@easymodo
Copy link
Owner

easymodo commented Jan 3, 2020

good to know, thanks

@alekksander
Copy link
Author

this one got worse. i'm unable to use zoom despite plasma is set not to use any gestures in qimgv. i'll report to bugs.kde.org

@easymodo
Copy link
Owner

how did you set it to ignore qimgv?

honestly i can't even find an option. i know you can disable global shortcuts but not sure about khotkeys

@easymodo
Copy link
Owner

perhaps it's wise in this situation to let user control which button to hold for zoom

I don't want to change how basic mouse interaction works in qimgv (clicks etc). The code will just turn into a mess.
Also what would you change it to? Using side mouse buttons would be uncomfortable, as well as mouse wheel (I've thought about changing zoom to wheel, even tried using it for a month, but it just felt too clunky

@alekksander
Copy link
Author

it's not so obvious, i have to admit it, but the whole gesture idea got to me profoundly, so i had to dig it ;) they are improving plasma quite intensively lately, so perhaps there is a chance they will rewrite this one too
Screenshot_20200517_225054

about the zoom button i also think current solution is best way to do it. before i found Your app i was using one with wheel as zoom, and ←LMB RMB→, but that wasn't working well ever after months of practice. in my first comment i suggested to give user option how to zoom and that still might be a good option, but to hardcode walkaround for a kde bug is probably not the way to go. folks at kde has to fix this one at some point. i will post a link to bug report once i ask devs at irc and actually report it.

@alekksander
Copy link
Author

alekksander commented May 17, 2020

wait… do You notice how i have 4 gestures not grouped? i decided to group them and write same exclude rule for them just to see if it works… and viola! plasma is not blocking RMB any more, and zooming works again :)

Screenshot_20200517_232226

seems the whole issue here was my mistake, and not a bug of any sort. sorry for the trouble. closing.

@easymodo
Copy link
Owner

Well that's an interesting UX decision from kde devs 😏

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

No branches or pull requests

2 participants