<table><tr><td style="">ndavis 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/D23759">View Revision</a></tr></table><br /><div><div><p>A few things to consider:</p>

<p>If a user uses an icon theme besides Breeze (e.g., Papirus), but keeps using the Breeze desktop theme, they could see a mix of icons from their icon theme and icons from the desktop theme. This is actually already the case in different places that this patch doesn't affect, but this would make things a little worse in that regard, unless all icons were removed from the desktop theme.</p>

<p>A more conservative option would be to just add more sizes for the icons in media.svg, but this means we have to maintain the same icons in 2 different repositories. This is also currently the case, but this would make things a little worse in that regard.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23759">https://phabricator.kde.org/D23759</a></div></div><br /><div><strong>To: </strong>ndavis, VDG, Plasma<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>