From a2e3b727bb1d474cf48284d74628b172c1356b67 Mon Sep 17 00:00:00 2001 From: Isarra Date: Thu, 13 Sep 2018 19:36:48 +0000 Subject: [PATCH] Don't hide labels for mobile cactions that have no icon Not necessarily ideal handling in terms of making things fit, but should at least avoid issues of there being no things at all, which is even worse. bug: T202213 Change-Id: I1aeeddbcac838b475372691d8d76c72018f85787 --- resources/extensions/VisualEditor.article.less | 2 ++ resources/screen-mobile.less | 10 ++++++---- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/resources/extensions/VisualEditor.article.less b/resources/extensions/VisualEditor.article.less index e093e9d..4a34d96 100644 --- a/resources/extensions/VisualEditor.article.less +++ b/resources/extensions/VisualEditor.article.less @@ -17,10 +17,12 @@ @media ( max-width: @mobile-width ) { #ca-ve-edit a { + .ca-icon(); .background-image-svg( '../images/pencil-grey.svg', '../images/pencil-grey.png' ); } #ca-edit a { + .ca-icon(); .background-image-svg( '../images/brackets-grey.svg', '../images/brackets-grey.png' ); } } diff --git a/resources/screen-mobile.less b/resources/screen-mobile.less index 8d4cd49..64c6079 100644 --- a/resources/screen-mobile.less +++ b/resources/screen-mobile.less @@ -173,31 +173,33 @@ h1.firstHeading { } } -#mw-page-header-links a { - .ca-icon(); -} - #ca-edit a { + .ca-icon(); .background-image-svg( 'images/pencil-grey.svg', 'images/pencil-grey.png' ); } #ca-history a { + .ca-icon(); .background-image-svg( 'images/clock-grey.svg', 'images/clock-grey.png' ); } #ca-talk a { + .ca-icon(); .background-image-svg( 'images/talk-grey.svg', 'images/talk-grey.png' ); } #t-contributions a { + .ca-icon(); .background-image-svg( 'images/puzzle-grey.svg', 'images/puzzle-grey.png' ); } #ca-addsection a { + .ca-icon(); .background-image-svg( 'images/plus-grey.svg', 'images/plus-grey.png' ); } *[ id^='ca-nstab-' ] a { + .ca-icon(); .background-image-svg( 'images/page-grey.svg', 'images/page-grey.png' ); }