fix(ui5-multiInput): prevent page reload on enter

[Sumon249]

The page should not reload when the 'enter' key is pressed.

Fixes #12167

[cla-assistant]

All committers have signed the CLA.

[nnaydenow]

Hello @Sumon249,

Thank you for the contribution!

---

Hello @SAP/ui5-webcomponents-topic-rl,

Could you please take a look at that PR?

Regards,
Nayden

[ndeshev]

Hello @Sumon249, could you please add Cypress test to cover this fix? Having tests in place will help us ensure the issue doesn’t regress in the future.

There are also some failing ui5-multi-input test cases that should be examined and fixed/adapted

[ndeshev]

Hello @Sumon249,
I've looked at the problem. The page reload in the example happens because the MultiInput is the single input element in a form.
And since the form has no action attribute to redirect to another URL - it gets reloaded.

The stopPropagation() solution is not ideal as it prevents the component consumers to handle its ENTER keydown event, for example for token creation - that was why some of the tests were failing.

On the other hand, the preventDefault() call on ENTER keydown seems to be exactly what we want - this prevents the default form submission behavior while still allowing event bubbling for token creation.

Thank you for submitting the issue and this pull-request. I created another based on this one - #12273 where I modified the fix and added some tests.

Kind regards,

Nikolay