<table><tr><td style="">mart 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 #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D19392#421950" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D19392#421950</a>, <a href="https://phabricator.kde.org/p/davidedmundson/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@davidedmundson</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><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></div>
</blockquote>
<p>which screws up if it's a dark background</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>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"</p></blockquote>
<p>so if i requested foo-symbolic but foo was returned and is not monochrome, i will get a broken icon.</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>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>
<p>tough is still a theme designer decision to put random monochrome icons in the non -symbolic section, that is a valid decision and i don't want to break it</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><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</p></blockquote>
<p>eh, in kf6 i would love to drop plasmacore.iconitem, having just this item instead.</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>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.</p></blockquote>
<p>well, looking at how the gtk one works, i just vastly prefer our approach, exactly because it styles all icons and doesn't require to have -symbolic technically(and that maps a big part of the system color palette, while the gtk one doesn't).<br />
while the approach they taken is only applied to -symbolic, because it would destroy any normal icon, for the following reason:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">the base class "fg" case which is most of the use case (what usually becomes black) is not indicated with a css class like "warning" or "success" is, but assumed for everything</li>
<li class="remarkup-list-item">in that case any color in the style: attribute of the tag is nuked, it seems with the !important css tag which isn'0t supported by either inkscape nor Qtsvg, so that the stylesheet can be applied (resulting in the screwup of any icon that wasn't explicitly done for it)</li>
</ul>
<p>while instead breeze needs a class set for everything and use fill:currentColor explicitly, so icons that don't want to use it are completely unaffected<br />
(the problem there is that is a bit inconvenient to do because inkscape is quite buggy in saving files using currentColor without breaking stuff)</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>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></blockquote>
<p>so, for gtk4/qt6, the base qiconengine included in qt should support whatever coloring is used, in which case i wouldn't eed to do anything on my part.. tough that's for Qt6.<br />
It's important for me tough that in the final version, any icon can be styled, not only those that end with -symbolic</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>