<table><tr><td style="">mtijink marked an inline comment as done.<br />mtijink added a comment.
</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/D9266" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D9266#178720" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D9266#178720</a>, <a href="https://phabricator.kde.org/p/nicolasfella/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@nicolasfella</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Works well so far :) Some small things I noticed: Toggling the option doesn't activate/deactivate the notification immediately.</p></div>
</blockquote>

<p>I know (should have mentioned that, though). It's because the plugin doesn't reload and doesn't get notified in any way of the preference change. I figured that this is the simplest solution, as the notification appears/disappears on the next track/device/player list changes. In general, I think it's good if the plugin can be notified of such changes, or automatically reloads, but I think that's out of scope for this diff.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>When my laptop screen turns off plasma pauses the media player, but the notification is still in play mode. Both are not critical, but a fix would be nice if it's not too complex.</p></blockquote>

<p>Does that close the connection? If so, KDE Connect generally does not notice that immediately, so nothing this code can do about that. Closed connections that KDE Connect knows about are handled correctly, as far as I know. Could you supply some more information so I can look into it?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9266" rel="noreferrer">https://phabricator.kde.org/D9266</a></div></div><br /><div><strong>To: </strong>mtijink, KDE Connect<br /><strong>Cc: </strong>nicolasfella<br /></div>