kmaterka added a comment.
In D22804#503733 <https://phabricator.kde.org/D22804#503733>, @broulik wrote: > > Yes, probably onPressed should be handled in PasmoidItem, not in the AbstractItem to avoid this kind of problems. > > Indeed, that would be the best solution. Should this be done together with this change? I shouldn't be hard, I can move onPressed to PlasmoidItem, which will simplify the fix also? >> I would personally opt for consistency, though. As far as I know GTK is showing context menu on mouse click, not pressed. The same is for MS Windows (this does not mean we must follow this or reimplement Windows, no). What is your opinion? > > Gimp, Firefox, Chrome, Gedit, etc, they open it on press here, and so does Qt and the rest of KDE software, so this must be consistent to here, regardless of what someone else does. Oh, right! I focused on tray icons only... That makes perfect sense. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D22804 To: kmaterka, #plasma_workspaces Cc: broulik, apol, plasma-devel, #plasma_workspaces, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
