Skip to content

fix(root): show notice when JavaScript is disabled - #1894

Open
sylvesterkaczmarek wants to merge 1 commit into
mi6:developfrom
sylvesterkaczmarek:fix/186-no-js-notice
Open

sylvesterkaczmarek wants to merge 1 commit into
mi6:developfrom
sylvesterkaczmarek:fix/186-no-js-notice

Conversation

@sylvesterkaczmarek

@sylvesterkaczmarek sylvesterkaczmarek commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Add a site-wide notice for users browsing the guidance site with JavaScript disabled.

The notice is rendered within the SSR layout using a native <noscript> element, so the existing page remains visible below it when JavaScript is unavailable.

Change

  • render the notice above the existing page navigation within the SSR layout;
  • explain that JavaScript is required for the best experience and full site functionality;
  • style the notice using existing spacing and typography tokens and system colours.

Closes #186

@GCHQ-Developer-530

Copy link
Copy Markdown
Contributor

Please can you rebase this pull request?

@sylvesterkaczmarek

Copy link
Copy Markdown
Contributor Author

Rebased onto the latest develop. Thanks!

Comment thread gatsby-ssr.js

@GCHQ-Developer-299 GCHQ-Developer-299 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From my understanding, this ticket requests that an alert be added to every page that warns users about disabling javascript, while still displaying the page below.

This full replacement of the site with a message does not fulfil the requirements

@CLAassistant

CLAassistant commented Sep 12, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@sylvesterkaczmarek

Copy link
Copy Markdown
Contributor Author

Thanks for the review. I’ve updated this so the <noscript> notice is rendered within the SSR layout above navigation, while the existing page remains visible below it. I’ve also rebased onto current develop and rerun formatting, type checks, lint, tests, and the production build. Happy to adjust anything else.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants