Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

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.
This commit is contained in:
David Taylor 2022-07-11 09:07:10 +01:00 committed by GitHub
parent a94dee31c4
commit e62730a4c8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 14 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
app/assets/javascripts/discourse/scripts/browser-update.js
View File

@ -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;
}
}