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

Reduce touch screen sensitivity #710

Closed
kieranc opened this issue Oct 1, 2021 · 4 comments
Closed

Reduce touch screen sensitivity #710

kieranc opened this issue Oct 1, 2021 · 4 comments
Labels
enhancement Enhancement to an existing app/feature

Comments

@kieranc
Copy link
Contributor

kieranc commented Oct 1, 2021

I find, and I don't think I'm alone, that the touch screen is extremely sensitive, probably since the new touch driver was merged. The new touch driver is clearly a huge step forward but I think it needs some simple limits putting in place to define a minimum length/distance of a swipe for example, or duration of a tap. I often find that I accidentally end up dragging down the notification shade when I try to tap a button, meaning that something with only a few pixels of movement was interpreted as a swipe rather than a tap, and I think it should behave differently.

@Riksu9000
Copy link
Contributor

Adding a minimum threshold is very easy, but I'm not sure it would make things better.

Due to the way LVGL handles touch, making swipes less sensitive would make clicks more sensitive, making it more likely for swipes to be recognized as clicks, and I don't think that would make things better overall.

The issue is that LVGL doesn't care if the touchpoint moved, it will always click on release anyway. In LVGL8 this would've clicked on the green button instead, which is better, but ideally it shouldn't activate a click on anything. #616

I find that clicks are too sensitive, but I think that's an issue with LVGL, and filtering the touch points before giving them to LVGL seems hacky and probably wouldn't work in all situations without bringing back different touch modes again.

VID_20211001_132744.mp4

@kieranc
Copy link
Contributor Author

kieranc commented Oct 1, 2021

Thanks for your input, I know you have a much better grasp of this than me! Given that upgrading to LVGL8 is going to be very complicated, would it be worth patching or backporting the LVGL event handler to give us better behaviour instead of filtering the data we pass to LVGL?

@Avamander
Copy link
Collaborator

I read the title as Enchantment and for a second I were very confused, it should probably be Enhancement.

@kieranc kieranc changed the title [Enchancement] Reduce touch screen sentitivity [Enchantment] Reduce touch screen sentitivity Oct 2, 2021
@Avamander Avamander changed the title [Enchantment] Reduce touch screen sentitivity Reduce touch screen sentitivity Oct 2, 2021
@Avamander Avamander added the enhancement Enhancement to an existing app/feature label Oct 2, 2021
@kieranc kieranc changed the title Reduce touch screen sentitivity Reduce touch screen sensitivity Oct 2, 2021
@ck-telecom
Copy link

ck-telecom commented Nov 16, 2021

the touch controllor should always updated(irq generated) when moving finger, or finger down and up, but do not know how to set the hardware register

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement to an existing app/feature
Projects
None yet
Development

No branches or pull requests

4 participants