From 625720420ea49f2a0d14aad9a6679541f77c7d69 Mon Sep 17 00:00:00 2001 From: jdlrobson Date: Tue, 9 Jul 2019 09:51:43 -0700 Subject: [PATCH] Set header height to fixed pixel Bug: T195795 Change-Id: I4eed89f6a64eeb369c1ab0bb47f0462de7b0ceb3 --- minerva.less/minerva.variables.less | 3 ++- skinStyles/mobile.startup/Overlay.less | 2 +- skinStyles/mobile.startup/search/SearchOverlay.less | 5 +++-- 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/minerva.less/minerva.variables.less b/minerva.less/minerva.variables.less index 6146182..3d41518 100644 --- a/minerva.less/minerva.variables.less +++ b/minerva.less/minerva.variables.less @@ -44,7 +44,8 @@ @contentMargin: 16px; // Header -@headerHeight: 3.35em; +@headerHeight: 54px; +@headerVerticalPadding: 0.15em; @headerMarginTop: -1px; // used to hide the header border top when a banner is not present @searchBoxWidth: 375/16em; @iconSizeTotal: @iconSize + @iconGutterWidth + @iconGutterWidth; diff --git a/skinStyles/mobile.startup/Overlay.less b/skinStyles/mobile.startup/Overlay.less index e5f8775..ac7119e 100644 --- a/skinStyles/mobile.startup/Overlay.less +++ b/skinStyles/mobile.startup/Overlay.less @@ -25,7 +25,7 @@ } .overlay-header .overlay-title { - padding: 0.15em 0; + padding: @headerVerticalPadding 0; &:last-child { // Reserve space to the right in case the search form in the header or diff --git a/skinStyles/mobile.startup/search/SearchOverlay.less b/skinStyles/mobile.startup/search/SearchOverlay.less index 70d532b..d6cd7fd 100644 --- a/skinStyles/mobile.startup/search/SearchOverlay.less +++ b/skinStyles/mobile.startup/search/SearchOverlay.less @@ -11,9 +11,10 @@ .clear { position: absolute; - top: ( @headerHeight / 2 ) - ( @iconSize / 2 ); + // the icon should take into account overlay-title top padding + // and then be centered + top: ( @iconSize / 2 ) + @headerVerticalPadding; right: 0; - margin-top: @headerMarginTop; } }