Separate out indicator and icon selectors

Follow up to I584491c280d93f18a48d8c90c4d8a6628f1a672f

Since these selectors are nested, this was generating unexpected
selectors, meaning the invert wasn't applying correctly to indicators.

This is evident on Special:Search

Bug: T365764
Change-Id: I40f8a8ad6598d145a6284c831e0cef944c5d7e75
This commit is contained in:
Jon Robson 2024-06-13 08:16:32 -07:00 committed by Jdlrobson
parent 74d5cd5cfc
commit 71c532e542

View file

@ -21,7 +21,8 @@
* This includes OOUI icons that are not colored (progressive/destructive), checkbox icons, and
* indicator widgets.
*/
@OOUIIconSelectors: ~'.oo-ui-iconElement-icon:not( .oo-ui-image-progressive ):not( .oo-ui-image-destructive ):not( .oo-ui-checkboxInputWidget-checkIcon ), .oo-ui-indicatorElement-indicator ';
@OOUIIconSelectors: ~'.oo-ui-iconElement-icon:not( .oo-ui-image-progressive ):not( .oo-ui-image-destructive ):not( .oo-ui-checkboxInputWidget-checkIcon )';
@OOUIIndicatorSelectors: ~'.oo-ui-indicatorElement-indicator';
/**
* Light mode
@ -58,13 +59,15 @@ html.skin-theme-clientpref-night {
// T365102 invert class specifically for image related elements
html.skin-theme-clientpref-night .skin-invert-image img,
html.skin-theme-clientpref-night .skin-invert,
html.skin-theme-clientpref-night @{OOUIIconSelectors} {
html.skin-theme-clientpref-night @{OOUIIconSelectors},
html.skin-theme-clientpref-night @{OOUIIndicatorSelectors} {
color-scheme: light;
filter: invert( 1 ) hue-rotate( 180deg );
}
html.skin-theme-clientpref-night .notheme,
html.skin-theme-clientpref-night @{OOUIOverrideSelectors} {
html.skin-theme-clientpref-night @{OOUIOverrideSelectors},
html.skin-theme-clientpref-night @{OOUIIndicatorSelectors} {
color-scheme: light;
color: var( --color-base );
}
@ -85,14 +88,16 @@ html.skin-theme-clientpref-night @{OOUIOverrideSelectors} {
}
html.skin-theme-clientpref-os .notheme,
html.skin-theme-clientpref-os @{OOUIOverrideSelectors} {
html.skin-theme-clientpref-os @{OOUIOverrideSelectors},
html.skin-theme-clientpref-os @{OOUIIndicatorSelectors} {
color-scheme: light;
}
// T365102 invert class specifically for image related elements
html.skin-theme-clientpref-os .skin-invert-image img,
html.skin-theme-clientpref-os .skin-invert,
html.skin-theme-clientpref-os @{OOUIIconSelectors} {
html.skin-theme-clientpref-os @{OOUIIconSelectors},
html.skin-theme-clientpref-os @{OOUIIndicatorSelectors} {
color-scheme: light;
filter: invert( 1 ) hue-rotate( 180deg );
}
@ -106,11 +111,14 @@ html.skin-theme-clientpref-night @{OOUIOverrideSelectors} {
* - OOUI icons inside elements with .skin-invert
* - OOUI icons inside elements with .notheme
*/
@{OOUIOverrideSelectors} @{OOUIIconSelectors},
.skin-invert @{OOUIIconSelectors},
.notheme @{OOUIIconSelectors} {
@{OOUIOverrideSelectors},
.skin-invert,
.notheme {
@{OOUIIndicatorSelectors},
@{OOUIIconSelectors} {
filter: none !important; /* stylelint-disable-line declaration-no-important */
}
}
/**
* Font size settings