<table><tr><td style="">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/D17623">View Revision</a></tr></table><br /><div><div><p>Nice work, these are good icons!</p>
<p>However, I wonder whether we really want to use the <tt style="background: #ebebeb; font-size: 13px;">yast-</tt> prefix for these images, especially the ones without explicit opensuse iconography. I can imagine some of these being useful in other apps' settings dialogs as well, but it would feel weird to use an icon that starts with <tt style="background: #ebebeb; font-size: 13px;">yast-</tt>. Can we not change YaST itself to use different names for its icons? If YaST has recently gained the ability to take icons from icon themes, it seems unreasonable for it to expect icons named <tt style="background: #ebebeb; font-size: 13px;">yast-<something></tt> rather than just using more neutrally-named icons.</p>
<p>For example <tt style="background: #ebebeb; font-size: 13px;">yast-auth-client</tt> could be be named <tt style="background: #ebebeb; font-size: 13px;">auth-kerberos</tt> or something so that any kerberos-related things can use that nice icon.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17623">https://phabricator.kde.org/D17623</a></div></div><br /><div><strong>To: </strong>ndavis, VDG<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>