Add skin-invert class to two icons

[jhsoby]

In order to support dark mode in Vector-2022 (and other places?). This pull request is tied with this Gerrit change to the ULS extension; that change depends on this one.

[jdlrobson]

I think this would break the ULS in current form given ULS is using the skin-theme class now? I suspect this was written prior to that change.

Can you confirm?

[jdlrobson]

Applying the classes to ULS this is what I would expect to see (I'm not setup to test the library so perhaps I'm misunderstanding how this is put together and maybe there are other local changes I am missing?)

[jhsoby]

I made this PR as part of https://gerrit.wikimedia.org/r/c/mediawiki/extensions/UniversalLanguageSelector/+/1098036 … not sure how to add "Depends-On" across Gerrit/GitHub, sorry (but I can just add it "manually" to the commit message of course).

[jdlrobson]

Ah got it! Thanks for that context!

[jhsoby]

Could we merge this? My understanding from Niklas's comment on Gerrit is that we will need to pull the changes from this GitHub repo into the ULS extension "manually", so merging this here won't immediately have any effect on-wiki. And these changes are needed for the Gerrit patch to work properly.

[jdlrobson]

Whoever merges this should be able to commit to making the necessary preparations in production extensions that use this library (as someone unfamiliar with this code and consumers I can't be that person but the change makes sense in the context of the other changes @jhsoby has proposed).

[jhsoby]

Actually, please hold on a bit – I think I found a way to do this in the extension without relying on skin-invert at all.