SAP / ui5-webcomponents

UI5 Web Components - the enterprise-flavored sugar on top of native APIs! Build SAP Fiori user interfaces with the technology of your choice.
https://sap.github.io/ui5-webcomponents/
Apache License 2.0
1.58k stars 270 forks source link

[@ui5/webcomponents/Inputt]: error thrown when using number type input #9921

Closed malonmiming closed 1 month ago

malonmiming commented 2 months ago

Bug Description

[Issue 1] When using focus input in number-type Input, an error is thrown:

TypeError: Cannot read properties of undefined (reading 'inputType') at Input._handleInput (Input.js:581:25) at z.handleEvent (lit-html.js:6:7169)

[Issue 2] The English letter "e" can be entered in the input type number.

Affected Component

No response

Expected Behaviour

No response

Isolated Example

No response

Steps to Reproduce

  1. focus input
  2. see error in console ... ``

Log Output, Stack Trace or Screenshots

TypeError: Cannot read properties of undefined (reading 'inputType') at Input._handleInput (Input.js:581:25) at z.handleEvent (lit-html.js:6:7169)

Priority

None

UI5 Web Components Version

"@ui5/webcomponents": "^2.1.1", "@ui5/webcomponents-fiori": "^2.1.1",

Browser

Chrome

Operating System

No response

Additional Context

No response

Organization

No response

Declaration

nnaydenow commented 2 months ago

Hi @malonmiming,

I think that https://github.com/SAP/ui5-webcomponents/pull/9616 is fixing similar issue https://github.com/SAP/ui5-webcomponents/issues/9588. Could you please try to update application to version 2.2.0 or to check if the issue is reproducible in: https://sap.github.io/ui5-webcomponents/

Regards, Nayden

malonmiming commented 2 months ago

HI @nnaydenow,

Thank you for your response!!! I update application to version 2.2.0. and then the [Issue 1] was resolved, but the [issue 2] was not resolved.

Best Regards, malonmiming

nnaydenow commented 2 months ago

Hi @malonmiming,

e is special symbol for input of type number with the idea to support numbers that contain epsilon symbol. Same symbol is supported in native input. There is restriction that the input might contain only 1 e symbol.

You can check following example

github-actions[bot] commented 1 month ago

Hello, everyone! The issue has been inactive for 21 days. If there are still questions or comments, please feel free to continue the discussion. Inactive issues will be closed after 7 days!

github-actions[bot] commented 1 month ago

Hello, everyone! The issue has been inactive for 28 days, so I am closing the issue.