-
-
Notifications
You must be signed in to change notification settings - Fork 49.8k
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
refactor: Menu migrate to @rc-component/trigger #42554
Conversation
size-limit report 📦
|
在PR里一并缩小limit size? |
小这么多…… |
New dependency changes detected. Learn more about Socket for GitHub ↗︎ 👍 No new dependency issues detected in pull request Bot CommandsTo ignore an alert, reply with a comment starting with Pull request alert summary
📊 Modified Dependency Overview:
|
应该能关掉不少 issues,可以搜一下关联进来。 |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## feature #42554 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 640 640
Lines 10865 10865
Branches 2955 2955
=========================================
Hits 10865 10865 ☔ View full report in Codecov by Sentry. |
这个 PR 的 changelog 可以好好写写
|
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
#41249
💡 Background and solution
干净了
📝 Changelog
@rc-component/trigger
. - Removerc-trigger
and reduce bundle size.@rc-component/trigger
重构 Menu 组件。 - 移除rc-trigger
依赖,缩小打包体积。☑️ Self-Check before Merge
🚀 Summary
🤖 Generated by Copilot at 56fdcd8
This pull request improves the
Menu
component by updating its dependencyrc-menu
and refining its submenu theme demo.🔍 Walkthrough
🤖 Generated by Copilot at 56fdcd8
rc-menu
dependency to~9.9.0
for bug fixes and improvements (link)import/order
(link)getPopupContainer
prop fromMenu
component as it is no longer needed afterrc-menu
refactoring (link)