Bug 476809

Summary:	Screenshot notifications don't get right click menu
Product:	[Plasma] plasmashell	Reporter:	iodreamify
Component:	Notifications	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	minor	CC:	agurenko, fanzhuyifan, kde, kde, nate, nicolas.fella
Priority:	NOR	Keywords:	qt6
Version:	5.27.80
Target Milestone:	1.0
Platform:	Neon
OS:	Linux
See Also:	https://bugs.kde.org/show_bug.cgi?id=472997
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/-/commit/5ed30d6af664ea582da0a3394a82f0686945bc28	Version Fixed In:
Sentry Crash Report:

Description iodreamify 2023-11-10 21:18:06 UTC

SUMMARY
On Wayland, if you use Shift+Prtscr to take a fullscreen screenshot and right click on the resulting image embedded in the  notification you won't get a context menu like on X11. Buttons "Annotate" and the explicit context menu button both work.

STEPS TO REPRODUCE
1. Shift+Prtscr to take a fullscreen screenshot 
2. Right click on the resulting thumbnail in the notification

OBSERVED RESULT
No context menu

EXPECTED RESULT
Context menu appears

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.27.80
KDE Frameworks Version: 5.245.0
Qt Version: 6.6.0
Kernel Version: 6.2.0-36-generic (64-bit)
Graphics Platform: Wayland

Comment 1 Kai Uwe Broulik 2023-11-10 21:19:52 UTC

Interesting, can confirm. The codepath should be the same for the menu button so I wonder why right-click doesn’t work…

Comment 2 Nicolas Fella 2023-12-08 23:37:23 UTC

Possible same root cause as https://bugs.kde.org/show_bug.cgi?id=472997

Comment 3 Bug Janitor Service 2023-12-15 09:49:18 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3685

Comment 4 fanzhuyifan 2024-01-13 03:18:33 UTC

Remarking as confirmed since the MR was closed.

Comment 5 Bug Janitor Service 2024-01-13 05:18:24 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3767

Comment 6 fanzhuyifan 2024-01-26 00:36:41 UTC

Git commit 441327b3f7afa8c2f409be22c98013bbca47a193 by Yifan Zhu.
Committed on 26/01/2024 at 01:05.
Pushed by fanzhuyifan into branch 'master'.

applet/notifications: fix contex menu on wayland

In general mouseArea should be used instead of TapHandler to handle
simple pressed events. Using TapHandler causes problems on wayland.

M  +4    -5    applets/notifications/package/contents/ui/DraggableFileArea.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/441327b3f7afa8c2f409be22c98013bbca47a193

Comment 7 fanzhuyifan 2024-01-26 00:49:40 UTC

Git commit 5ed30d6af664ea582da0a3394a82f0686945bc28 by Yifan Zhu.
Committed on 26/01/2024 at 01:37.
Pushed by fanzhuyifan into branch 'Plasma/6.0'.

applet/notifications: fix contex menu on wayland

In general mouseArea should be used instead of TapHandler to handle
simple pressed events. Using TapHandler causes problems on wayland.


(cherry picked from commit 441327b3f7afa8c2f409be22c98013bbca47a193)

M  +4    -5    applets/notifications/package/contents/ui/DraggableFileArea.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/5ed30d6af664ea582da0a3394a82f0686945bc28