generalize commandbox-box ESC handler

Author: Erwin Dondorp

saltgui/static/scripts/CommandBox.js

@@ -416,16 +416,11 @@ export class CommandBox {
 
     const targetField = document.getElementById("target");
     TargetType.autoSelectTargetType(targetField.value);
-    targetField.onkeyup = (keyUpEvent) => {
-      if (keyUpEvent.key === "Escape") {
-        CommandBox.hideManualRun();
-      }
-    };
 
-    const commandField = document.getElementById("command");
-    commandField.onkeyup = (keyUpEvent) => {
+    document.onkeyup = (keyUpEvent) => {
       if (keyUpEvent.key === "Escape") {
         CommandBox.hideManualRun();
+        keyUpEvent.stopPropagation();
       }
     };
 
@@ -455,6 +450,8 @@ export class CommandBox {
       targetList.appendChild(option);
     }
 
+    const commandField = document.getElementById("command");
+
     // give another field (which does not have a list) focus first
     // because when a field gets focus 2 times in a row,
     // the dropdown box opens, and we don't want that...