Review Request 123360: Introduce LoopSound flag

Martin Klapetek martin.klapetek at gmail.com
Tue Apr 14 14:41:44 UTC 2015


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

Review request for KDE Frameworks.


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


Repository: knotifications


Description
-------

Some notifications might want to loop the sound, eg. calling apps playing the ringing sound which is not wanted to play once. Currently the sound is looped for all persistent notifications, but that's not ideal as not all persistent notifications with sound want/need sound looping. This new LoopSound flag solves that.


Diffs
-----

  src/knotification.h 8efd92a 
  src/notifybyaudio.cpp ee3ef1e 

Diff: https://git.reviewboard.kde.org/r/123360/diff/


Testing
-------


Thanks,

Martin Klapetek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150414/3e5c93e9/attachment.html>


More information about the Kde-frameworks-devel mailing list