microsoft / fluentui-blazor

Microsoft Fluent UI Blazor components library. For use with ASP.NET Core Blazor applications

https://www.fluentui-blazor.net

MIT License

3.84k stars 372 forks source link

fix: Immediate doesn't work in FluentComboxBox #2672

Closed JamesNK closed 1 month ago

JamesNK commented 1 month ago

🐛 Bug Report

FluentComboxBox has Immediate and ImmediateDelay options to react to the value changing immediately. They don't work right now.

I want to use ValueChanged with a combo box to update the items as the user types. Note: I'm aware of the Autocomplete options on FluentComboxBox and they don't have the behavior I want.

💻 Repro or Code Sample

The problem appears to be that FluentComboxBox doesn't bind oninput:

https://github.com/microsoft/fluentui-blazor/blob/503d2eee1b631d38e83f24d45dfdda80df98a32a/src/Core/Components/List/FluentCombobox.razor

🤔 Expected Behavior

ValueChanged and raised while typing in a combo box when immediate is true.

😯 Current Behavior

ValueChanged is only raised when focus leaves the combo box.

💁 Possible Solution

🔦 Context

🌍 Your Environment

OS & Device: [e.g. MacOS, iOS, Windows, Linux] on [iPhone 7, PC]
Browser [e.g. Microsoft Edge, Google Chrome, Apple Safari, Mozilla FireFox]
.NET and Fluent UI Blazor library Version [e.g. 8.0.2 and 4.4.1]

dvoituron commented 1 month ago

See #2685