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
- Branch
- hide-cursor-on-touch
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 5635 Build 5653: arc lint + arc unit
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)