Mattflaschen has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/282714

Change subject: FooterNoticeWidget: Set image height explicitly
......................................................................

FooterNoticeWidget: Set image height explicitly

If we don't do this, the footer becomes slightly taller
when the image loads, which makes the popup clip incorrectly.

Change-Id: Ica84d704692676d884b847f8fe01c49319f04931
(cherry picked from commit a4ea2de1239fd6c9bea2c299285f11f9d751cbef)
---
M modules/ooui/mw.echo.ui.FooterNoticeWidget.js
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo 
refs/changes/14/282714/1

diff --git a/modules/ooui/mw.echo.ui.FooterNoticeWidget.js 
b/modules/ooui/mw.echo.ui.FooterNoticeWidget.js
index dbffd5c..e34f2bc 100644
--- a/modules/ooui/mw.echo.ui.FooterNoticeWidget.js
+++ b/modules/ooui/mw.echo.ui.FooterNoticeWidget.js
@@ -23,7 +23,7 @@
                if ( config.iconUrl ) {
                        $icon = $( '<div>' )
                                .addClass( 'mw-echo-ui-footerNoticeWidget-icon' 
)
-                               .append( $( '<img>' ).attr( 'src', 
config.iconUrl ) );
+                               .append( $( '<img>' ).attr( { src: 
config.iconUrl, width: 30, height: 30 } ) );
 
                        $row.append( $icon );
                }

-- 
To view, visit https://gerrit.wikimedia.org/r/282714
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ica84d704692676d884b847f8fe01c49319f04931
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: wmf/1.27.0-wmf.20
Gerrit-Owner: Mattflaschen <mflasc...@wikimedia.org>
Gerrit-Reviewer: Catrope <roan.katt...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to