Fix Codex icon on progressive buttons in dark mode

Bug: T340258
Bug: T364254
Change-Id: Ia4b7db3cfec07a83d74893d0600fd1ec1f9b772d
This commit is contained in:
Simon Legner 2024-05-04 22:37:24 +03:00 committed by Simon04
parent b124f28092
commit 2f05b66988
2 changed files with 4 additions and 2 deletions

View file

@ -84,7 +84,8 @@ const { EmbedFileFormatter, Utils } = require( 'mmv.ui.ondemandshareddependencie
.attr( 'target', '_blank' )
.attr( 'download', '' )
.addClass( 'cdx-button cdx-button--weight-primary cdx-button--action-progressive cdx-button--fake-button cdx-button--fake-button--enabled' )
.html( '<span class="cdx-button__icon cdx-button__icon--download" aria-hidden="true"></span>' + mw.message( 'multimediaviewer-download' ).text() )
// FIXME T364254: using "notheme" here since codex does not apply @color-inverted-fixed instead of @color-inverted for progressive/destructive button
.html( '<span class="cdx-button__icon cdx-button__icon--download notheme" aria-hidden="true"></span>' + mw.message( 'multimediaviewer-download' ).text() )
.appendTo( $container );
}

View file

@ -101,7 +101,8 @@ const UiElement = require( './mmv.ui.js' );
}
$button.empty()
.append( $( '<span>' ).addClass( 'cdx-button__icon' ) )
// FIXME T364254: using "notheme" here since codex does not apply @color-inverted-fixed instead of @color-inverted for progressive/destructive button
.append( $( '<span>' ).addClass( 'cdx-button__icon notheme' ) )
.append( mw.message( 'multimediaviewer-repository-local' ).text() )
.attr( 'href', descriptionUrl );