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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dropdown: Stop Dropdown accepting a function as children #65467
Conversation
You have successfully added a new CodeQL configuration |
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.
I think we should have a clear deprecation process and follow it even when we don't think it matters. Ignoring the process seems to go against all our goals for helping users upgrade with confidence, and creating a UI library that developers can rely on.
I don't think the cost of keeping this, deprecated, in the Dropdown component is that high.
@joshhunt i mean i agree in principle, i just don't think anyone is using this at all 馃槄 so put this in the v10 milestone, add a breaking change label+notice and do it that way then? 馃 |
remove dropdown accepting function children
What is this feature?
onVisibleChange
onVisibleChange
prop. think using the new prop is clearer/probably more performant. 馃憦Dropdown
onVisibleChange
insteadWhy do we need this feature?
Who is this feature for?
Which issue(s) does this PR fix?:
Fixes #
Special notes for your reviewer:
Please check that:
Release notice breaking change
We've removed the ability for functions to be passed as children to the
Dropdown
component. Previously, this was used to access theisOpen
state of the dropdown. This can be now be achieved with theonVisibleChange
prop.Before:
After: