<table><tr><td style="">ngraham added subscribers: GB_2, ndavis.<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/D24407">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/D24407#541824" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D24407#541824</a>, <a href="https://phabricator.kde.org/p/mmustac/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mmustac</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Not related to this patch, but...<br />
 While looking at these screenshots I find it not self-explaining but more confusing that both buttons look the same, except one of them is in "active" state.<br />
 I would propose to change the icon for the non-default device to an empty star, with just an outline to be more clear which device is the current default.</p></div>
</blockquote>

<p>I actually did that in the original diff after <a href="https://phabricator.kde.org/p/GB_2/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@GB_2</a> suggested it and quite liked the idea. However we ran into a  potential issue after <a href="https://phabricator.kde.org/p/ndavis/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ndavis</a> rightly pointed out that we couldn't guarantee that for any given icon name pair in any icon theme, one would have a filled star and the other would have an empty star. This is the way it is in the Breeze icon theme, but it's not necessarily set up like that in other icon themes.</p>

<p>Maybe we can revisit that later or put appropriate icons in the plasma icon theme or something. I'd like to find a way to implement that, too.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R115 Plasma Audio Volume Applet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24407">https://phabricator.kde.org/D24407</a></div></div><br /><div><strong>To: </strong>ngraham, VDG, Plasma, drosca, broulik, sitter<br /><strong>Cc: </strong>ndavis, GB_2, mmustac, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>