On touch events the mouse cursor gets hidden, on next mouse event the
mouse cursor is shown again. This IMHO significantly improves the system
interaction if touch is the primary user interface.
Details
Details
- Reviewers
zzag davidedmundson - Group Reviewers
KWin - Commits
- R108:070b46f45566: Hide mouse cursor when interacting with touch screen
Test case added and ctest passes
Diff Detail
Diff Detail
- Repository
- R108 KWin
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Actually nowadays I consider adding all the spies and filters to input.cpp a mistake. I find it very cumbersome to read this file and it makes it impossible to unit test the filters (some could be unit tested instead of only integration testing).
Comment Actions
Looks good to me. (Please keep in mind that my decision is totally based on the provided test, I didn't test this patch on a physical device)
Off topic:
This file is part of the KDE project.
I've been wondering for a long time whether this statement is correct. (KDE is no longer a project, it's a community)