<table><tr><td style="">ngraham updated this revision to Diff 65888.<br />ngraham marked an inline comment as done.<br />ngraham 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/D23389">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Fix it by using <tt style="background: #ebebeb; font-size: 13px;">Button</tt> from PlasmaComponents3, which handles exclusive checkable state properly (it's impossible to do with PC2)</li>
<li class="remarkup-list-item">Use an outline-style star for the non-default item's button</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R115 Plasma Audio Volume Applet</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D23389?vs=65354&id=65888">https://phabricator.kde.org/D23389?vs=65354&id=65888</a></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D23389</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23389">https://phabricator.kde.org/D23389</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applet/contents/ui/ListItemBase.qml<br />
src/kcm/package/contents/ui/DefaultDeviceButton.qml<br />
src/kcm/package/contents/ui/DeviceListItem.qml<br />
src/kcm/package/contents/ui/MuteButton.qml</div></div></div><br /><div><strong>To: </strong>ngraham, drosca, Plasma, VDG, ndavis<br /><strong>Cc: </strong>broulik, GB_2, alexde, ndavis, anthonyfieroni, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>