Differential D7013

Middle-click on QSystemTrayIcon сauses context menu
ClosedPublic
Actions

Authored by zerg on Jul 31 2017, 9:15 AM.

Details

Reviewers
davidedmundson
Group Reviewers
Plasma
Commits
R854:abdc4c0013a4: Probably moot: Middle-click on QSystemTrayIcon сauses context menu
R135:b0059b1c8342: Middle-click on QSystemTrayIcon сauses context menu
Summary

Middle-click on QSystemTrayIcon сauses context menu because inconstant with
http://code.qt.io/cgit/qt/qtbase.git/tree/src/platformsupport/dbustray/qstatusnotifieritemadaptor.cpp?h=5.7#n177
QStatusNotifierItemAdaptor::SecondaryActivate()

Diff Detail

Repository
R135 Integration for Qt applications in Plasma
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
zerg created this revision.Jul 31 2017, 9:15 AM
McPain added a subscriber: McPain.Jul 31 2017, 10:09 AM
McPain added a reviewer: Plasma.Jul 31 2017, 10:53 AM
McPain set the repository for this revision to R135 Integration for Qt applications in Plasma.
McPain added a project: Plasma.
davidedmundson accepted this revision.Jul 31 2017, 10:53 AM
davidedmundson added a subscriber: davidedmundson.

Do you have commit access?

If not can you list your real name + email address.

This revision is now accepted and ready to land.Jul 31 2017, 10:53 AM
zerg added a comment.Jul 31 2017, 12:18 PM
In D7013#130424, @davidedmundson wrote:

your real name + email address.

"Sergey V Turchin" <zerg@altlinux.org>

cfeck added a subscriber: cfeck.Aug 1 2017, 7:27 PM

BUG: 382855

Closed by commit R135:b0059b1c8342: Middle-click on QSystemTrayIcon сauses context menu (authored by zerg, committed by davidedmundson). · Explain WhyAug 9 2017, 9:23 PM
This revision was automatically updated to reflect the committed changes.
volkov added a subscriber: volkov.Jan 2 2018, 9:03 PM

Is there a reason why KDEPlatformSystemTrayIcon is supported?
Why don't drop it and rely on Qt's dbus tray implementation?

Restricted Application added a subscriber: plasma-devel. · View Herald TranscriptJan 2 2018, 9:03 PM

Revision Contents
Changeset List

Diff 17949

View Options

src/platformtheme/kdeplatformsystemtrayicon.cpp

Loading...