Catrope has uploaded a new change for review.

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

Change subject: NotificationBadgeWidget: Fix button icon positioning for change 
in OOUI
......................................................................

NotificationBadgeWidget: Fix button icon positioning for change in OOUI

OOUI now uses absolute positioning for these icons, which breaks
the centered text model we'd like to use. Longer-term we should
use a different widget for these buttons, or perhaps a widget of
our own, but for now let's hack around that by repositioning
the icons.

Change-Id: I8cc1fd69eeedde00facec80d82a21d5cc5610992
---
M modules/ooui/styles/mw.echo.ui.NotificationBadgeWidget.less
1 file changed, 8 insertions(+), 0 deletions(-)


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

diff --git a/modules/ooui/styles/mw.echo.ui.NotificationBadgeWidget.less 
b/modules/ooui/styles/mw.echo.ui.NotificationBadgeWidget.less
index 09c018b..f99ae83 100644
--- a/modules/ooui/styles/mw.echo.ui.NotificationBadgeWidget.less
+++ b/modules/ooui/styles/mw.echo.ui.NotificationBadgeWidget.less
@@ -32,6 +32,14 @@
                                line-height: 4em;
                                margin: 0;
                                border: 0;
+
+                               .oo-ui-iconElement-icon {
+                                       // HACK: these are absolutely 
positioned in OOUI, and our line-height: 4em;
+                                       // rule breaks that.
+                                       // FIXME: We should really use a 
different widget here
+                                       left: 3em;
+                                       top: 1.5em;
+                               }
                        }
                }
        }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8cc1fd69eeedde00facec80d82a21d5cc5610992
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to