https://bugs.kde.org/show_bug.cgi?id=510702

skierpage <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|[email protected]   |[email protected]
            Summary|No notification sounds from |Sounds don't play in
                   |KDE apps                    |Configure Notifications...
                   |                            |dialog
     Ever confirmed|0                           |1
             Status|RESOLVED                    |REOPENED
         Resolution|WORKSFORME                  |---
          Component|general                     |general
            Product|frameworks-knotifications   |frameworks-knotifyconfig

--- Comment #5 from skierpage <[email protected]> ---
The good news is the KTeaTime RPM and Skrooge nightly flatpak play their
notification sounds in normal operation. But not in their Configure
Notifications... dialog (see "STEPS TO REPRODUCE").

If you look at KNotifyConfigActionsWidget::slotPlay() in 
knotifyconfig/src/knotifyconfigactionswidget.cpp , it doesn't seem to be doing
any sound theme setting or semantic sound naming, it just assumes the
soundFileName is a local file and sets CA_PROP_MEDIA_FILENAME to it.

The applications themselves play the right notifications because
NotifyByAudio::playSound() in knotifications/src/notifybyaudio.cpp sets the
semantic CA_PROP_EVENT_ID to the soundName and set
CA_PROP_CANBERRA_XDG_THEME_NAME (while also supporting setting
CA_PROP_MEDIA_FILENAME to a fallbackUrl). System Settings > Notifications >
Application Settings does something similar in
plasma-workspace/kcms/notifications/kcm.cpp.

It seems to me KDE should have a single lower-level playSound() shared between
apps, knotifyconfig dialog, and kcm_notifications that handles both semantic
sound names and file paths.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to