wikimedia/mediawiki-skins-Vector
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/skins/Vector.git synced 2024-11-27 17:10:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixes: Echo icon not visible after click

Browse source 
Due to T343838, we need to adjust the Echo icon after clicking
so it renders consistently with Codex. This is a longstanding bug
and a slight adjustment to how we approach the icon now that Codex
and Vector 2022 use mask-image for their icons.

Bug: T373936
Change-Id: Icdb9cc49c157cb5cbf13fe1f7d5e91b7e986e92e
This commit is contained in:
Jon Robson 2024-09-04 10:41:48 -07:00 committed by Jdlrobson
parent b0b269e2cd
commit ffca370fe8
2 changed files with 11 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
resources/skins.vector.js/echo.js
View file

@ -23,12 +23,13 @@ function init() {
// which directly sets the contents of the anchor element every update
// which would clear out any icon children that we define
if ( element.id === 'pt-notifications-alert' ) {
anchor.classList.add( 'vector-icon', 'mw-ui-icon-wikimedia-bell' );
anchor.classList.add( 'vector-icon' );
}
if ( element.id === 'pt-notifications-notice' ) {
anchor.classList.add( 'vector-icon', 'mw-ui-icon-wikimedia-tray' );
anchor.classList.add( 'vector-icon' );
}
anchor.classList.remove( 'oo-ui-icon-bell', 'oo-ui-icon-tray' );
// Workaround T343838
anchor.classList.add( 'skin-invert' );
} );
}

7
skinStyles/ext.echo.styles.badge.less
View file

@ -49,6 +49,13 @@
display: inline-flex;
margin: 0;
opacity: 1;
// Workaround T343838.
// The OOUI icon uses the ::after pseudo element so will not work with mask image.
// Because of this we disable the mask image rule.
/* stylelint-disable declaration-no-important */
-webkit-mask-image: none !important;
mask-image: none !important;
/* stylelint-enable declaration-no-important */
// T341490 We are incorrectly applying Codex icon and button styles on the
// same element because ULS frequently overwrites the contents of the element
// so it is too difficult to maintain the correct markup. Because we are mixing