<table><tr><td style="">Fuchs 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/D26861">View Revision</a></tr></table><br /><div><div><p>Looks good to me, thank you very much for the work :-)</p>
<p>Sidesnotes that I mentioned in the group, just so they are documented here:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">I think for a clickable action it can become too small, so it takes too much aiming. However, it doesn't get in the way, and people annoyed by it can just disable it completely, so that's probably fine</li>
</ul>
<ul class="remarkup-list">
<li class="remarkup-list-item">Icon wise, given that it can still overlap with the app icon, I would prefer something that has a contrasting border, so e.g. a dark audio indicator is still visible on a dark app icon. But that can probably be addressed in a separate task, especially as it also affects other icons we have</li>
</ul></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/D26861">https://phabricator.kde.org/D26861</a></div></div><br /><div><strong>To: </strong>gvgeo, VDG, Plasma, hein, ngraham<br /><strong>Cc: </strong>broulik, Fuchs, ndavis, filipf, cblack, plasma-devel, Orage, 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>