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
chore: feature merge master #45944
chore: feature merge master #45944
Conversation
Co-authored-by: zbw01218944 <zbw01218944@alibaba-inc.com>
* fix: disabled btn with link should not be focused * chore: update snapshot * chore: update snapshot
* docs: update Nextjs docs * Update use-with-next.en-US.md Signed-off-by: lijianan <574980606@qq.com> * Update use-with-next.zh-CN.md Signed-off-by: lijianan <574980606@qq.com> * update * Update use-with-next.en-US.md Signed-off-by: lijianan <574980606@qq.com> * Update use-with-next.zh-CN.md Signed-off-by: lijianan <574980606@qq.com> --------- Signed-off-by: lijianan <574980606@qq.com>
* chore: add image to Next.js doc * Update use-with-next.en-US.md Signed-off-by: lijianan <574980606@qq.com> * Update use-with-next.zh-CN.md Signed-off-by: lijianan <574980606@qq.com> --------- Signed-off-by: lijianan <574980606@qq.com>
* feat: export sider context from layout * refactor: use internal naming
* fix: zIndex miss of message * fix: modal static index * test: update snapshot * test: update snapshot * test: update snapshot * test: fix zIndex test * test: update test case * chore: clean up * test: coverage
Run & review this pull request in StackBlitz Codeflow. |
size-limit report 📦
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit ff462ad:
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feature #45944 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 706 706
Lines 12117 12133 +16
Branches 3196 3199 +3
=========================================
+ Hits 12117 12133 +16 ☔ View full report in Codecov by Sentry. |
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
📝 Changelog
☑️ Self-Check before Merge
🚀 Summary
🤖[deprecated] Generated by Copilot at ff462ad
This pull request fixes some zIndex issues for various components, adds a new prop to the
Button
component, improves theuseZIndex
hook and its test cases, cleans up some unused or redundant code and tokens, refactors theConfirmDialog
component, enhances the nested modal example, updates the Next.js documentation, and marks a deprecated version.🔍 Walkthrough
🤖[deprecated] Generated by Copilot at ff462ad
useZIndex
hook logic and test cases, where the zIndex value is computed based on the component type, parent zIndex, and token value, and the max zIndex value for the container components is limited by theCONTAINER_MAX_OFFSET
constant (link,link,link,link,link,link,link)Button
component, where thetabIndex
attribute can be controlled by themergedDisabled
state (link)SiderContext
as an internal prop for theLayout
component, where the context can be used for testing purposes (link,link,link)Menu
component and its style functions, where thepopupBg
anddarkPopupBg
values are not used (link,link,link,link,link,link,link,link,link,link)CONTAINER_MAX_OFFSET
constant, where the components can use the constant imported from theuseZIndex
hook and theprepareComponentToken
function (link,link,link,link)ConfirmDialog
component and its wrapper, where theDialog
component is moved to the wrapper and uses themergedZIndex
value computed by theuseToken
hook and thezIndex
prop, and some unused props are removed (link,link,link,link,link,link,link)useModal
hook test case, where thezIndex
prop and style are set to 903 to match the expected value (link,link)nested.tsx
file, where theButton
,message
,notification
, andSpace
components are used to demonstrate the zIndex behavior of the static and hook methods of the message and notification components, and theSwitch
component style is dynamically changed based on theisModalOpen
state (link,link,link,link)use-with-next.en-US.md
anduse-with-next.zh-CN.md
files, where the notes warn about the potential issue of installing@ant-design/cssinjs
with a different version than the one used byantd
, which can cause multiple React instances and ctx errors (link,link,link,link)ci-mock-project-build.sh
script, where the script adds some echo messages to indicate the progress of the commands, and adds a command to clean up thepackageManager
field from thepackage.json
file, which can cause issues with the yarn install command (link)DEPRECIATED_VERSION
object in thepost-script.ts
file, where the entry marks the version 5.11.1 as deprecated and links to the issue that explains the reason (link)