Show a context menu on pressAndHold
Needs ReviewPublic

Authored by apol on Mar 28 2019, 2:14 PM.

Details

Reviewers
mart
Group Reviewers
Plasma
Summary

Depends on D20085

Diff Detail

Repository
R119 Plasma Desktop
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 10219
Build 10237: arc lint + arc unit
apol created this revision.Mar 28 2019, 2:14 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMar 28 2019, 2:14 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
apol requested review of this revision.Mar 28 2019, 2:14 PM
hein added a subscriber: hein.Mar 28 2019, 2:25 PM

I guess this brings us back to the old debate of a generic solution vs. patching all the UIs. I'm inclined to agree with this approach because it should probably be up to the specific UI what it wants to happen on the gesture.

It means we need to be careful how we use it, though. On the desktop we also have press and hold to engage applet move mode, which means it's already off-limits for applet context menus there. Is this change going to conflict with that wrt/ user expectations?

davidedmundson added inline comments.
applets/taskmanager/package/contents/ui/Task.qml
134

This item also has drag and drop

Will the context menu open mid-way through dragging?