fix(storefront): BCTHEME-301 Header content placed out of the header …

…block on mobile (#1907)
bigcommerce · Nov 18, 2020 · 93849c4 · 93849c4
1 parent 4d4f9f2
commit 93849c4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 - HTML Entity displayed as is via system/error message on a Storefront. [#1888](https://github.com/bigcommerce/cornerstone/pull/1888)
 
 ## Draft
+- Header content placed out of the header block on mobile. [#1907](https://github.com/bigcommerce/cornerstone/pull/1907)
 - Shoppers are not anchor-linked to reviews on PDPs if product description tabs are enabled. [#1883](https://github.com/bigcommerce/cornerstone/pull/1883)
 - Fixed text contrast for brand name on Cart page. [#1882](https://github.com/bigcommerce/cornerstone/pull/1882)
 - Add sufficient contrast for Upsell Banners in Cornerstone Theme according to AA Standard. [#1891](https://github.com/bigcommerce/cornerstone/pull/1891)

diff --git a/assets/scss/components/stencil/adminBar/_adminBar.scss b/assets/scss/components/stencil/adminBar/_adminBar.scss
@@ -32,7 +32,7 @@
 // =============================================================================
 
 body.hasAdminBar {
-    padding-top: 101px; // 55px (Mobile Header) + 46px (Admin Bar)
+    padding-top: $header-height; // (Mobile Header)
 
     @include breakpoint("medium") {
         padding-top: 0;
@@ -42,16 +42,12 @@ body.hasAdminBar {
         }
 
         .header {
-            padding-top: 0;
+            padding-top: $adminBar-height;
         }
     }
 
-    .header {
-        padding-top: $adminBar-height;
-    }
-
     .navPages-container.is-open {
-        padding-top: 101px;
+        padding-top: $header-height;
     }
 }