Skip to content
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

Merged
merged 8 commits into from
Sep 13, 2024

Conversation

siddharthkp
Copy link
Member

@siddharthkp siddharthkp commented Jun 6, 2024

Copy link

changeset-bot bot commented Jun 6, 2024

🦋 Changeset detected

Latest commit: 3e96d91

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

Copy link
Contributor

github-actions bot commented Jun 6, 2024

size-limit report 📦

Path Size
packages/react/dist/browser.esm.js 97.14 KB (+0.07% 🔺)
packages/react/dist/browser.umd.js 97.59 KB (+0.14% 🔺)

@github-actions github-actions bot temporarily deployed to storybook-preview-4648 June 6, 2024 16:40 Inactive
@siddharthkp siddharthkp self-assigned this Jun 18, 2024
@siddharthkp siddharthkp added patch release bug fixes, docs, housekeeping integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm labels Jun 18, 2024
@github-actions github-actions bot temporarily deployed to storybook-preview-4648 June 18, 2024 11:04 Inactive
@siddharthkp siddharthkp changed the title wip: ActionMenu: Make sure event handlers on trigger are called ActionMenu: Make sure event handlers on trigger are called Jun 18, 2024
Copy link
Contributor

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.

@github-actions github-actions bot added the Stale label Aug 19, 2024
@github-actions github-actions bot closed this Aug 26, 2024
@github-actions github-actions bot deleted the fix/actionmenu-button-onclick branch August 26, 2024 10:05
@siddharthkp siddharthkp restored the fix/actionmenu-button-onclick branch September 6, 2024 12:49
@siddharthkp
Copy link
Member Author

@siddharthkp siddharthkp reopened this Sep 6, 2024
@github-actions github-actions bot added the staff Author is a staff member label Sep 6, 2024
@github-actions github-actions bot temporarily deployed to storybook-preview-4648 September 6, 2024 12:53 Inactive
@github-actions github-actions bot removed the Stale label Sep 6, 2024
@primer-integration
Copy link

👋 Hi from github/github! Your integration PR is ready: https://github.com/github/github/pull/341579

@siddharthkp siddharthkp added integration-tests: passing Changes in this PR do NOT cause breaking changes in gh/gh and removed integration-tests: recommended This change needs to be tested for breaking changes. See https://arc.net/l/quote/tdmpakpm labels Sep 11, 2024
@siddharthkp
Copy link
Member Author

No breaking changes confirmed!

@siddharthkp siddharthkp marked this pull request as ready for review September 12, 2024 09:47
@siddharthkp siddharthkp requested a review from a team as a code owner September 12, 2024 09:47
@siddharthkp siddharthkp added this pull request to the merge queue Sep 13, 2024
Merged via the queue into main with commit c6931d2 Sep 13, 2024
35 checks passed
@siddharthkp siddharthkp deleted the fix/actionmenu-button-onclick branch September 13, 2024 10:02
@primer primer bot mentioned this pull request Sep 13, 2024
TylerJDev pushed a commit that referenced this pull request Sep 23, 2024
* wip: add mergeAnchorHandlers

* merged onClick and onKeyDown in ActionMenu.Anchor

* improve types

* Create green-schools-smell.md

* cover cases with Tooltip
@primer primer bot mentioned this pull request Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
integration-tests: passing Changes in this PR do NOT cause breaking changes in gh/gh patch release bug fixes, docs, housekeeping staff Author is a staff member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ActionMenu.Button] onClick doesn't function
2 participants