| Summary: | Spotify notifications (coming from plasma-workspace, through XDG portal) plays error sound | ||
|---|---|---|---|
| Product: | [Plasma] xdg-desktop-portal-kde | Reporter: | Linus Kardell <linus.kardell> |
| Component: | general | Assignee: | Plasma Bugs List <plasma-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | agurenko, aleixpol, angusfk, bizyaev, eric.donkersloot, fedin-ilja2010, fella, jgrulich, katyaberezyaka, kdebugs.81do7, kdelibs-bugs, lukas, michele.kipiel, nanoblade9, nate, progexer, spry.bed7373, zellox |
| Priority: | NOR | ||
| Version: | 5.26.3 | ||
| Target Milestone: | --- | ||
| Platform: | Flatpak | ||
| OS: | Linux | ||
| See Also: | https://github.com/telegramdesktop/tdesktop/issues/25469 | ||
| Latest Commit: | https://invent.kde.org/frameworks/frameworkintegration/commit/f4a899b321948bfa9a0a890f2027678557944205 | Version Fixed In: | 5.105 |
| Sentry Crash Report: | |||
|
Description
Linus Kardell
2022-08-09 12:00:37 UTC
I don't want to presume, but I'd consider this a duplicate of 462283 (I've posted a workaround given by a Telegram contributor there). It's the opposite; the newer ones should be marked as a duplicate of this. But yes, you're right! *** Bug 462357 has been marked as a duplicate of this bug. *** *** Bug 462283 has been marked as a duplicate of this bug. *** Wouldn't the right fix be to just remove the KNotification::DefaultEvent flag? The portal creates applications' notifications, not system notifications after all. https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/blob/master/src/notification.cpp#L36 Happens to me as well, it's annoying as heck. Box details as follows: Operating System: openSUSE Tumbleweed 20230226 KDE Plasma Version: 5.27.1 KDE Frameworks Version: 5.103.0 Qt Version: 5.15.8 Kernel Version: 6.1.12-1-default (64-bit) Graphics Platform: Wayland Processors: 4 × Intel® Core™ i5-6200U CPU @ 2.30GHz Memory: 7.5 GiB of RAM Graphics Processor: Mesa Intel® HD Graphics 520 Manufacturer: LENOVO Product Name: 20FB002UMS System Version: ThinkPad X1 Carbon 4th A possibly relevant merge request was started @ https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/merge_requests/169 *** Bug 466441 has been marked as a duplicate of this bug. *** Git commit 183c0525e7a33996126c9438ce2a17065a274e54 by Nate Graham, on behalf of Nicolas Fella. Committed on 29/03/2023 at 16:04. Pushed by ngraham into branch 'master'. Don't play sound for plain notification We typically don't play a sound by default for most notifications. However the 'notification' event for the Plasma default notifyrc has a sound by default Since this event is used by xdg-desktop-portal-kde, currently all notifications from Flatpak apps have a sound by default, making them inconsistent with their non-Flatpak counterparts. Remove the default sound to align them. FIXED-IN: 5.105 M +1 -2 plasma_workspace.notifyrc https://invent.kde.org/frameworks/frameworkintegration/commit/183c0525e7a33996126c9438ce2a17065a274e54 Git commit f4a899b321948bfa9a0a890f2027678557944205 by Nate Graham, on behalf of Nicolas Fella. Committed on 29/03/2023 at 16:06. Pushed by ngraham into branch 'kf5'. Don't play sound for plain notification We typically don't play a sound by default for most notifications. However the 'notification' event for the Plasma default notifyrc has a sound by default Since this event is used by xdg-desktop-portal-kde, currently all notifications from Flatpak apps have a sound by default, making them inconsistent with their non-Flatpak counterparts. Remove the default sound to align them. FIXED-IN: 5.105 (cherry picked from commit 183c0525e7a33996126c9438ce2a17065a274e54) M +1 -2 plasma_workspace.notifyrc https://invent.kde.org/frameworks/frameworkintegration/commit/f4a899b321948bfa9a0a890f2027678557944205 |