-
Notifications
You must be signed in to change notification settings - Fork 64
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
Hide cursor on touch events. #52
base: master
Are you sure you want to change the base?
Conversation
This might be a silly question (I got no touch pad to test this), but isn't the touch pad basically a mouse-replacement, so when it get activated, then the mouse pointer should NOT be hidden? |
It is a mouse replacement. However I'd consider the cursor part of the mouse user scheme. It really doesn't add anything to a touch experience since you are not trying to move a cursor around with your finger but to just hit a thing on your screen and for that action to get executed. I don't see how having a virtual keyboard would be different from hitting any other button on your screen. If you are still wondering I can also record a video later for you to see the difference |
I think this needs to be put behind an option to enable it. |
AtomToast, thanks for the explanation. I was not thinking about touch screens at all. Makes perfect sense there to hide the pointer. |
I now added the option to keep the cursor after there was a touch event. I still think the correct default behaviour should be to hide it when using a touch screen |
based on #25 but with fixed indentation
works perfectly on my gpd win max