<table><tr><td style="">ngraham accepted this revision.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</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/D26271">View Revision</a></tr></table><br /><div><div><p>I'm good with this now as an initial implementation of the mockup. However we're a bit close to the Plasma 5.18 tagging date (January 16th). My gut instinct is to land these early in the 5.19 cycle rather than rushing to get everything into 5.18 and potentially shipping with un-noticed regressions.</p>

<p>On the other hand, the UI to switch the default device is much nicer with this patch series, and we've received a lot of user complaints about the big buttons I added to implement that in 5.17 (sorry).</p>

<p>Thoughts?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R115 Plasma Audio Volume Applet</div></div></div><br /><div><strong>BRANCH</strong><div><div>layout (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26271">https://phabricator.kde.org/D26271</a></div></div><br /><div><strong>To: </strong>gvgeo, VDG, Plasma, manueljlin, drosca, ngraham<br /><strong>Cc: </strong>ndavis, filipf, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>