-
Notifications
You must be signed in to change notification settings - Fork 552
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
ActionMenu: Make sure event handlers on trigger are called #4648
Conversation
siddharthkp
commented
Jun 6, 2024
•
edited
Loading
edited
- Fixes [ActionMenu.Button] onClick doesn't function #4647
- Merge event handlers from props and AnchoredOverlay
🦋 Changeset detectedLatest commit: 3e96d91 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 📦
|
…ct into fix/actionmenu-button-onclick
Hi! This pull request has been marked as stale because it has been open with no activity for 60 days. You can comment on the pull request or remove the stale label to keep it open. If you do nothing, this pull request will be closed in 7 days. |
|
👋 Hi from github/github! Your integration PR is ready: https://github.com/github/github/pull/341579 |
No breaking changes confirmed! |
* wip: add mergeAnchorHandlers * merged onClick and onKeyDown in ActionMenu.Anchor * improve types * Create green-schools-smell.md * cover cases with Tooltip