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

onChange supplies value to msg function #181

Merged
merged 2 commits into from
Mar 22, 2023

Conversation

davesmith00000
Copy link
Member

FAO: @snichele

Quick fix to make onChange pass its value to the function that creates the Msg. In the end, it was very similar to the solution described in issue #180, and the pattern was already established by onInput, which does the exact same thing.

Borrowed the MDN onChange example to check it works as expected:

image

@davesmith00000 davesmith00000 requested a review from a team March 21, 2023 22:09
@davesmith00000 davesmith00000 self-assigned this Mar 21, 2023
@davesmith00000 davesmith00000 changed the title Issue/180/onchange supplies value onChange supplies value to msg function Mar 21, 2023
@davesmith00000 davesmith00000 merged commit 27c8c2e into main Mar 22, 2023
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.

None yet

1 participant