jenkins-bot has submitted this change and it was merged.

Change subject: Fixing mobile menu so that it displays correctly in no-JS and 
RTL
......................................................................


Fixing mobile menu so that it displays correctly in no-JS and RTL

Bug: 60186
Change-Id: I7aaa4ffc0a5a4fa2e292d7d9971d7103449baf17
---
M less/common/mainmenu.less
1 file changed, 8 insertions(+), 6 deletions(-)

Approvals:
  Kaldari: Looks good to me, approved
  JGonera: Looks good to me, but someone else must approve
  Jdlrobson: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/less/common/mainmenu.less b/less/common/mainmenu.less
index 979f38b..2bd9223 100644
--- a/less/common/mainmenu.less
+++ b/less/common/mainmenu.less
@@ -35,7 +35,7 @@
        &:first-child {
                border-bottom: @menuBorder solid @menuBorderColor;
 
-               li:first-child a {
+               li:first-child {
                        border-top: none;
                }
        }
@@ -44,6 +44,8 @@
                position: relative; // ensure the logout link in beta can be 
position absolute
                font-size: 1.2em;
                text-shadow: 0 1px 0 rgba(0, 0, 0, 0.6);
+               border-bottom: 1px solid #3e3e3e;
+               border-top: 1px solid #717171;
 
                &:hover {
                        background-color: #5C5C5C;
@@ -73,13 +75,12 @@
                a {
                        color: white;
                        display: block;
-                       padding: 13px 10px 11px 42px;
+                       padding: 13px 10px 11px 34px;
                        line-height: 22px;
                        background-repeat: no-repeat;
                        .background-size(24px, 24px);
-                       background-position: 4% 50%;
-                       border-bottom: 1px solid #3e3e3e;
-                       border-top: 1px solid #717171;
+                       background-position: left center;
+                       margin-left: 9px;
                        &:hover {
                                text-decoration: none;
                        }
@@ -132,7 +133,8 @@
                        a {
                                color: #ccc;
                                border: none;
-                               padding: .7em 14px;
+                               padding: .7em 12px;
+                               margin-left: 0;
                        }
                }
        }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7aaa4ffc0a5a4fa2e292d7d9971d7103449baf17
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Kaldari <[email protected]>
Gerrit-Reviewer: JGonera <[email protected]>
Gerrit-Reviewer: Jdlrobson <[email protected]>
Gerrit-Reviewer: Kaldari <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to