-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
Fix issues with pointer visibility duration and click delays #10
Comments
I might suggest that it will be a great idea if you will add a "long press" enable \ disable option in the app and should only be used when needed. Thanks for your work! |
@4388450 thanks for the suggestion |
Thanks. Because in the last few hours I had a lot of issues on certain apps because the long click option. So I will really appreciate if you can add a option to disable it |
Pointer visibility Long press enable disable fixed by ae5ca99 in the TV version Closing this issue. |
the pointer is visible but it doesn't perform the click until you release the button |
@KFMDMSolutions , do you think it'd be more intuitive if the click is performed as soon as the button is pressed instead of button release? Any ideas on how to distinguish between a short press and a long press in this case? |
lets say i wanna record a 5 min voice message it definably needs some delay to differentiate between a short press and a long press |
you can start the click after a 2 sec or more delay and stop it onkeyup |
makes sense, although the clicks sent by the gesture API, don't have that kind of leverage. Android will need to provide something else altogether in order for us to deal with this issue. This functionality exists in Android for sure (they use it with the inbuilt remotes and touchscreens), but haven't exposed the API for use by third-party apps like mine. What you're describing will become possible with #11 |
Point 2 of #8 (comment)
and https://gitter.im/virresh/community?at=60729023ae90f36840733a3d
sums up the issue.
Need to debug and fix timers on posting of runnables.
The text was updated successfully, but these errors were encountered: