<table><tr><td style="">davidhurka added a comment.
</td></tr></table><br /><div><div><p>I have to take some short notes here.</p>

<p>I have written a .sed script which we can use to convert icons/ files to icons-dark/ files. It does the same substitutions as my perl script above, but matches only in the stylesheet. That means, fixed colors will not be changed. That should fix things like this: <a href="https://phabricator.kde.org/F8199106" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8199106: devices_64_network-rj45-female.png</a></p>

<p>The script must be called with the -E option, because it uses new POSIX regexes.</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="console" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);"><span style="color: #000080">$ sed -E -f swap_colors.sed -i file/to-change.svg</span></pre></div>

<p><a href="https://phabricator.kde.org/F8200468" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8200468: swap_colors.sed</a><br />
Of course you may give it a better name.</p>

<p>Open questions are:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The icons in applets/ use #314045 as ColorScheme-ViewBackground, altough Icon_Workflow_Tips defines #232629. I think it should really be #232629, because that is the color which is actually used by the Breeze Dark theme. swap_colors.sed substitutes to #232629. The following picture illustrates what that means: (Auto-generated left, original right) <a href="https://phabricator.kde.org/F8200470" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8200470: montage.png</a> I think we can do it how swap_colors.sed does it now, and get rid of icons-dark/applets/.</li>
<li class="remarkup-list-item">Icon_Workflow_Tips defines #232629 for ColorScheme-Text, but many icons have #4d4d4d as ColorScheme-Text, for example those in icons/applets/, but also some in icons-dark/actions/. The latter is a mistake which will be fixed by auto-generating. The former is currently respected by swap_colors.sed, and auto-generated icons correspond to those in icons-dark. (I think the icons should actually use #232629, but than can be fixed independently.)</li>
</ul>

<p>Next steps:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Run the compare-dark-light.pl again with the new swap_colors.sed script, to be sure it does color swapping fine now.</li>
</ul></div></div><br /><div><strong>TASK DETAIL</strong><div><a href="https://phabricator.kde.org/T12855">https://phabricator.kde.org/T12855</a></div></div><br /><div><strong>To: </strong>davidhurka<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, davidhurka, dmenig, manueljlin, Orage, LeGast00n, cblack, konkinartem, ian, jguidon, Ghost6, jraleigh, fbampaloukas, squeakypancakes, alexde, IohannesPetros, GB_2, trickyricky26, michaelh, crozbo, ndavis, firef, bruns, skadinna, aaronhoneycutt, mbohlender<br /></div>