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.

Sign up for GitHub

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

Jump to bottom

SearchBox: Clearing the field should fire onChanged event #276

Closed
mikewheaton opened this issue Sep 13, 2016 · 1 comment · Fixed by #300
Closed

SearchBox: Clearing the field should fire onChanged event #276

mikewheaton opened this issue Sep 13, 2016 · 1 comment · Fixed by #300
Labels
Type: Bug 🐛

Comments

@mikewheaton
Copy link
Contributor

When clearing the SearchBox using the X button, the onChanged event should be called so that the app can respond to the change in text.
@dzearing
Copy link
Member

onChange should not be past tense as well..

onChange(...)

@mikewheaton mikewheaton mentioned this issue Sep 19, 2016
Merged
@microsoft microsoft locked as resolved and limited conversation to collaborators Sep 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Type: Bug 🐛
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Call onChange() on SearchBox when field cleared
2 participants
@dzearing @mikewheaton