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.87k stars 376 forks source link

feat: Make FluentSearch allow Generic Type arguments #1585

Closed Hona closed 8 months ago

Hona commented 8 months ago

At the moment the fluent search box only allows string.

It would be nice if we could provide a generic value & customise display/search field.

Unsure if this goes against the library's philosophy.

vnbaaij commented 8 months ago

Not sure how a user would then input the generic info to search on. Do you have an example to illustrate the issue you are trying to solve with this?

Hona commented 8 months ago

Honestly - it seems to be the FluentAutocomplete covers this behaviour. But I also then feel that FluentSearch is not needed because of this. What is the main difference?

vnbaaij commented 8 months ago

Fluent Search is just a simple wrapper of the same-named web component that does not offer the capabilities of the Autocomplete (which is native to the Blazor library).

Closing this issue then.