fix: undo redo on preview text inputs leaking to editor

Author: abose

src/LiveDevelopment/BrowserScripts/LiveDevProtocolRemote.js

@@ -445,29 +445,13 @@
     }
     window.document.addEventListener("click", onDocumentClick);
     window.document.addEventListener("keydown", function (e) {
-        // for undo. refer to LivePreviewEdit.js file 'handleLivePreviewEditOperation' function
-        if ((e.ctrlKey || e.metaKey) && e.key.toLowerCase() === "z" && !e.shiftKey) {
-            MessageBroker.send({
-                livePreviewEditEnabled: true,
-                undoLivePreviewOperation: true
-            });
-        }
-
-        // for redo - supports both Ctrl+Y and Ctrl+Shift+Z (Cmd+Y and Cmd+Shift+Z on Mac)
-        if ((e.ctrlKey || e.metaKey) && (e.key.toLowerCase() === "y" || (e.key.toLowerCase() === "z" && e.shiftKey))) {
-            MessageBroker.send({
-                livePreviewEditEnabled: true,
-                redoLivePreviewOperation: true
-            });
-        }
-
         // Check if user is editing text content - if so, allow normal text cut
         // Get the truly active element, even if inside shadow roots
         let activeElement = document.activeElement;
 
         const isEditingText = activeElement && (
             // Check for standard form input elements
-            ['INPUT', 'TEXTAREA', 'SELECT', 'OPTION'].includes(activeElement.tagName) ||
+            ['INPUT', 'TEXTAREA'].includes(activeElement.tagName) ||
             // Check for contentEditable elements
             activeElement.isContentEditable ||
             // Check for ARIA roles that indicate text input
@@ -476,6 +460,24 @@
             (activeElement.hasAttribute("contenteditable") && activeElement.hasAttribute("data-brackets-id"))
         );
         const isInEditMode = window._LD && window._LD.getMode && window._LD.getMode() === 'edit';
+
+        // for undo. refer to LivePreviewEdit.js file 'handleLivePreviewEditOperation' function
+        if (!isEditingText && isInEditMode && (e.ctrlKey || e.metaKey) && e.key.toLowerCase() === "z" && !e.shiftKey) {
+            MessageBroker.send({
+                livePreviewEditEnabled: true,
+                undoLivePreviewOperation: true
+            });
+        }
+
+        // for redo - supports both Ctrl+Y and Ctrl+Shift+Z (Cmd+Y and Cmd+Shift+Z on Mac)
+        if (!isEditingText && isInEditMode && (e.ctrlKey || e.metaKey) &&
+            (e.key.toLowerCase() === "y" || (e.key.toLowerCase() === "z" && e.shiftKey))) {
+            MessageBroker.send({
+                livePreviewEditEnabled: true,
+                redoLivePreviewOperation: true
+            });
+        }
+
         // Cut: Ctrl+X / Cmd+X - operates on selected element
         if ((e.ctrlKey || e.metaKey) && e.key.toLowerCase() === "x") {