Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NumericInput]: change value onScroll #835

Closed
JuliaMV opened this issue Nov 24, 2022 · 2 comments
Closed

[NumericInput]: change value onScroll #835

JuliaMV opened this issue Nov 24, 2022 · 2 comments
Assignees
Labels
improvement Something that could be better

Comments

@JuliaMV
Copy link
Contributor

JuliaMV commented Nov 24, 2022

If user focuses on NumeriсInput and starts scrolling (using mouse or touchpad) the input value will be changed with scroll page simultaneously. https://uui.epam.com/documents?id=numericInput&mode=doc&skin=UUI4_promo&category=components

Looks like this behavior related to React events facebook/react#14856.
But in vanilla input type=number state isn't changed on scroll. https://codepen.io/JuliaMV/pen/jOKzgQz

I think would be better to do one thing or scroll or change input state.

@dzevakov
Copy link
Collaborator

The proposed implementation describes the following behavior:

1 When the mouse cursor is over the input and the input is in focus - when scrolling, the value of the input does not change, the page does not scroll

2 When the mouse cursor is over the input and the input is not in focus - when scrolling, the value of the input does not change, the page scrolls

3 When the mouse cursor is outside the input and the input is in focus - when scrolling, the value of the input does not change, the page scrolls

@NatalliaAlieva NatalliaAlieva added the improvement Something that could be better label Dec 3, 2022
@NatalliaAlieva
Copy link
Collaborator

Released in 4.9.1 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Something that could be better
Projects
Archived in project
Development

No branches or pull requests

3 participants