<table><tr><td style="">apol added inline comments.
</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/D10703">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D10703#inline-53344">View Inline</a><span style="color: #4b4d51; font-weight: bold;">batteryplugin.cpp:79</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; ">    <span style="color: #aa4000">if</span> <span class="p">(</span> <span class="n">thresholdEvent</span> <span style="color: #aa2211">==</span> <span class="n">ThresholdBatteryLow</span> <span style="color: #aa2211">&&</span> <span style="color: #aa2211">!</span><span class="n">isCharging</span> <span class="p">)</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #304a96">#ifdef SAILFISHOS</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">Notification</span> <span style="color: #aa2211">*</span><span class="n">notification</span> <span style="color: #aa2211">=</span> <span style="color: #aa4000">new</span> <span class="n">Notification</span><span class="p">(</span><span style="color: #aa4000">this</span><span class="p">);</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Could we use something like Daemon::reportError to abstract such notifications?</p>

<p style="padding: 0; margin: 8px;">Something like`Daemon::instance()->sendNotification(notificationName, icon, title, text)` could also be used in other places.</p>

<p style="padding: 0; margin: 8px;">Another thing to consider is to adapt knotifications to work on sailfish would be great (solving this problem elsewhere too). Or maybe considering using snorenotify. Another possibility would be to have a proxy for xdg notifications for sailfish.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D10703#inline-53095">View Inline</a><span style="color: #4b4d51; font-weight: bold;">piggz</span> wrote in <span style="color: #4b4d51; font-weight: bold;">sendnotificationsplugin.cpp:29</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">It is just so that on sfos I can set the plugin to be enabled by default.  Is there a better way?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">There should be. Having this line is ugly as is, let alone having to ifdef it on every plugin.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10703">https://phabricator.kde.org/D10703</a></div></div><br /><div><strong>To: </strong>piggz, KDE Connect, apol, mtijink<br /><strong>Cc: </strong>kossebau, mtijink, vonreth, apol, KDE Connect, nicolasfella, adeen-s, SemperPeritus, ahmedbesbes, daniel.z.tg, jeanv, aboudhar, seebauer, bugzy, MayeulC, menasshock, ach<br /></div>