<table><tr><td style="">filipf requested changes to this revision.<br />filipf added a comment.<br />This revision now requires changes to proceed.
</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/D26739">View Revision</a></tr></table><br /><div><div><p>I think what Noah means is that the media controls row should perfectly touch the left, right and bottom border. The margins in this patch are too big so they effectively create another frame in a frame.</p>

<p>But what's more important is that having the highlight effect above everything just doesn't work with themes that have a filled highlight style. We should scrap that idea.</p>

<p><a href="https://phabricator.kde.org/F7889878" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7889878: Screenshot_20200118_120047.png</a></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/D26739">https://phabricator.kde.org/D26739</a></div></div><br /><div><strong>To: </strong>ngraham, Plasma, VDG, filipf<br /><strong>Cc: </strong>filipf, ndavis, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>