From 77eeb213bd3594bb42b185a43f89748470865568 Mon Sep 17 00:00:00 2001 From: Tom Evans Date: Fri, 13 Mar 2026 15:04:36 +0000 Subject: [PATCH 1/2] fix: changing lightbox scroll-fix styles We now use `removeProperty` to remove them rather than resetting to eg `auto` because that was breaking the nav --- src/components/Organisms/DynamicGallery/_ScrollFix.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/components/Organisms/DynamicGallery/_ScrollFix.js b/src/components/Organisms/DynamicGallery/_ScrollFix.js index 8584dc15a..9ff50bee8 100644 --- a/src/components/Organisms/DynamicGallery/_ScrollFix.js +++ b/src/components/Organisms/DynamicGallery/_ScrollFix.js @@ -36,8 +36,8 @@ export default function ScrollFix() { // this is then removed when the component unmounts useLayoutEffect(() => { function resetScrollbar() { - document.documentElement.style.setProperty('overflow', 'auto'); - document.documentElement.style.setProperty('padding-right', '0px'); + document.documentElement.style.removeProperty('overflow'); + document.documentElement.style.removeProperty('padding-right'); } // check that the page content is longer than the viewport From 41421b95ebbb9932ce6e2ca4b630024203d6c20f Mon Sep 17 00:00:00 2001 From: Tom Evans Date: Fri, 13 Mar 2026 16:09:02 +0000 Subject: [PATCH 2/2] targeting body to match nav behaviour --- src/components/Organisms/DynamicGallery/_ScrollFix.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/components/Organisms/DynamicGallery/_ScrollFix.js b/src/components/Organisms/DynamicGallery/_ScrollFix.js index 9ff50bee8..f86c854f6 100644 --- a/src/components/Organisms/DynamicGallery/_ScrollFix.js +++ b/src/components/Organisms/DynamicGallery/_ScrollFix.js @@ -36,16 +36,16 @@ export default function ScrollFix() { // this is then removed when the component unmounts useLayoutEffect(() => { function resetScrollbar() { - document.documentElement.style.removeProperty('overflow'); - document.documentElement.style.removeProperty('padding-right'); + document.body.style.removeProperty('overflow'); + document.body.style.removeProperty('padding-right'); } // check that the page content is longer than the viewport - if (document.documentElement.scrollHeight <= window.innerHeight) { + if (document.body.scrollHeight <= window.innerHeight) { resetScrollbar(); } else { - document.documentElement.style.setProperty('overflow', 'hidden'); - document.documentElement.style.setProperty( + document.body.style.setProperty('overflow', 'hidden'); + document.body.style.setProperty( 'padding-right', `${scrollPadding}px` );