Undo/redo works unexpectedly in BKM

[barmac]

Describe the bug

When I am modeling a BKM, and trigger undo while typing the BKM name, it triggers undo outside of the name box.

Screen.Recording.2024-10-16.at.14.52.47.mov

Steps to reproduce

1. Write in body or change formal parameters
2. Write in the name box and hit undo
3. now this happens

Expected behavior

Undo/redo stack should be isolated to the currently selected input. Changes outside should be undone only after input loses focus.

Environment

• OS: [e.g. MacOS 10.2, Windows 10]
• Camunda Modeler Version: all since BKM was introduced
• Execution Platform: [e.g. Camunda 7, Camunda 8]
• Installed plug-ins: [...]

Additional context

No response

[barmac]

One could argue that we should commit all of the changes in the input to the command stack, but I'd rather leverage the browser input stack handling, so that we can undo the entire change if desired.

[nikku]

Let's discuss this, and consider what will be our future scheme. Relates to #3902, but also relevant to properties panel UX.

[nikku]

CC @lmbateman

[jarekdanielak]

The root cause of this issue is lack of consistency in undo/redo behavior across Desktop Modeler. When I started testing different combinations of typing and undoing/redoing between text inputs, FEEL editors and canvas, it's clear there is no one strategy that we follow.

It requires more discussion (cc. @lmbateman). I will put it in the backlog as I don't have the capacity to work on it in the near future.