Support ReactNode in ActionMenu.Overlay

[siddharthkp]

Created from https://github.com/primer/react/pull/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

• [ ] Added/updated tests - current testing doesn't seem granular enough to warrant testing every possible prop type, I can add a test if it seems necessary though.
• [x] Added/updated documentation
• [x] Tested in Chrome
• [x] Tested in Firefox
• [x] Tested in Safari
• [x] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

[changeset-bot[bot]]

🦋 Changeset detected

Latest commit: 23ee0fe563f58766795e166d01f4ff374926a081

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@primer/react	Patch

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

[github-actions[bot]]

size-limit report 📦

Path	Size
dist/browser.esm.js	75.63 KB (0%)
dist/browser.umd.js	76.24 KB (0%)

[siddharthkp]

@jdrush89 A canary version is published for each commit

you can install this one by running npm install @primer/[email protected] in the repo you want to install like memex

[jdrush89]

@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.

[siddharthkp]

That's perfect! I'll merge this tomorrow and add it to the next release :)