From ec4333a6d1e4ffa08f15d8171fa2a856567f5410 Mon Sep 17 00:00:00 2001
From: Mike Alhayek <mike@crestapps.com>
Date: Mon, 6 Nov 2023 11:09:18 -0800
Subject: [PATCH] Fix the Content-preview script (#14644)

---
 .../Views/Preview/Index.cshtml                  | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

diff --git a/src/OrchardCore.Modules/OrchardCore.ContentPreview/Views/Preview/Index.cshtml b/src/OrchardCore.Modules/OrchardCore.ContentPreview/Views/Preview/Index.cshtml
index b3949e587c7..e9852ace02e 100644
--- a/src/OrchardCore.Modules/OrchardCore.ContentPreview/Views/Preview/Index.cshtml
+++ b/src/OrchardCore.Modules/OrchardCore.ContentPreview/Views/Preview/Index.cshtml
@@ -25,16 +25,13 @@
 
 <resources type="Footer" />
 
-<script at="Foot" depends-on="jQuery">
-    var renderPreviewUrl = $(document.getElementById('renderPreviewUrl')).data('value');
-    var iframe = document.getElementById('iframe');
-    var previewEventTimer = null;
-    var previewRenderTimer = null;
-    var previewRendering;
-    var previewRenderData;
-    var initialized;
+<script>
+    let iframe = document.getElementById('iframe');
+    let previewRenderTimer = null, previewRendering, previewRenderData, initialized, renderPreviewUrl;
 
     $(function () {
+        var previewEventTimer = null;
+        renderPreviewUrl = document.getElementById('renderPreviewUrl').getAttribute('data-value');
         $(window).on('storage', function (ev) {
             if (ev.key.indexOf('contentpreview:not-connected:') !== -1) {
                 $(notConnectedWarning).show();
@@ -50,7 +47,7 @@
         });
 
         // override default behaviour of Bootstrap's. We only hide, not remove the alert.
-        $('#close-connect-warning').on('click', function(){
+        $('#close-connect-warning').on('click', function () {
             $('#notConnectedWarning').hide();
         });
 
@@ -72,7 +69,7 @@
 
     iframe.onload = function () {
         // The iframe may not be well setup after the first loading and then on the first editor update
-        // we may lose the scrolling position. A workaround is to refrech the page once after or to add
+        // we may lose the scrolling position. A workaround is to re-fetch the page once after or to add
         // here a delay that really block the thread before exiting. Here, we just rewrite the document.
         if (!initialized) {
             if (this.contentWindow) {