<table><tr><td style="">rjvbb created this revision.<br />rjvbb added a project: Frameworks.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D5070" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This is a "port" of <a href="https://git.reviewboard.kde.org/r/128043/" class="remarkup-link" target="_blank" rel="noreferrer">https://git.reviewboard.kde.org/r/128043/</a><br />
which is itself a follow-up to discussion on the plasma-devel ML.</p>

<p>The main goal behind the patch is to ensure audio notifications will sound when they are not configured properly, for instance because the necessary files are missing when not running under a Plasma session.</p>

<p>The current state of the RR can be summarised as "awaiting brilliant ideas".</p>

<p>When not running under a Plasma session it would be preferable if generic alert sounds are (can be) delivered using the default system sound. The easiest way to do that is to call QApplication::beep() ; at least on OS X this will play the user's selected alert sound.</p>

<p>I have implemented this for now by assuming the existence of a virtual (relative) beep sound file that always plays the default sound.</p>

<p>In addition, this patch provides better warning feedback if a soundfile isn't found (making it easier to figure out which alert is misconfigured), and will play the sysbeep in that case (so that the alert doesn't go lost if it's sound-only).<br />
Finally, no Phonon::AudioOutput object is created until that is actually required.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R289 KNotifications</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5070" rel="noreferrer">https://phabricator.kde.org/D5070</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/notifybyaudio.cpp</div></div></div><br /><div><strong>To: </strong>rjvbb, Frameworks<br /><strong>Cc: </strong>kde-mac, Frameworks<br /></div>