Bug 476809 - Screenshot notifications don't get right click menu
Summary: Screenshot notifications don't get right click menu
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Notifications (show other bugs)
Version: 5.27.80
Platform: Neon Linux
: NOR minor
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: qt6
Depends on:
Blocks:
 
Reported: 2023-11-10 21:18 UTC by iodreamify
Modified: 2024-01-26 00:49 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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