From e62730a4c8acfdb547a3a7d6e43b968fa0622a5d Mon Sep 17 00:00:00 2001
From: David Taylor <david@taylorhq.com>
Date: Mon, 11 Jul 2022 09:07:10 +0100
Subject: [PATCH] FIX: Ensure splash screen `<noscript>` is loaded for legacy
 browsers (#17413)

Previously we would only expand the main `<noscript>` element for older browsers. This commit ensures that we expand all noscript elements on the page, including the one used by the splashscreen to hide itself on the no-js view.
---
 .../discourse/scripts/browser-update.js       | 26 ++++++++++---------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/app/assets/javascripts/discourse/scripts/browser-update.js b/app/assets/javascripts/discourse/scripts/browser-update.js
index 31a5b410f60..f421056a610 100644
--- a/app/assets/javascripts/discourse/scripts/browser-update.js
+++ b/app/assets/javascripts/discourse/scripts/browser-update.js
@@ -24,21 +24,23 @@
     }
 
     document.getElementsByTagName("body")[0].className += " crawler";
-    var mainElement = document.getElementById("main");
     var noscriptElements = document.getElementsByTagName("noscript");
     // find the element with the "data-path" attribute set
-    for (var i = 0; i < noscriptElements.length; ++i) {
-      if (noscriptElements[i].getAttribute("data-path")) {
-        // noscriptElements[i].innerHTML contains encoded HTML, so we need to access
-        // the childNodes instead. Browsers seem to split very long content into multiple
-        // text childNodes.
-        var result = "";
-        for (var j = 0; j < noscriptElements[i].childNodes.length; j++) {
-          result += noscriptElements[i].childNodes[j].nodeValue;
-        }
+    for (var i = noscriptElements.length - 1; i >= 0; i--) {
+      var element = noscriptElements[i];
 
-        mainElement.outerHTML = result;
-        break;
+      // noscriptElements[i].innerHTML contains encoded HTML, so we need to access
+      // the childNodes instead. Browsers seem to split very long content into multiple
+      // text childNodes.
+      var result = "";
+      for (var j = 0; j < element.childNodes.length; j++) {
+        result += element.childNodes[j].nodeValue;
+      }
+
+      if (element.getAttribute("data-path")) {
+        document.getElementById("main").outerHTML = result;
+      } else {
+        element.outerHTML = result;
       }
     }