Skip to content

Conversation

@mo3ayka
Copy link
Contributor

@mo3ayka mo3ayka commented Dec 13, 2023

In some of our cases, we need to be able to receive a signal about a search change, so I would like to suggest adding an event

@mckaragoz
Copy link
Contributor

Actually, you have SearchFunc parameter that you can do whatever you want when search string changed. Do you have something that you can't do in SearchFunc?

@mo3ayka
Copy link
Contributor Author

mo3ayka commented Dec 14, 2023

Yes, I tried to use it, but with its help there is no way to know when the search will be finished. We need to know when the search started, the user entered some character and when the user clear the search value

if (!SearchBox || ItemCollection == null || _searchString == null)

If the search string is empty, then SearchFunc is not called

@mckaragoz
Copy link
Contributor

Understand.

The only thing is SomethingChanged is two way bind pattern in Blazor.

We should use OnSearchStringChange instead of SearchStringChanged

@mo3ayka
Copy link
Contributor Author

mo3ayka commented Dec 14, 2023

We should use OnSearchStringChange instead of SearchStringChanged

I fixed it 🙂

@mckaragoz
Copy link
Contributor

We should use OnSearchStringChange instead of SearchStringChanged

I fixed it 🙂

Sorry again, not "Changed", should be "Change" :) Blazor understand "Changed" keyword with different meaning.

@mo3ayka
Copy link
Contributor Author

mo3ayka commented Dec 15, 2023

Blazor understand "Changed" keyword with different meaning.

Wow, I'm learning more and more about the nuances of the blazor, cool 🤯
Fixed it )

@mckaragoz mckaragoz merged commit 19777b1 into CodeBeamOrg:dev Dec 15, 2023
@mckaragoz
Copy link
Contributor

Thanks 🥇

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 this pull request may close these issues.

2 participants