diff options
author | Pierre Ossman <ossman@cendio.se> | 2019-08-16 11:18:45 +0200 |
---|---|---|
committer | Samuel Mannehed <samuel@cendio.se> | 2019-09-24 15:34:59 +0200 |
commit | 0b51419ca4eecfbdaef3d171a8f6278e14f64858 (patch) | |
tree | 3e89ebf6c72658676d95ff0176d2a9813b81b614 | |
parent | e1d50c8c10563edcacebcaeb32d022906aa003d2 (diff) | |
download | novnc-0b51419ca4eecfbdaef3d171a8f6278e14f64858.tar.gz |
Use "nomodule" instead of manual check
Very few browsers are left in the wild that supports modules but not
"nomodule", so let's simplify our handling a bit.
Safari 10 supports modules but not 'nomodule', this means that this
particular version of Safari will be broken. Due to this we have to
bump up the required Safari version to 11.
-rw-r--r-- | README.md | 2 | ||||
-rwxr-xr-x | utils/use_require.js | 24 | ||||
-rw-r--r-- | vnc.html | 12 | ||||
-rw-r--r-- | vnc_lite.html | 13 |
4 files changed, 7 insertions, 44 deletions
@@ -89,7 +89,7 @@ noVNC uses many modern web technologies so a formal requirement list is not available. However these are the minimum versions we are currently aware of: -* Chrome 49, Firefox 44, Safari 10, Opera 36, IE 11, Edge 12 +* Chrome 49, Firefox 44, Safari 11, Opera 36, IE 11, Edge 12 ### Server Requirements diff --git a/utils/use_require.js b/utils/use_require.js index 2487927..d6e8d6a 100755 --- a/utils/use_require.js +++ b/utils/use_require.js @@ -105,27 +105,11 @@ function transform_html(legacy_scripts, only_legacy) { new_script += ` <script src="${legacy_scripts[i]}"></script>\n`; } } else { - // Otherwise detect if it's a modern browser and select - // variant accordingly - new_script += `\ - <script type="module">\n\ - window._noVNC_has_module_support = true;\n\ - </script>\n\ - <script>\n\ - window.addEventListener("load", function() {\n\ - if (window._noVNC_has_module_support) return;\n\ - let legacy_scripts = ${JSON.stringify(legacy_scripts)};\n\ - for (let i = 0;i < legacy_scripts.length;i++) {\n\ - let script = document.createElement("script");\n\ - script.src = legacy_scripts[i];\n\ - script.async = false;\n\ - document.head.appendChild(script);\n\ - }\n\ - });\n\ - </script>\n`; - - // Original, ES6 modules + // Otherwise include both modules and legacy fallbacks new_script += ' <script type="module" crossorigin="anonymous" src="app/ui.js"></script>\n'; + for (let i = 0;i < legacy_scripts.length;i++) { + new_script += ' <script nomodule src="${legacy_scripts[i]}"></script>\n'; + } } contents = contents.slice(0, start_ind) + `${new_script}\n` + contents.slice(end_ind); @@ -57,17 +57,7 @@ <!-- promise polyfills promises for IE11 --> <script src="vendor/promise.js"></script> <!-- ES2015/ES6 modules polyfill --> - <script type="module"> - window._noVNC_has_module_support = true; - </script> - <script> - window.addEventListener("load", function() { - if (window._noVNC_has_module_support) return; - var loader = document.createElement("script"); - loader.src = "vendor/browser-es-module-loader/dist/browser-es-module-loader.js"; - document.head.appendChild(loader); - }); - </script> + <script nomodule src="vendor/browser-es-module-loader/dist/browser-es-module-loader.js"></script> <!-- actual script modules --> <script type="module" crossorigin="anonymous" src="app/ui.js"></script> <!-- end scripts --> diff --git a/vnc_lite.html b/vnc_lite.html index 426d79b..adc6abd 100644 --- a/vnc_lite.html +++ b/vnc_lite.html @@ -61,18 +61,7 @@ <script src="vendor/promise.js"></script> <!-- ES2015/ES6 modules polyfill --> - <script type="module"> - window._noVNC_has_module_support = true; - </script> - <script> - window.addEventListener("load", function() { - if (window._noVNC_has_module_support) return; - const loader = document.createElement("script"); - loader.src = "vendor/browser-es-module-loader/dist/" + - "browser-es-module-loader.js"; - document.head.appendChild(loader); - }); - </script> + <script nomodule src="vendor/browser-es-module-loader/dist/browser-es-module-loader.js"></script> <!-- actual script modules --> <script type="module" crossorigin="anonymous"> |