Skip to content

chore: revert overflowmenu child typing#22197

Merged
tay1orjones merged 5 commits into
carbon-design-system:mainfrom
tay1orjones:task/revert-overflowmenu-child-typing
May 7, 2026
Merged

chore: revert overflowmenu child typing#22197
tay1orjones merged 5 commits into
carbon-design-system:mainfrom
tay1orjones:task/revert-overflowmenu-child-typing

Conversation

@tay1orjones

Copy link
Copy Markdown
Member

Reverts the OverflowMenu child item cloning behavior change made in #22011

This broke a consumer so I've also added a test that covers this use case to ensure we don't break it again.

Changelog

New

  • Added a test covering this behavior

Changed

  • Revert overflowmenu child cloning behavior/check

Testing / Reviewing

  • Review that the revert matches the previous PR

PR Checklist

As the author of this PR, before marking ready for review, confirm you:

  • Reviewed every line of the diff
    - [ ] Updated documentation and storybook examples
  • Wrote passing tests that cover this change
    - [ ] Addressed any impact on accessibility (a11y)
    - [ ] Tested for cross-browser consistency
  • [s] Validated that this code is ready for review and status checks should pass

More details can be found in the pull request guide

@tay1orjones tay1orjones requested a review from a team as a code owner May 6, 2026 18:10
@tay1orjones tay1orjones requested review from emyarod and heloiselui May 6, 2026 18:10
@netlify

netlify Bot commented May 6, 2026

Copy link
Copy Markdown

Deploy Preview for v11-carbon-web-components ready!

Name Link
🔨 Latest commit 6ca8a3e
🔍 Latest deploy log https://app.netlify.com/projects/v11-carbon-web-components/deploys/69fcc33d9941580008809bb9
😎 Deploy Preview https://deploy-preview-22197--v11-carbon-web-components.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented May 6, 2026

Copy link
Copy Markdown

Deploy Preview for v11-carbon-react ready!

Name Link
🔨 Latest commit 6ca8a3e
🔍 Latest deploy log https://app.netlify.com/projects/v11-carbon-react/deploys/69fcc33d42747b0008111239
😎 Deploy Preview https://deploy-preview-22197--v11-carbon-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov

codecov Bot commented May 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.10%. Comparing base (13c5c42) to head (6ca8a3e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #22197      +/-   ##
==========================================
- Coverage   95.11%   95.10%   -0.01%     
==========================================
  Files         549      549              
  Lines       45644    45645       +1     
  Branches     6528     6556      +28     
==========================================
- Hits        43413    43412       -1     
- Misses       2101     2103       +2     
  Partials      130      130              
Flag Coverage Δ
main-packages 89.16% <100.00%> (-0.01%) ⬇️
web-components 97.89% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@netlify

netlify Bot commented May 6, 2026

Copy link
Copy Markdown

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit 6ca8a3e
🔍 Latest deploy log https://app.netlify.com/projects/carbon-elements/deploys/69fcc33db75aff0008ba4127
😎 Deploy Preview https://deploy-preview-22197--carbon-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@emyarod emyarod left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me!

@tay1orjones

Copy link
Copy Markdown
Member Author

Looks like node:slim must've switched to a newer version of node that no longer includes yarn. I switched out the dockerfiles to use npm instead.

Merged via the queue into carbon-design-system:main with commit 38ca66c May 7, 2026
40 checks passed
@tay1orjones tay1orjones deleted the task/revert-overflowmenu-child-typing branch May 7, 2026 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

severity: 1 https://ibm.biz/carbon-severity status: ready to merge 🎉

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants