iTwin / appui

Monorepo for iTwin.js AppUi
MIT License
9 stars 2 forks source link

Unable to type letters in `AccuInputField` #1055

Closed mathieu-fournier closed 1 month ago

mathieu-fournier commented 2 months ago

Describe the bug

When typing in the AccuInputField, letters are not entered, so we cannot input units 'ft', 'in', 'm', etc.. image

In pink, we can see that some accudraw shortcuts have been mapped to keys that might be pressed for units. We should make sure that when typing in the input fields, the event does not conflict with shortcuts and vice versa.

I think that this line might be the reason why letters are not registered. https://github.com/iTwin/appui/blob/252752fb8244125cec0d0e87dd235175ed966ea7/ui/appui-react/src/appui-react/accudraw/AccuDrawInputField.tsx#L154

To Reproduce

No response

Expected Behavior

No response

Screenshots

No response

Desktop (please complete the applicable information)

No response

Additional context

No response

mathieu-fournier commented 1 month ago

Related to : https://github.com/iTwin/iTwinUI/issues/2284