| apol | |
| niccolove |
| Plasma | |
| VDG |
Right now ExpandableListItem is not very touch-friendly due to use of a mousearea
hover effect for choosing the selected item. This patch fixes that by using a TapHandler
to handle left-clicks and touch input. We still need a MouseArea inside it to handle
non-touch hover and right-click behaviors.
FEATURE: 393749
FIXED-IN: 5.70
Click behavior is identical to how it was before, but now touch works too:
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
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:
(ps yay, first time I actually manage to try a patch and review it. I'm getting better!)
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.