primer
Revert "Revert "Add support for nested submenus to `ActionMenu`""
🦋 Changeset detected
Latest commit: 741b14987b8b2654d56747fcbf7a8eb7f1fa684a
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 | Minor |
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
![changeset-bot[bot] avatar](https://avatars.githubusercontent.com/in/28616?v=4 "Published by a pub.dev verified publisher"){kind=small}
size-limit report 📦
| Path | Size |
|---|---|
| packages/react/dist/browser.esm.js | 88.28 KB (+0.27% 🔺) |
| packages/react/dist/browser.umd.js | 88.57 KB (+0.22% 🔺) |
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
Hi @iansan5653! Would be great to get this code back in ActionMenu. Could you list any todos to push this PR to ready and merge it?
I'm happy to take it over incase you cannot prioritise it.
What I understand is that there is no particular issue in this PR itself. But after this is merged in, we need to upgrade dotcom to use this component instead of NestableActionMenu. Approving this PR.
Hi @iansan5653, our canary publishing is working smoothly again, created a integration PR to check what were the errors: https://github.com/github/github/pull/323719
