Bug 419263

Summary:	Right click on system tray icon brings wrong menu
Product:	[Plasma] lattedock	Reporter:	Yannick Roehlly <yannick.roehlly>
Component:	application	Assignee:	Michail Vourlakos <mvourlakos>
Status:	RESOLVED FIXED
Severity:	normal	CC:	chris, raul338
Priority:	NOR
Version:	git (master)
Target Milestone:	---
Platform:	Debian unstable
OS:	Linux
Latest Commit:	https://commits.kde.org/latte-dock/85247d37b5540d5e2e731dca6b8aaa4f6e6bccb4	Version Fixed In:	0.9.11

Description Yannick Roehlly 2020-03-26 12:13:28 UTC

SUMMARY

Right clicking on a system tray icon brings the system tray global configuration
menu instead of the one from the clicked icon. The only way to access the icon
contextual menu is to do a long click on the icon.

STEPS TO REPRODUCE
1. Launch Kmail with system tray integration
2. Right click on the Kmail icon in the system tray

OBSERVED RESULT

The contextual menu appearing is (maybe not the exact wording as my locale is not
English)

- Configure system tray
- Configure Latte...
- Quit Latte
- Add widgets...
- Edit Panel...

EXPECTED RESULT

The Kmail contextual menu should appear:

- Check mail
- Check mail in...
- etc...

SOFTWARE/OS VERSIONS
Locally compiled package on Debian Sid.
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.17.5
KDE Frameworks Version: 5.62.0
Qt Version: 5.12.5

ADDITIONAL INFORMATION

The problem is also present in the version 0.9.10 (not available in the version
selection box of the bug tracker ;-))

Comment 1 Michail Vourlakos 2020-03-26 12:20:10 UTC

when you click in the systray the audio icon,
do you get:

"Configure Audio Volume" as first entry?

Comment 2 Michail Vourlakos 2020-03-26 12:45:56 UTC

Git commit ef2650e711806ca1f26bb47d640c7888270e4b64 by Michail Vourlakos.
Committed on 26/03/2020 at 12:40.
Pushed by mvourlakos into branch 'master'.

systray,support context menus for SNI items

--SNI items are handling the context menu by their
own, so Latte code should not interfere
FIXED-IN:0.9.11

M  +9    -1    app/view/contextmenu.cpp

https://commits.kde.org/latte-dock/ef2650e711806ca1f26bb47d640c7888270e4b64

Comment 3 Michail Vourlakos 2020-03-26 12:46:37 UTC

Git commit 85247d37b5540d5e2e731dca6b8aaa4f6e6bccb4 by Michail Vourlakos.
Committed on 26/03/2020 at 12:46.
Pushed by mvourlakos into branch 'v0.9'.

systray,support context menus for SNI items

--SNI items are handling the context menu by their
own, so Latte code should not interfere
FIXED-IN:0.9.11

M  +9    -1    app/view/contextmenu.cpp

https://commits.kde.org/latte-dock/85247d37b5540d5e2e731dca6b8aaa4f6e6bccb4

Comment 4 Michail Vourlakos 2020-03-26 12:48:05 UTC

long story, short, noone had reported the issue in older versions... The bug must exist almost in all v0.9.x versions, the group plasmoid fix was not the blamer.

Comment 5 Yannick Roehlly 2020-03-26 13:24:28 UTC

Hi Michail.

I've tested the new code. It works! Thanks for the super fast answer and correction.

Concerning your last comment, I don't have the problem with the 0.9.9 version.

Comment 6 Michail Vourlakos 2020-03-26 13:42:34 UTC

(In reply to Yannick Roehlly from comment #5)
> Hi Michail.
> 
> I've tested the new code. It works! Thanks for the super fast answer and
> correction.
> 
> Concerning your last comment, I don't have the problem with the 0.9.9
> version.

You are right, I checked it a line that was important, was lost during 0.9.10 if this is noticed by many users I will release 0.9.11 sooner

Comment 7 Raúl Arellano 2020-04-09 15:54:58 UTC

*** Bug 419862 has been marked as a duplicate of this bug. ***

Comment 8 Michail Vourlakos 2020-04-10 08:30:32 UTC

*** Bug 419907 has been marked as a duplicate of this bug. ***