<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/D18940">View Revision</a></tr></table><br /><div><div><p>Ideally I would like to move all icons out of Plasma themes, because these themes are not 100% self-contained and often use icons from the icon theme anyway, and users get confused as to why changing their icon theme only changes some (but not all) Plasma icons. And it makes life a bit harder for us to maintain icons in two different places.</p>

<p>However that is indeed another discussion and this looks good to me the way it is.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>BRANCH</strong><div><div>battery-icons (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18940">https://phabricator.kde.org/D18940</a></div></div><br /><div><strong>To: </strong>ndavis, VDG, hein, ngraham<br /><strong>Cc: </strong>broulik, ngraham, filipf, trickyricky26, kde-frameworks-devel, hein, michaelh, bruns<br /></div>