-
Notifications
You must be signed in to change notification settings - Fork 350
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
fix(Select): clear button appears with any typeahead input #3502
Conversation
104c1bf
to
130c0e8
Compare
PatternFly-React preview: https://patternfly-react-pr-3502.surge.sh |
Codewise it looks really good and simple! Can we also call |
@karelhala I could do that. Might it be a breaking change if consumers have written onSelect functions without designing them to handle empty inputs? We do provide an onClear property that is only called when the user clicks the clear button. Can that not serve the same purpose? |
@nicolethoen It does look like there's an issue with the clear button not resetting the filtering when cleared in this way. When the menu is reopened it retains the previous filtered state until new input is entered. |
130c0e8
to
719c6c6
Compare
719c6c6
to
549da12
Compare
Codecov Report
@@ Coverage Diff @@
## master #3502 +/- ##
=======================================
Coverage 67.09% 67.09%
=======================================
Files 903 903
Lines 25482 25482
Branches 2257 2257
=======================================
Hits 17097 17097
Misses 7343 7343
Partials 1042 1042
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Addresses #3460