Also expose a method to show the context menu so plasmoids that implement a MouseArea that will eat the event can display it too.
Details
Details
Diff Detail
Diff Detail
- Repository
- R242 Plasma Framework (Library)
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 10178 Build 10196: arc lint + arc unit
Comment Actions
[testing with D20043] The context menu that is shown on press-and-hold seems to differ from the one shown when right-clicked:
Right-clicked:
Pressed-and-held:
And the press-and-hold version opens the Alternatives panel in the wrong place, with the wrong assortment of items:
Comment Actions
Convert to the containment coordinate system, otherwise we show the context menu of the applet at 0,0
Comment Actions
what happens on long tap on the desktop?
this introduces different behavior between long tap on panel and desktop
should it go in a sort of config mode instead?
Comment Actions
for now we could go with this...
can you make it work *only* when the applet's formfactor is horizontal or vertical?
so this codepath wouldn't touch the desktop case at all
Comment Actions
this patch is not "on desktop specific"
also, in the desktop area, long press always was select/move for plasmoids. if now it would open the context menu, there would be no way at all to move them around anymore
and in general i think on touchscreen there are very few places (tough some definitely are) where is acceptable to open a context menu, it's just not what the pattern ended up being
Comment Actions
We discussed it and decided long tap is not the way to go, will have to find alternative ways to access the options on the context menu on a case-by-case basis.
Comment Actions
That seems... quite sub-optimal to me.
Is all of this to preserve press-and-hold for moving widgets? Is there a centralized place for this discussion? If not I can create a Phab task for it.
Comment Actions
no, there isn't. If you can coordinate it through a phabricator task, I would really appreciate it.