<table><tr><td style="">nicolasfella 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/D10796" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I like the color, but I'm not sure about the disabling.</p>

<p>Right now, only the playPause button is disabled, but what about the other buttons?.  If we go with this approach, we should disable all buttons instead of removing them. Why should the playPause button be any different?</p>

<p>IMHO we should remove buttons if the functionality is unavailable, disabled buttons might look a bit broken. The only place where it might make sense to disable a button is when either next or previous is unavailable (probably because we are at the beginning/end of a track list) to keep symmetry.</p>

<p>The reason for you to make this patch in the first place was because Plasma Browser Integration was causing glitches, but that was because it didn't comply to the specification and is fixed now.</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/D10796" rel="noreferrer">https://phabricator.kde.org/D10796</a></div></div><br /><div><strong>To: </strong>mtijink, KDE Connect, VDG, albertvaka<br /><strong>Cc: </strong>albertvaka, nicolasfella<br /></div>