mirror of
https://gerrit.wikimedia.org/r/mediawiki/skins/Vector.git
synced 2024-12-22 04:32:55 +00:00
a6e4370bbd
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
37 lines
1.2 KiB
JavaScript
37 lines
1.2 KiB
JavaScript
/**
|
|
* Upgrades Echo for icon consistency.
|
|
* Undos work inside Echo to replace our button.
|
|
*/
|
|
function init() {
|
|
if ( document.querySelectorAll( '#pt-notifications-alert a, #pt-notifications-notice a' ).length !== 2 ) {
|
|
return;
|
|
}
|
|
|
|
mw.hook( 'ext.echo.NotificationBadgeWidget.onInitialize' ).add( ( badge ) => {
|
|
const element = badge.$element[ 0 ];
|
|
element.classList.add( 'mw-list-item' );
|
|
const anchor = /** @type {HTMLElement} */ ( element.querySelector( 'a' ) );
|
|
anchor.classList.add(
|
|
'cdx-button',
|
|
'cdx-button--weight-quiet',
|
|
'cdx-button--icon-only',
|
|
'cdx-button--fake-button',
|
|
'cdx-button--fake-button--enabled'
|
|
);
|
|
// Icon classes shouldn't go on the same element as button classes
|
|
// However this cant be avoided due to Echo button's implementation
|
|
// 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' );
|
|
}
|
|
if ( element.id === 'pt-notifications-notice' ) {
|
|
anchor.classList.add( 'vector-icon' );
|
|
}
|
|
// Workaround T343838
|
|
anchor.classList.add( 'skin-invert' );
|
|
} );
|
|
}
|
|
|
|
module.exports = init;
|