<table><tr><td style="">kossebau 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">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/D10796#218320" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D10796#218320</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;">@nicolasfella</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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></div>
</blockquote>

<p>From my usage experience (though with developer testing mode, might not completely reflect normal usage):<br />
not only for symmetry, but also because the Next/Previous buttons disappearing randomly also results in unwanted behaviour. E.g. if you quickly press a few times the Previous button to get to the first media item, once you reach the first item, the Previous button has disappeared and the space is taken by the Next button instead. But as one is still tapping a few more times, as one did not know in advance how much tapping is needed, one ends up switching between first and second media item (due to buttons moving under your finger), and by half the chance one ends up with the second, not the first :) (same with going to last).</p>

<p>So when it comes to Next/Previous buttons, I would propose to show them as pair always (if any shown), and the one currently not working (due to CanGo*=false) just shown as disabled.</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">https://phabricator.kde.org/D10796</a></div></div><br /><div><strong>To: </strong>mtijink, KDE Connect, VDG, albertvaka<br /><strong>Cc: </strong>kossebau, albertvaka, nicolasfella<br /></div>