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

Number with big precision cannot be changed with direct input representation #268

Closed
vaadin-miki opened this issue Jan 20, 2021 · 1 comment · Fixed by #272
Closed

Number with big precision cannot be changed with direct input representation #268

vaadin-miki opened this issue Jan 20, 2021 · 1 comment · Fixed by #272
Labels
bug Something isn't working

Comments

@vaadin-miki
Copy link
Owner

I have no idea how to title this ticket properly.

Anyway, assume SuperBigDecimalField with maximum fraction length 3 and scientific notation support. Entering 1e-5 will assign the value to be, well, 1e-5 represented by 0,0. Entering exactly that string into the number field will not trigger value change and hence will not modify the value of the number field to be 0.

This should not happen, entering any string should overwrite the value of the field to whatever the value represents. Note this only happens when scientific notation is supported in SuperBigDecimalField.

@vaadin-miki vaadin-miki added the bug Something isn't working label Jan 20, 2021
@vaadin-miki vaadin-miki added this to the Backlog milestone Jan 20, 2021
vaadin-miki added a commit that referenced this issue Feb 14, 2021
@vaadin-miki
Copy link
Owner Author

fixed, and as a side effect numbers entered with scientific notation are no longer reformatted (so 2e-5 stays formatted as 2e-5 even when the cursor leaves)

@vaadin-miki vaadin-miki modified the milestones: Backlog, Next release Feb 14, 2021
vaadin-miki added a commit that referenced this issue Feb 14, 2021
also, numbers entered with scientific notation now do not lose formatting on field blur
vaadin-miki added a commit that referenced this issue Mar 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant