SAP / openui5

OpenUI5 lets you build enterprise-ready web applications, responsive to all devices, running on almost any browser of your choice.
http://openui5.org
Apache License 2.0
2.94k stars 1.23k forks source link

Setting ValueState to error clears the field #4090

Closed ekansh005 closed 1 month ago

ekansh005 commented 1 month ago

OpenUI5 version: 1.108.33, works as expected till 1.108.32

Browser/version (+device/version): Chrome/Safari

Any other tested browsers/devices(OK/FAIL): Fails on all browsers

URL (minimal example if possible): demo - https://ekansh005.github.io/Ui5LiveChangeValueStateIssue/ repo - https://github.com/ekansh005/Ui5LiveChangeValueStateIssue

User/password (if required and possible - do not post any confidential information here):

Steps to reproduce the problem:

  1. start typing any value in the input field
  2. you will notice that as soon as you type the first character, it got cleared
  3. type 9th character and it will clear again

What is the expected result? It should just set the value state to Error or None but should not cleat the user input

What happens instead? When value state is set, it is clearing the user input as well which is wrong.

Any other information? (attach screenshot if possible) Since 1.108 is LTS, the fix should be made available in this version.

tsanislavgatev commented 1 month ago

Hello @ekansh005 , Thank you for sharing this finding. I've created an internal incident DINC0041816. The status of the issue will be updated here in GitHub. Regards, Tsanislav

MapTo0 commented 1 month ago

Hey @ekansh005

We have already fixed that error with https://github.com/SAP/openui5/commit/29d58b5e151d4b7a3a0c3209fa8c24a86748b3fc

I see that you've loaded 1.108.30 https://github.com/ekansh005/Ui5LiveChangeValueStateIssue/blob/main/index.html#L9

The fix will be available in 1.108.32