Bug 430235 - UI regression: Very difficult to configure per-event notification
Summary: UI regression: Very difficult to configure per-event notification
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_notify (other bugs)
Version First Reported In: 5.20.4
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Kai Uwe Broulik
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2020-12-10 22:05 UTC by Yichao Yu
Modified: 2024-03-08 21:41 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 6
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Yichao Yu 2020-12-10 22:05:04 UTC
SUMMARY

AFAICT, there has never been an easy way to disable sound notification for all/most of the events. However, it was OK before since the per-event notification setting are very easy to see and change. It takes a single key press or click to go from event config for one program to the next and the icon makes it clear at a glance which one potentially have sound on.

However, since a few versions ago (I only noticed this when I want to change this setting again after getting a new computer) the per-event config is now in its own popup window while leaving a lot of empty space in the parent window that can easily contain the event config even for the smallest window size allowed. Given this, I believe the event config should be put back to the parent window instead of having to be it's own window (allowing it to popup when accessed from somewhere else (if that's the goal) is certainy fine and AFAICT the two can coexist with little duplicated code).

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: ArchLinux
(available in About System)
KDE Plasma Version: 5.20.4
KDE Frameworks Version: 5.76.0
Qt Version: 5.15.2
Comment 1 Nate Graham 2020-12-16 22:03:09 UTC
Yeah we might be able to put it inside the window.
Comment 2 Ismael Asensio 2024-03-08 21:40:53 UTC
Fixed for Plasma 6 with https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3055