<table><tr><td style="">abetts 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/D17073">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/D17073#378898" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17073#378898</a>, <a href="https://phabricator.kde.org/p/trmdi/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@trmdi</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>How about this? I'd send a new patch if it's possible to change 2 things in one patch.<br />
 <a href="https://phabricator.kde.org/F6486761" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6486761: test.png</a></p></div>
</blockquote>

<p>I was thinking here that you might have to visually prioritize the content and controls.</p>

<p>For example, instead of having the playback buttons on a small corner on the bottom right, have them centered and a little bigger. I would remove the duplicate song title. In the end, the user will probably want to interact with the popup more than trying to read the labels. In my mind, the user appreciates the content but his primary action is to work with the controls. Therefore, those controls should have more visual prominence.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17073">https://phabricator.kde.org/D17073</a></div></div><br /><div><strong>To: </strong>trmdi, hein, broulik, ngraham, VDG<br /><strong>Cc: </strong>filipf, ndavis, abetts, anthonyfieroni, plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart<br /></div>