KDE Plasma: Right-click on tray icon doesn't work #6545

michaelstingl · 2018-05-24T12:35:21Z

You need to right-click tray icon very often. No effect. If it worked once, then it seems more likely to work again… (🤔sounds strange, I know)

Steps to reproduce

Install 2.4.1 on Kubuntu 18.04
Right-click on tray icon

Client configuration

Client version:
2.4.1

Operating system:
Kubuntu 18.04

OS language:
EN

Qt version used by client package (Linux only, see also Settings dialog):
5.6.2

Client package (From ownCloud or distro) (Linux only):
From ownCloud

Installation path of client:
???

00009497

guruz · 2018-05-24T13:50:49Z

@michaelstingl Do you have a machine to reproduce this? If yes try the 2.5.0 nightlies with newer Qt?

ogoffart · 2018-05-24T14:15:52Z

"Works for me" is all i can say. (But i'm using Qt 5.10.1 on Archlinux)

This looks like a Qt or Plasma bug.

michaelstingl · 2018-05-24T19:49:33Z

Same with 2.5.0 daily with Qt 5.10.1

I run a VirtualBox machine fresh installed from kubuntu-18.04-desktop-amd64.iso (no special options, everything default)

BTW Why does the "Close" button has the "X"?

ckamm · 2018-05-25T06:09:52Z

I'm also on kubuntu 18.04 but haven't seen this issue. I was using my own builds with a different Qt version though. I'll see - at best we can find yet another tray icon workaround though. :/

ckamm · 2018-05-25T06:52:45Z

I can reproduce with the kubuntu-18.04 iso. Interestingly floating systray widgets work (on the second click, it seems focus needs to go to the tray first - that's not the case for other indicators), but systrays in panels don't do right-clicks reliably at all. Left-clicks work every time.

I tested with 2.4.1 (built with and running with Qt 5.6.2)

ckamm · 2018-05-25T07:26:26Z

And I can confirm that my manually built version with Qt 5.9.1 using the system Qt 5.9.5 works as expected. And with the nightly using 5.10.1 it doesn't work again.

(also note that my custom build is the only instance to use the ubuntu monochome icon (black cloud with white outline, I didn't find it in our repo, where does it come from?), the others use the default colored owncloud icon)

ckamm · 2018-05-25T07:45:23Z

It seems to relate to some of our other tray workarounds - with OWNCLOUD_MINIMAL_TRAY_MENU=1 it works. Sadly, OWNCLOUD_FORCE_QDBUS_TRAY_WORKAROUND=1 makes it work too.

ckamm · 2018-05-25T08:18:59Z

@ogoffart I currently have no good guess as to why my 5.9 version would work when the 5.6 and 5.10 versions don't. Any ideas?

I don't want to enable the workaround we built for qdbusmenu because that triggered another set of tray bugs. :/

ogoffart · 2018-05-25T08:51:23Z

Maybe because of a plugin that is only loaded with system's Qt (such as the KDE integration plugin). It might have a different systray implementation.

SamuAlfageme · 2018-05-28T09:04:03Z

Reminds me somehow to #5968 (see also https://github.com/owncloud/enterprise/issues/2151)

guruz · 2018-05-28T10:53:41Z

Maybe because of a plugin that is only loaded with system's Qt (such as the KDE integration plugin).

https://unix.stackexchange.com/a/358500
https://unix.stackexchange.com/a/220110

ogoffart · 2018-05-30T12:08:02Z

In my opinion, we should not ship Qt for distribution that have a Qt that is recent enough.
So distribution with Qt >5.6.1 (or 5.9.1?) should simply use system Qt

guruz · 2018-05-30T14:42:04Z

They won't get HTTP2 then though

guruz · 2018-05-30T16:26:49Z

(but tray is more important than HTTP2 so @ogoffart is right)

michaelstingl · 2018-06-07T21:08:29Z

adding @dschmidt to the discussion…

ckamm · 2018-06-11T13:29:36Z

I've tried to verify that this is indeed about the platform plugins. Comparing the loaded libraries doesn't help unfortunately since both versions load things like /usr/lib/x86_64-linux-gnu/qt5/plugins/kf5/org.kde.kwindowsystem.platforms/KF5WindowSystemX11Plugin.so - though it could be that the distributed version discards it somehow. I haven't yet figured out how to tell what's actually happening with the tray.

ckamm · 2018-06-11T13:36:28Z

What I could verify is that my self-build version also starts having a broken tray if I remove /usr/lib/x86_64-linux-gnu/qt5/plugins/platformthemes/KDEPlasmaPlatformTheme.so and that makes sense as this library contains kdeplatformsystemtrayicon.cpp

ckamm · 2018-06-13T09:57:03Z

So: Yes, using the system Qt for the distributions where it's new enough to not cause other issues would be the correct solution.

I'll also look for workarounds (the minimal tray menu works, after all), but those will likely need compromises with respect to features, or introduce new issues down the line.

ckamm · 2018-06-14T13:00:32Z

Some debugging has shown me that contextMenu->isVisible() always returns false if the kde integration plugin isn't loaded. And building the context menu in aboutToShow() doesn't work either, similar to OSX.

* Disentangle the previous 'qdbusWorkarounds' into three different things * Make not trusting tray.isVisible() a new workaround * Introduce env vars for all workaround flags * Use the workaround flags for OSX * Determine workaround flags for KDE when the plasma integration plugin is missing

jnweiger · 2018-06-15T15:33:56Z

@michaelstingl here are linux packages built from ckamm' tray-workarounds branch. Can you try if this improves the situation on your kubuntu too. for me is seems to ease the pain.

https://download.owncloud.com/internal/repo/desktop:/testing/download/

jnweiger · 2018-06-18T09:13:54Z

I am unhappy with my test environment:

The old 241 code now behaves acceptably well. The right click menu shows up on every second click.
The new code from Tray workarounds #6586 seem to just behave the same :-(

ckamm · 2018-06-18T10:22:51Z

@jnweiger Odd. Are all of these tests done with binaries that link to a custom Qt version that has a different minor version number from the system Qt? The important bit is to ensure that KDEPlasmaPlatformTheme.so can't be loaded - otherwise it should just work.

jnweiger · 2018-06-18T12:42:02Z

gna my tests were faulty, I'll recheck and keep an eye on KDEPlasmaPlatformTheme.so -- thanks for raising doubt :-)

jnweiger · 2018-06-18T15:05:30Z

cernbox 241 with qt562 is without KDEPlasmaPlatformTheme.so and right click on the tray icon is very hard.
testpilotcloud 242-daily with qt562 is without KDEPlasmaPlatformTheme.so and right click on the tray icon is very hard.
testpilotcloud 250-daily with qt5101 is without KDEPlasmaPlatformTheme.so and right click on the tray icon is very hard.
owncloud 250-trayworkarounds with qt5101 is without KDEPlasmaPlatformTheme.so and right click on the tray icon works almost ever time. Occasionally a second right click is needed, (seems to coincident with messages fading in or out) .

All of these done on one and the same VM in that order, with purge and autoremove in between and ca. 240 online updates waiting to be installed.
All done linked with our own QT, either 5.6.2 or 5.10.1, never with a system qt.
No dolphin, nemo, nautilus plugins installed.

grepped for KDEPlasmaPlatfomTheme in ldd and lsof output using grep -i pla.

There are matches only for things like

.../libcomposeplatforminputcontextplugin.so
.../platforms/libqxcb.so
.../plasmashellVr1298.4.slave-socket

jnweiger · 2018-06-18T15:34:53Z

Reproduced the same behaviour after installing all pending ubuntu online updates.

@ckamm, I now officially love your trayworkarounds branch!

jnweiger · 2018-06-18T15:41:30Z

@ckamm, what would be the effort to backport this PR for 2.4.2 ?

* Disentangle the previous 'qdbusWorkarounds' into three different things * Make not trusting tray.isVisible() a new workaround * Introduce env vars for all workaround flags * Use the workaround flags for OSX * Determine workaround flags for KDE when the plasma integration plugin is missing

ckamm · 2018-06-19T10:54:56Z

@jnweiger Thanks for the elaborate testing! The patch applies cleanly to 2.4, would be no effort to backport.

jnweiger · 2018-06-19T11:02:58Z

Great! Also for 2.4.* with Qt-5.6.2 ?

* Disentangle the previous 'qdbusWorkarounds' into three different things * Make not trusting tray.isVisible() a new workaround * Introduce env vars for all workaround flags * Use the workaround flags for OSX * Determine workaround flags for KDE when the plasma integration plugin is missing (cherry picked from commit 99116ac) (cherry picked from commit ca033b9) (cherry picked from commit 8b1d979) (cherry picked from commit a0d6139)

ckamm · 2018-06-19T11:06:29Z

@jnweiger Backport PR is #6594

I don't understand the question about 2.4.* with Qt-5.6.2. A bugfix would only go into 2.4.2? And hopefully the Qt version doesn't matter - it might though! Could I ask you to test the 2.4 builds as thoroughly? I'd be sad if we broke trays for 2.4.2 instead of improving the situation.

jnweiger · 2018-06-19T12:33:53Z

building test packages...
I was afraid Qt-5.6.2 might need other workarounds than 5.10.1 - if not, the better!

jnweiger · 2018-06-19T12:52:41Z

https://download.owncloud.com/internal/repo/ownbrander:/Qt5101:/testpilotcloud/download/
has 2.4.2 Linux builds from the backports branch trayworkaround-24 against Qt-5.10.1
https://download.owncloud.com/internal/repo/ownbrander:/Qt562:/testpilotcloud/download/
has 2.4.2 Linux builds from the backports branch trayworkaround-24 against Qt-5.6.2

jnweiger · 2018-06-19T15:48:56Z

Thumbs up for the backported PR! Approved from my side, but please doublecheck Olivier's comment.

* Disentangle the previous 'qdbusWorkarounds' into three different things * Make not trusting tray.isVisible() a new workaround * Introduce env vars for all workaround flags * Use the workaround flags for OSX * Determine workaround flags for KDE when the plasma integration plugin is missing (cherry picked from commit 99116ac) (cherry picked from commit ca033b9) (cherry picked from commit 8b1d979) (cherry picked from commit a0d6139)

michaelstingl · 2018-07-02T16:42:13Z

@jnweiger @ogoffart close now?

jnweiger · 2018-07-11T10:19:32Z

Backport test done:
Install 242rc1 both testpilotcloud-client and owncloud-client on a ubuntu 18.04 plasma VM.

ubuntu

wget -nv https://download.opensuse.org/repositories/isv:ownCloud:desktop:client-2.4.2rc1/Ubuntu_18.04/Release.key -O Release.key
sudo apt-key add - < Release.key
sudo sh -c "echo 'deb http://download.opensuse.org/repositories/isv:/ownCloud:/desktop:/client-2.4.2rc1/Ubuntu_18.04/ /' > /etc/apt/sources.list.d/isv:ownCloud:desktop:client-2.4.2rc1.list"
sudo sh -c "echo 'deb http://download.opensuse.org/repositories/isv:/ownCloud:/testpilot:/testing/Ubuntu_18.04/ /' > /etc/apt/sources.list.d/isv:ownCloud:testpilot:testing.list"
sudo apt-get update
sudo apt purge -y owncloud-client libowncloudsync0
sudo apt purge -y testpilotcloud-client libtestpilotcloudsync0
sudo apt-get install -y owncloud-client
sudo apt-get install -y testpilotcloud-client
owncloudcmd --version
testpilotcloudcmd --version

leap

zypper addrepo https://download.opensuse.org/repositories/isv:ownCloud:desktop:client-2.4.2rc1/openSUSE_Leap_42.2/isv:ownCloud:desktop:client-2.4.2rc1.repo
zypper addrepo https://download.opensuse.org/repositories/isv:ownCloud:testpilot:testing/openSUSE_Leap_42.2/isv:ownCloud:testpilot:testing.repo
zypper refresh
zypper rm -y owncloud-client libowncloudsync0
zypper rm -y testpilotcloud-client libtestpilotcloudsync0
zypper install -y owncloud-client
zypper install -y testpilotcloud-client
owncloudcmd --version
testpilotcloudcmd --version

Both systems

log in on the desktop
left click the system menue: a dialog opens containing plasma search.
run the installatin procedures above. the correct cliebt version numbers are printed at the end.
Both systems, both clients:
right click the tray icon multople times. a dialog containing quit opens and closes
left click the icon, the settings window opens.
use right click -> quit to end the client.

Success. Thank you all.Closing.

jnweiger · 2018-07-12T10:36:44Z

Backport test done:

#### Task

Install 242rc1 both testpilotcloud-client and owncloud-client on a ubuntu 18.04 plasma VM.

##### ubuntu
wget -nv https://download.opensuse.org/repositories/isv:ownCloud:desktop:client-2.4.2rc1/Ubuntu_18.04/Release.key -O Release.key
sudo apt-key add - < Release.key
sudo sh -c "echo 'deb http://download.opensuse.org/repositories/isv:/ownCloud:/desktop:/client-2.4.2rc1/Ubuntu_18.04/ /' > /etc/apt/sources.list.d/isv:ownCloud:desktop:client-2.4.2rc1.list"
sudo sh -c "echo 'deb http://download.opensuse.org/repositories/isv:/ownCloud:/testpilot:/testing/Ubuntu_18.04/ /' > /etc/apt/sources.list.d/isv:ownCloud:testpilot:testing.list"
sudo apt-get update
sudo apt purge -y owncloud-client libowncloudsync0
sudo apt purge -y testpilotcloud-client libtestpilotcloudsync0
sudo apt-get install -y owncloud-client
sudo apt-get install -y testpilotcloud-client
owncloudcmd --version
testpilotcloudcmd --version

##### leap
zypper addrepo https://download.opensuse.org/repositories/isv:ownCloud:desktop:client-2.4.2rc1/openSUSE_Leap_42.2/isv:ownCloud:desktop:client-2.4.2rc1.repo
zypper addrepo https://download.opensuse.org/repositories/isv:ownCloud:testpilot:testing/openSUSE_Leap_42.2/isv:ownCloud:testpilot:testing.repo
zypper refresh
zypper rm -y owncloud-client libowncloudsync0
zypper rm -y testpilotcloud-client libtestpilotcloudsync0
zypper install -y owncloud-client
zypper install -y testpilotcloud-client
owncloudcmd --version
testpilotcloudcmd --version


Both systems
* log in on the desktop
* left click the system menue: a dialog opens containing plasma search.
* run the installatin procedures above. the correct cliebt version numbers are printed at the end.
Both systems, both clients:
* right click the tray icon multople times. a dialog containing quit opens and closes
* left click the icon, the settings window opens.
* use right click -> quit to end the client.

Success. Thank you all.Closing.

michaelstingl added Platform Specific blue-ticket packaging Linux labels May 24, 2018

michaelstingl assigned jnweiger May 24, 2018

guruz assigned ckamm and ogoffart May 24, 2018

ckamm added this to the 2.5.0 milestone May 25, 2018

ckamm mentioned this issue Jun 14, 2018

Tray workarounds #6586

Merged

ckamm added the PR available label Jun 14, 2018

ckamm added ReadyToTest QA, please validate the fix/enhancement and removed PR available labels Jun 19, 2018

ckamm mentioned this issue Jun 19, 2018

Tray workarounds backport for 2.4 #6594

Merged

jnweiger closed this as completed Jul 11, 2018

camilasan mentioned this issue Sep 9, 2018

Tray workarounds #6545 nextcloud/desktop#637

Merged

KDE Plasma: Right-click on tray icon doesn't work #6545

KDE Plasma: Right-click on tray icon doesn't work #6545

Comments

michaelstingl commented May 24, 2018

Steps to reproduce

Client configuration

guruz commented May 24, 2018

ogoffart commented May 24, 2018

michaelstingl commented May 24, 2018 • edited Loading

ckamm commented May 25, 2018

ckamm commented May 25, 2018 • edited Loading

ckamm commented May 25, 2018 • edited Loading

ckamm commented May 25, 2018 • edited Loading

ckamm commented May 25, 2018

ogoffart commented May 25, 2018

SamuAlfageme commented May 28, 2018 • edited Loading

guruz commented May 28, 2018

ogoffart commented May 30, 2018

guruz commented May 30, 2018

guruz commented May 30, 2018

michaelstingl commented Jun 7, 2018

ckamm commented Jun 11, 2018

ckamm commented Jun 11, 2018 • edited Loading

ckamm commented Jun 13, 2018

ckamm commented Jun 14, 2018

jnweiger commented Jun 15, 2018

jnweiger commented Jun 18, 2018

ckamm commented Jun 18, 2018 • edited Loading

jnweiger commented Jun 18, 2018

jnweiger commented Jun 18, 2018 • edited Loading

jnweiger commented Jun 18, 2018

jnweiger commented Jun 18, 2018

ckamm commented Jun 19, 2018

jnweiger commented Jun 19, 2018

ckamm commented Jun 19, 2018 • edited Loading

jnweiger commented Jun 19, 2018

jnweiger commented Jun 19, 2018 • edited Loading

jnweiger commented Jun 19, 2018

michaelstingl commented Jul 2, 2018 • edited Loading

jnweiger commented Jul 11, 2018 • edited Loading

ubuntu

leap

jnweiger commented Jul 12, 2018

michaelstingl commented May 24, 2018 •

edited

Loading

ckamm commented May 25, 2018 •

edited

Loading

ckamm commented May 25, 2018 •

edited

Loading

ckamm commented May 25, 2018 •

edited

Loading

SamuAlfageme commented May 28, 2018 •

edited

Loading

ckamm commented Jun 11, 2018 •

edited

Loading

ckamm commented Jun 18, 2018 •

edited

Loading

jnweiger commented Jun 18, 2018 •

edited

Loading

ckamm commented Jun 19, 2018 •

edited

Loading

jnweiger commented Jun 19, 2018 •

edited

Loading

michaelstingl commented Jul 2, 2018 •

edited

Loading

jnweiger commented Jul 11, 2018 •

edited

Loading