This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit R242:52c0a136f9ff: [ExpandableListItem] make it touch-friendly
(authored by ngraham).
REPOSITORY
R242 Plasma Framework
ngraham added a comment.
Good question. I'm not sure it's possible here; the list views that use this
component as a delegate might have to learn how to unset the selection with
their own `TapHandler`. I'm not 100% sure on that though.
REPOSITORY
R242 Plasma Framework (Library)
REVISION
niccolove added a comment.
This generically works, except for: would it be possible to make it so that
clicking on an empty point will drop the list and button hover status?
otherwise, you kind of get stuck to this situation:
F8242311: Screenshot_20200416_224831.png
niccolove added a reviewer: niccolove.
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D28808
To: ngraham, #plasma, apol, #vdg, niccolove
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
ngraham updated this revision to Diff 80218.
ngraham added a comment.
Simpler enabled handling
REPOSITORY
R242 Plasma Framework (Library)
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D28808?vs=80217=80218
BRANCH
make-ExpandableListItem-touchable (branched from master)
ngraham edited the summary of this revision.
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D28808
To: ngraham, #plasma, apol, #vdg
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
ngraham updated this revision to Diff 80217.
ngraham added a comment.
Use TapHandler instead of MultiPointTouchArea
REPOSITORY
R242 Plasma Framework (Library)
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D28808?vs=80054=80217
BRANCH
make-ExpandableListItem-touchable (branched
ngraham planned changes to this revision.
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D28808
To: ngraham, #plasma, apol, #vdg
Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns
ngraham created this revision.
ngraham added reviewers: Plasma, apol, VDG.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
ngraham requested review of this revision.
REVISION SUMMARY
Use a MultiPointTouchHandler as the top-level item to handle touch input.