Make inputs commit on blur, and let browser handle undo/redo about dmn-js HOT 4 CLOSED

We could consider to change the FEEL editor change handler once we settled on unifying the behavior.

from dmn-js.

Additional context:

There is a difference between how change events are handled in text inputs and contenteditable elements. For a text input, you can listen to a change event which is dispatched only on blur and if input value was changed. In contrast, contenteditable does not fire the change event, and a common solution is to listen to the input . However, it behaves differently as input event is dispatched after each keystroke. In a naive implementation of contenteditable, we may flood the editor command stack with changes which leads to a problems like in #859
https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/change_event

https://camunda.slack.com/archives/GP70M0J6M/p1715331428555569

from dmn-js.

The FEEL editor relies on CodeMirror's updateListener to call the change handler. However, such updates behave much more closely to the browser input events, as the handler is called each time the editor view updates. This is way too often, so in the LiteralExpression component we should emulate browser's change event behavior, i.e. call the handler only on blur when the value changed.

from dmn-js.

Agreed, but that means a breaking change if we modify the onChange callback behavior.

from dmn-js.