<table><tr><td style="">ndavis accepted this revision.<br />ndavis 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/D26555">View Revision</a></tr></table><br /><div><div><p>Icons in the desktop theme that are made to replace icons in the icon theme need to be kept up to date with the icon theme. I'm accepting this as it is, but if <a href="https://phabricator.kde.org/p/davidre/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@davidre</a> thinks we should continue to support the use of the <tt style="background: #ebebeb; font-size: 13px;">nepomuk</tt> icon name, then include both in the desktop theme. If this seems inelegant, that's because comprehensive icon support is not very pretty, technically. breeze-icons is filled with tons of symlinks for compatibility and there's really nothing we can do about it.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26555">https://phabricator.kde.org/D26555</a></div></div><br /><div><strong>To: </strong>The-Feren-OS-Dev, Plasma, VDG, ndavis<br /><strong>Cc: </strong>ndavis, davidre, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>