-
Notifications
You must be signed in to change notification settings - Fork 376
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
[ComboBox] Fix OnValueChanged being called multiple times #2855
Conversation
✅ All tests passed successfully Details on your Workflow / Core Tests page. |
- Add cb-specific js custom event handler
Summary - Unit Tests Code CoverageSummary
CoverageMicrosoft.FluentUI.AspNetCore.Components - 61.1%
|
Image below shows output from the test code. At first green arrow, text is being deleted from the text input with |
Fix #2851 by not calling
base.ChangeHandlerAsync
when no item selected. This could happen if the Value and the Text of an option differed.See original issue for before output. Now all test situations render as expected.