@itwin/core-frontend 5.11.0-dev.8 → 5.12.0-dev.1

package/lib/esm/tools/ToolAdmin.js

@@ -1189,15 +1189,30 @@ export class ToolAdmin {
             this.updateDynamics(undefined, undefined, true); // Don't wait for motion to update dynamics...
         }
     }
-    get _isFocusHome() {
+    static isFocusHome() {
         return (document.body === document.activeElement);
     }
-    _setFocusHome() {
+    static setFocusHome() {
         const element = document.activeElement;
-        if (element && element !== document.body)
+        if (element instanceof HTMLElement && element !== document.body)
             element.blur();
         document.body.focus();
     }
+    static isContextMenuOpen() {
+        return document.querySelector(".core-context-menu-opened") !== null;
+    }
+    static isElement(target) {
+        return target instanceof Element;
+    }
+    static isEditable(element) {
+        const tagName = element.tagName.toLowerCase();
+        const editableTags = ["input", "textarea", "select"];
+        if (editableTags.includes(tagName))
+            return true;
+        if (element instanceof HTMLElement && element.isContentEditable)
+            return true;
+        return false;
+    }
     static getModifierKey(event) {
         switch (event.key) {
             case "Alt": return BeModifierKeys.Alt;
@@ -1239,15 +1254,23 @@ export class ToolAdmin {
      * @return true if handled and no further processing of event should occur.
      */
     processKeyboardEvent(keyEvent, _wentDown) {
-        if (this._isFocusHome || undefined !== IModelApp.accuDraw.getFocusItem())
+        if (ToolAdmin.isFocusHome() || undefined !== IModelApp.accuDraw.getFocusItem())
             return false; // Focus is Home or AccuDraw, allow shortcuts...
+        if (keyEvent.defaultPrevented || keyEvent.isComposing)
+            return true; // Respect UI handling / IME composition; don't process shortcuts
         // NOTE: Provide a convenient way for the user to move focus to Home to use shortcuts.
         // Escape is the only practical choice with its default behavior that can cancel/close/blur.
         // Intentionally not checking wentDown as some ui elements stop propagation of down but not up (moving to capture is not a good option).
         // Apps that want to use Escape to start the default tool still can with Home focus in processShortcutKey.
-        if (keyEvent.key === "Escape" && ToolSettings.escapeMovesFocusToHome && !keyEvent.defaultPrevented && !keyEvent.isComposing)
-            this._setFocusHome();
-        return true;
+        if (keyEvent.key === "Escape" && ToolSettings.escapeMovesFocusToHome) {
+            ToolAdmin.setFocusHome();
+            return true;
+        }
+        if (ToolAdmin.isContextMenuOpen())
+            return true; // Don't allow shortcuts if context menu is open...
+        if (ToolAdmin.isElement(keyEvent.target) && ToolAdmin.isEditable(keyEvent.target))
+            return true; // Don't allow shortcuts when event target is editable...
+        return false;
     }
     /** Need to check for modifier changes on capture phase as a ui element that calls stopPropagation
      * prevents onKeyTransition from being called when bubbling.