From 0867ff03aa5ed6c6ead84d9a95569f9d97f43364 Mon Sep 17 00:00:00 2001 From: Stephen Niedzielski Date: Fri, 2 Aug 2019 14:35:57 -0600 Subject: [PATCH] [fix] [notifications] fix icon width When notifications are present, the wrong icon width was used. Set the icon width and remove a useless class, main-header-button. https://codesearch.wmflabs.org/search/?q=main-header-button&i=nope&files=&repos= Change-Id: Ie274ffcc93cf576f36eb0fccee80ecd3d4ac3a21 --- includes/skins/userNotifications.mustache | 2 +- resources/skins.minerva.loggedin.styles/styles.less | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/includes/skins/userNotifications.mustache b/includes/skins/userNotifications.mustache index df44d28..45df9eb 100644 --- a/includes/skins/userNotifications.mustache +++ b/includes/skins/userNotifications.mustache @@ -1,6 +1,6 @@ {{^hasNotifications}} + class="{{notificationIconClass}} user-button" id="user-notifications"> {{title}} {{/hasNotifications}} diff --git a/resources/skins.minerva.loggedin.styles/styles.less b/resources/skins.minerva.loggedin.styles/styles.less index 0ca61f5..ad459f4 100644 --- a/resources/skins.minerva.loggedin.styles/styles.less +++ b/resources/skins.minerva.loggedin.styles/styles.less @@ -14,6 +14,10 @@ } } +.minerva-user-notifications { + width: @menuButtonIconSize; +} + .notification-count { @circleSize: 24px; @borderSize: 2px;