[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 Aleix Pol changed: What|Removed |Added Version Fixed In||5.26.5 -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 --- Comment #12 from Aleix Pol --- Git commit 511bced812cc0615dfa8b827a1c108db4b3e919f by Aleix Pol. Committed on 15/12/2022 at 15:02. Pushed by apol into branch 'Plasma/5.26'. notification: Trigger org.freedesktop.Application.Activate* as spec'd For some reason, it was spec'd that these should be called, so we better follow through. (cherry picked from commit 77c312560f0d240a553c074d96df438c75e1430a) A +17 -0data/org.freedesktop.Application.xml M +2-0src/CMakeLists.txt M +24 -5src/notification.cpp https://invent.kde.org/plasma/xdg-desktop-portal-kde/commit/511bced812cc0615dfa8b827a1c108db4b3e919f -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 Aleix Pol changed: What|Removed |Added Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas |ma/xdg-desktop-portal-kde/c |ma/xdg-desktop-portal-kde/c |ommit/c53684f5b6d27b1793898 |ommit/4f8df470f8b3b9df1844d |2e6333d574636dd4016 |606a05b3c83cce88772 --- Comment #11 from Aleix Pol --- Git commit 4f8df470f8b3b9df1844d606a05b3c83cce88772 by Aleix Pol. Committed on 15/12/2022 at 15:02. Pushed by apol into branch 'Plasma/5.26'. notifications: Properly communicate the triggered action Infer the action name from the originally communicated action name. (cherry picked from commit c53684f5b6d27b17938982e6333d574636dd4016) M +34 -10 src/notification.cpp https://invent.kde.org/plasma/xdg-desktop-portal-kde/commit/4f8df470f8b3b9df1844d606a05b3c83cce88772 -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 --- Comment #10 from Bug Janitor Service --- A possibly relevant merge request was started @ https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/merge_requests/142 -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 --- Comment #9 from Aleix Pol --- Git commit 77c312560f0d240a553c074d96df438c75e1430a by Aleix Pol Gonzalez, on behalf of Aleix Pol. Committed on 15/12/2022 at 00:00. Pushed by apol into branch 'master'. notification: Trigger org.freedesktop.Application.Activate* as spec'd For some reason, it was spec'd that these should be called, so we better follow through. A +17 -0data/org.freedesktop.Application.xml M +2-0src/CMakeLists.txt M +24 -5src/notification.cpp https://invent.kde.org/plasma/xdg-desktop-portal-kde/commit/77c312560f0d240a553c074d96df438c75e1430a -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 Aleix Pol changed: What|Removed |Added Latest Commit||https://invent.kde.org/plas ||ma/xdg-desktop-portal-kde/c ||ommit/c53684f5b6d27b1793898 ||2e6333d574636dd4016 Resolution|--- |FIXED Status|ASSIGNED|RESOLVED --- Comment #8 from Aleix Pol --- Git commit c53684f5b6d27b17938982e6333d574636dd4016 by Aleix Pol. Committed on 15/12/2022 at 00:52. Pushed by apol into branch 'master'. notifications: Properly communicate the triggered action Infer the action name from the originally communicated action name. M +34 -10 src/notification.cpp https://invent.kde.org/plasma/xdg-desktop-portal-kde/commit/c53684f5b6d27b17938982e6333d574636dd4016 -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 Andrey changed: What|Removed |Added CC||butir...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 Bug Janitor Service changed: What|Removed |Added Status|REPORTED|ASSIGNED Ever confirmed|0 |1 --- Comment #7 from Bug Janitor Service --- A possibly relevant merge request was started @ https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/merge_requests/140 -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 --- Comment #6 from Ilya Fedin --- The application I use to monitor D-Bus traffic is Bustle -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 --- Comment #5 from Ilya Fedin --- It's also pretty obvious from the code: https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/blob/master/src/notification.cpp#L125 It should have some map between the uint id and action name/target parameter supplied by the application, but it just supplies the uint id from KNotification library instead. -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 --- Comment #4 from Ilya Fedin --- I don't think I'll be able to write a small test app (as GNotification uses ActivateAction, KNotification is written that it avoids this issue and writing new spec implementation wouldn't be small). What I propose instead is to look at d-bus traffic between the portal and any application using GNotification. Even though GNotification doesn't watch from ActionInvoked signal, it's seen that the content of the signal is broken. Here's a screenshot of the action list from Telegram and the ActionInvoked signal produced by the portal implementation. It's seen that Telegram has no numeric actions and has target parameter for every of them, but ActionInvoked content is completely different. -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 --- Comment #3 from Ilya Fedin --- Created attachment 154118 --> https://bugs.kde.org/attachment.cgi?id=154118=edit ActionInvoked -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 --- Comment #2 from Ilya Fedin --- Created attachment 154117 --> https://bugs.kde.org/attachment.cgi?id=154117=edit AddNotification -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 --- Comment #1 from Aleix Pol --- I've been looking your report up, but I was unable to reproduce the behaviour you are describing. Would you be able to provide a small test app that exposes the problem? Regarding the latter case, it indeed is missing but is a separate issue. -- You are receiving this mail because: You are watching all bug changes.
[xdg-desktop-portal-kde] [Bug 462278] Actions in Notification portal totally broken
https://bugs.kde.org/show_bug.cgi?id=462278 Wyatt Childers changed: What|Removed |Added CC||kdebugs.81do7@haxing.ninja -- You are receiving this mail because: You are watching all bug changes.
