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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: FluentNumberField's two-way data binding on Value Property does not update appropriately #1531

Closed
arazni opened this issue Feb 17, 2024 · 0 comments 路 Fixed by #1576
Closed

Comments

@arazni
Copy link

arazni commented Feb 17, 2024

This is extremely similar to #1450

馃悰 Bug Report

In my web app, I have a component that takes a numeric input and divies a portion to a wallet and a portion to a stash. After a number is selected, a button triggers a command and this command also resets the value of the numeric field to 0.

Unfortunately, the numeric field does not respect the behavior of being reset to 0 by the button command.

馃捇 Repro or Code Sample

Download and run the code on this tag in dotnet8: https://github.com/arazni/blazor-fluent-ui-slider-bug/releases/tag/number-field-test

馃 Expected Behavior

When I click a button that resets the two-way bound value of the NumberField to 0, the NumberField's Value property should change to 0 and the field should visibly appear to reset to 0.

馃槸 Current Behavior

When I click a button that resets the two-way bound value of the NumberField to 0, the NumberField's Value property stays at its current value and the field's visually apparent value stays at its current value.

馃拋 Possible Solution

Perhaps something very similar to how the Slider was fixed for #1450

馃敠 Context

In my web app, I have a component that takes a numeric input and divies a portion to a wallet and a portion to a stash. After a number is selected, a button triggers a command and this command also resets the value of the numeric field to 0.

馃實 Your Environment

OS & Device: Windows 11 PC
Browser at least: Microsoft Edge and Mozilla FireFox
.NET 8.0.2 and FluentUI Version 4.4.1

vnbaaij added a commit that referenced this issue Feb 27, 2024
* Fix #1531 by using web components current-value instead of value.

* Fix #1540 by only invoking RowFocus/CellFocus for default rows/cells
vnbaaij added a commit that referenced this issue Mar 6, 2024
* Fix #1531 by using web components current-value instead of value.

* Fix #1540 by only invoking RowFocus/CellFocus for default rows/cells
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant