Ideally this change would have been made in a way that regenerating the icons with make_icons.py maintains the explicit 'style="fill:black"', but I see that backend_svg.py has a cutout for black text and explicitly saves some output by not outputting the color in that case, so I guess it's not really worth the trouble.

@QuLogic do you know if this will interfere with gtk's "symbolic" icons though?

I see the filter for black text inbackend_svg.py:

        if color != '#000000':            style['fill'] = color

I assume we don't want to change that.

I was not familiar withmake_icons.py. If we want to preserve its functionality intact, then we could hack the functionmake_icons in the script to make a string replacement to add to the right files after they are saved in../lib/matplotlib/mpl-data/images. We could make this change as part of this PR.

I modified the script. It feels a little hacky since it performs a purely syntactic search and replace with no svg model underlying it, but I think it does the job.

I tested it in my system where the svg backend produces very different icon code from what is currently committed, although the icons appear visually indistinguishable. So it should definitely work on systems that produce the svg code that is currently committed.

In case this turns out to be a problem for gtk, I guess the "base" files couldnot include the explicit foreground color but just have a xml marker-comment as to where the wx backend should insert the foreground color ("\> <!-- insert color in previous tag --> ->" ...\>).

From my testing, it appears that this does not break GTK3 dark mode. GTK4 dark mode is already broken because it uses some other method that hasn't been implemented by us.

@jmoraleda Can you squash? (Not compulsory.)

Thanks for the PR!