-
Notifications
You must be signed in to change notification settings - Fork 520
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
Support ReactNode in ActionMenu.Overlay #2188
Conversation
* Supporting ReactNode type for ActionMenu children * Fixing spacing * Updating ActionMenu.Overlay props docs * Adding changeset
🦋 Changeset detectedLatest commit: 23ee0fe The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
size-limit report 📦
|
@jdrush89 A canary version is published for each commit you can install this one by running |
@siddharthkp I tested with the canary package in a memex codespace and all looks good, I'm able to specify false and null as Overlay children and everything's working as expected. |
That's perfect! I'll merge this tomorrow and add it to the next release :) |
Created from #2168 by @jdrush89
Describe your changes here.
Add support for ReactNode as the child type in ActionMenu.Overlay so that conditional children can be rendered within an overlay.
Closes #2067
Screenshots
No visual changes
Merge checklist
Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.