chore: Simplify hiding js-only elements

DFE-Digital · Sep 5, 2024 · 9367c39 · 9367c39
1 parent 592a0a6
commit 9367c39
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 27 deletions.
diff --git a/src/Dfe.ContentSupport.Web/Views/Content/CsIndex.cshtml b/src/Dfe.ContentSupport.Web/Views/Content/CsIndex.cshtml
@@ -76,28 +76,4 @@
             addPrintButtonEventListener();
         </script>
     }
-
-    <script nonce="@Context.Items["nonce"]" defer>
-        const hiddenClass = "govuk-visually-hidden";
-
-        //Some elements should remain hidden
-        const idsToNotShow = ["support-links"];
-
-        //Remove hidden visibility CSS class if JS enabled
-        const removeHiddenClassFromElements = () => {
-            const hiddenElements = Array.from(document.querySelectorAll(`.${hiddenClass}`));
-
-            for(const element of hiddenElements){
-                if(idsToNotShow.indexOf(element.id) > -1){
-                    continue;
-                }
-
-                element.classList.remove(hiddenClass);
-                //Also unset "aria-hidden" attribute
-                element.ariaHidden = undefined;
-            }
-        }
-
-        removeHiddenClassFromElements();
-    </script>
-}
+}
diff --git a/src/Dfe.ContentSupport.Web/Views/Shared/_CsLayout.cshtml b/src/Dfe.ContentSupport.Web/Views/Shared/_CsLayout.cshtml
@@ -69,6 +69,13 @@
     <script defer type="module" nonce="@Context.Items["nonce"]">
         import * as govuk from '/js/govuk-frontend.min.js';
         govuk.initAll();
+
+        function showHiddenElement(element) {
+            element.classList.remove("govuk-visually-hidden");
+            element.ariaHidden = undefined;
+        }
+        // unhide all js-only elements when js is enabled
+        [...document.getElementsByClassName("js-only")].forEach(showHiddenElement)
     </script>
 
     @RenderSection("BodyEnd", false)

diff --git a/src/Dfe.ContentSupport.Web/Views/Shared/_Feedback.cshtml b/src/Dfe.ContentSupport.Web/Views/Shared/_Feedback.cshtml
@@ -17,7 +17,7 @@
 
 @if (track)
 {
-    <div class="dfe-feedback-banner govuk-!-margin-top-9 govuk-visually-hidden" aria-hidden="true" id="feedback-banner">
+    <div class="dfe-feedback-banner govuk-!-margin-top-9 js-only govuk-visually-hidden" aria-hidden="true" id="feedback-banner">
         <div class="dfe-feedback-banner--content">
             <form id="feedbackForm">
                 <div class="dfe-feedback-banner--content-questions">

diff --git a/src/Dfe.ContentSupport.Web/Views/Shared/_Print.cshtml b/src/Dfe.ContentSupport.Web/Views/Shared/_Print.cshtml
@@ -1,3 +1,3 @@
-<div class="print-button govuk-visually-hidden" aria-hidden="true">
+<div class="js-only print-button govuk-visually-hidden" aria-hidden="true">
     <button class="govuk-link print-link-button" data-module="print-link" id="print-link" >Print this page</button>
 </div>