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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added UpdateOnInput to MatInputText to allow @bind:event="oninput". #574

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

darickc
Copy link

@darickc darickc commented May 26, 2020

Added UpdateOnInput as a boolean to MatInputText to allow @Bind:event="oninput" as described in this article: https://docs.microsoft.com/en-us/aspnet/core/blazor/data-binding?view=aspnetcore-3.1

@SamProf
Copy link
Owner

SamProf commented Jun 13, 2020

Thank you for cointribution, Darick!
As I check this problem some times ago, @bind:event="something" can not be as parameter. It is compile time parameter. Did you properly test this and sire that is works?

@darickc
Copy link
Author

darickc commented Jun 13, 2020

Yes, I added an example of using it to the TextField demo. You will see at the end of the page. When typing in the first field, "Update on Change", the value doesn't change until the text box loses focus. But the "Update On Input" changes as you type. This is working for both the client and server demo app.

@Christian-Oleson
Copy link
Contributor

@darickc ,
There are merge conflicts here

@Rickybarb5
Copy link

Any news on this? Reactive components are a must in today's web browsing experience 😄

@darickc
Copy link
Author

darickc commented Apr 1, 2021

@SamProf, I believe the merge conflicts have been resolved. Is there more that I need to do?

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.

4 participants