Merge pull request #948 from alphagov/no-js-cookie-banner

Hide 'Accept Cookies' button when Javascript not available
alphagov · Jun 25, 2019 · 57d8d3c · 57d8d3c
2 parents c6ea893 + aef4cf5
commit 57d8d3c
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,6 +10,7 @@
 ## Unreleased
 
 * Fix confirmation message being read to screenreaders (PR #952)
+* Hide 'Accept Cookies' button when Javascript not available (PR #948)
 
 ## 17.6.0
 

diff --git a/app/assets/stylesheets/govuk_publishing_components/components/_cookie-banner.scss b/app/assets/stylesheets/govuk_publishing_components/components/_cookie-banner.scss
@@ -55,14 +55,16 @@ $govuk-cookie-banner-text-green: #00823b;
   width: 100%;
 
   @include govuk-media-query($from: mobile, $until: desktop) {
-    width: 49%;
-
     &.gem-c-cookie-banner__button-accept {
       float: left;
+      width: 49%;
     }
 
     &.gem-c-cookie-banner__button-settings {
-      float: right;
+      .js-enabled & {
+        float: right;
+        width: 49%;
+      }
     }
   }
 
@@ -75,6 +77,14 @@ $govuk-cookie-banner-text-green: #00823b;
   }
 }
 
+.gem-c-cookie-banner__button-accept {
+  display: none;
+
+  .js-enabled & {
+    display: inline-block;
+  }
+}
+
 .gem-c-cookie-banner__confirmation {
   display: none;
   position: relative;