Review Request 121356: Do not change the volume when playing a notification

[Martin Klapetek]

> On Dec. 5, 2014, 11:59 a.m., Martin Klapetek wrote:
> > For frameworks version it's actually using notifybyaudio plugin for playing the sounds and that's using phonon exclusively, there's no more volume fiddling in fact. The notifybysound should just be removed altogether..don't know why it's still around :O
> 
> David Edmundson wrote:
>     so the entire page about choosing an external player in plasma-desktop is invalid?

Seems like it. If we still want that, I think it would be easy to implement into the notifybyaudio.


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121356/#review71411
-----------------------------------------------------------


On Dec. 4, 2014, 10:12 p.m., Albert Astals Cid wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121356/
> -----------------------------------------------------------
> 
> (Updated Dec. 4, 2014, 10:12 p.m.)
> 
> 
> Review request for KDE Frameworks and David Edmundson.
> 
> 
> Repository: knotifications
> 
> 
> Description
> -------
> 
> Playing a notification should be that, playing a notification.
> 
> Inspired by David's comment on https://git.reviewboard.kde.org/r/121278/
> 
> 
> Diffs
> -----
> 
>   src/notifybysound.cpp 09a80dd 
> 
> Diff: https://git.reviewboard.kde.org/r/121356/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Albert Astals Cid
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20141205/74fc89af/attachment.html>