From f05f7b26616b3a8c39b2cff00844e67678215c84 Mon Sep 17 00:00:00 2001 From: Tom Byers Date: Thu, 28 Nov 2019 11:27:24 +0000 Subject: [PATCH] Return main content area to position: static Was set to `position: relative` to solve an issue on the produce page. This introduced other issues with the sticky JS and the fix is only needed for the product page so this moves it to that CSS. The previous fix was applied here: https://github.com/alphagov/notifications-admin/commit/9441dd0b375ecb26dd42949a14fb41dad5a1bebb --- app/assets/stylesheets/govuk-frontend/overrides.scss | 8 -------- app/assets/stylesheets/views/product-page.scss | 5 +++++ 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/app/assets/stylesheets/govuk-frontend/overrides.scss b/app/assets/stylesheets/govuk-frontend/overrides.scss index 30ca08c30..4269c4181 100644 --- a/app/assets/stylesheets/govuk-frontend/overrides.scss +++ b/app/assets/stylesheets/govuk-frontend/overrides.scss @@ -11,14 +11,6 @@ } -// The GOV.UK Frontend header component wraps content that is `position: relative` -// This changes its z-index position, putting it above the `
` section -// We need the `
` section to be above it, like the default order, to hide the theme colour -// bar on full-width pages -.govuk-main-wrapper { - position: relative; -} - // Additional padding-bottom override, following the GOV.UK Frontend spacing scale: // https://design-system.service.gov.uk/styles/spacing/#the-responsive-spacing-scale .govuk-\!-padding-bottom-12 { diff --git a/app/assets/stylesheets/views/product-page.scss b/app/assets/stylesheets/views/product-page.scss index 21b9eff30..afc0d09bb 100644 --- a/app/assets/stylesheets/views/product-page.scss +++ b/app/assets/stylesheets/views/product-page.scss @@ -2,6 +2,11 @@ &-intro { + // The GOV.UK Frontend header component wraps content that is `position: relative` + // This changes its z-index position, putting it above the `
` section in the stacking order + // We need the `
` section to be above it, like the default order, so when we apply the + // negative margin-top it overlaps the theme bar at the bottom of the header + position: relative; margin: -10px 0 $gutter * 1.5 0; padding: 0 0 $gutter * 2 0; background: $govuk-blue;