<table><tr><td style="">davidedmundson 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/D19392">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The problem with using -symbolic for all monochrome icons is that we'd have to create color icons without -symbolic to replace the old monochrome icons without -symbolic.</p></blockquote>

<p>Why would breeze need two versions?</p>

<p>If an app requests -symbolic you get a black icon which we know we can colourise.<br />
If the app doesn't request -symbolic you get an icon which happens to be all black.</p>

<p>Also it's worth noting there's an FD.O icon spec rule (that will apply for all QIconEngines hopefully) if an app requests  "foo-bar"   and "foo-bar" doesn't exist, it will automatically load "foo"  <br />
Us requesting "-symbolic" at an app level won't break anything. (though obviously we'll have to port a bunch of app code)</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>this tough is about doing something about it in KIconLoader which is completely ortogonal to this patch.</p></blockquote>

<p>That's absolutely not what this is about, because as you note putting it in KIconLoader wouldn't fix anything at all!</p>

<p>We've got code in Plasma that colours icons. Then we put code in KIconLoader because we didn't account for widgets, now we're putting code here because we didn't account for non-plasma - and we know this is a bodge that's going to get replaced as it doesn't work in a lot of cases.<br />
If we are running Kirigami on gnome, you probably want to use a gnome icon set at which point this all breaks. I think the app would /need/ to request the "-symbolic" icon?</p>

<p>I'm fully aware I'm being difficult, but I'm doing it because these half solutions keep going on forever and I'm trying to save us some time in the future. <br />
If we do want changes in GTK4 and Qt6 we should be thinking about them now.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19392">https://phabricator.kde.org/D19392</a></div></div><br /><div><strong>To: </strong>mart, Kirigami<br /><strong>Cc: </strong>ngraham, GB_2, ndavis, VDG, cfeck, davidedmundson, plasma-devel, domson, dkardarakos, apol, mart, hein<br /></div>