fix: do not propagate events handled by keyboard bindings

Closes #3016
camunda · Aug 18, 2022 · 3ce535d · 3ce535d
1 parent 6c75db0
commit 3ce535d
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/...pmn/modeler/features/properties-panel-keyboard-bindings/PropertiesPanelKeyoardBindings.js b/...pmn/modeler/features/properties-panel-keyboard-bindings/PropertiesPanelKeyoardBindings.js
@@ -56,19 +56,27 @@ export default class PropertiesPanelKeyboardBindings {
     if (isUndo(event)) {
       commandStack.canUndo() && commandStack.undo();
 
-      event.preventDefault();
+      this._cancel(event);
     }
 
     if (isRedo(event)) {
       commandStack.canRedo() && commandStack.redo();
 
-      event.preventDefault();
+      this._cancel(event);
     }
   }
 
   _getContainer() {
     return this._propertiesPanel._container;
   }
+
+  /**
+   * @param {KeyboardEvent} event
+   */
+  _cancel(event) {
+    event.preventDefault();
+    event.stopPropagation();
+  }
 }
 
 PropertiesPanelKeyboardBindings.$inject = [ 'commandStack', 'eventBus', 'propertiesPanel' ];