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

[pull] master from ant-design:master #233

Closed
wants to merge 140 commits into from

Conversation

pull[bot]
Copy link

@pull pull bot commented Nov 19, 2023

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

zombieJ and others added 30 commits October 10, 2023 16:22
* docs: init

* chore: update example

* test: update test

* docs: update doc

* chore: fix lint

* chore: update limit
* feat: add ref

* feat: table support reference

* chore: rename

* chore: proxy

* test: add test case

* chore: fix lint

* docs: update desc

* docs: update desc

* docs: update desc

* chore: clean up

* chore: fix lint
chore: feature merge master
* feat: App support component

* test: update test case

* chore: update ts
* docs: init

* feat: support inherit

* test: update snapshot
* chore: bump rc-input-number

* test: update test case
* chore: bump rc-tabs

* chore: update limit

* chore: fix lint
chore: feature merge master
* fix: dropdown not trigger onOpenChange when click menu item

* chore: add source

* docs: update

* test: update test
* feat: start the implementation of the fullscreen prop in Spin

* docs: change main spin demo

* docs: enhance demo

* test: update snapshot

* fix: address pr comments

* fix: use logical property on fullscreen class

* fix: address pr review

* feat: Added background color token

* fix: remove onClick and change demo

* feat: change spin to white when fullcreen

also use the bgmask as background, removing the bgColor token

* fix: unused import

* test: update snapshot

* Update components/spin/style/index.tsx

Signed-off-by: lijianan <574980606@qq.com>

* fix: use white color from token

* fix: not needed interpolation and version

* fix: address pr review

* fix: tip prop was not working

* test: cover tip & fullscreen case

* fix: addrress pr coments

---------

Signed-off-by: lijianan <574980606@qq.com>
Co-authored-by: lijianan <574980606@qq.com>
Co-authored-by: MadCcc <1075746765@qq.com>
* feat: z-index manager

* feat: z-index manager

* feat: update snap

* chore: update site-limit

* feat: optimize code

* feat: optimize code

* feat: add test case

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code
chore: merge master into feature
* feat: Popconfirm support z-index manager

* Update components/popconfirm/__tests__/index.test.tsx

Signed-off-by: kiner-tang(文辉) <1127031143@qq.com>

---------

Signed-off-by: kiner-tang(文辉) <1127031143@qq.com>
* feat: Popover support z-index manager

* feat: update snap

* Update components/popover/__tests__/index.test.tsx

Signed-off-by: kiner-tang(文辉) <1127031143@qq.com>

---------

Signed-off-by: kiner-tang(文辉) <1127031143@qq.com>
* feat: Tooltip support z-index manager

* feat: optimize code
zombieJ and others added 17 commits November 14, 2023 11:01
… was abnormal in the pop-up container (#45864)

* fix: Fixed an issue where the z-index setting of the pop-up component was abnormal in the pop-up container

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* feat: optimize code

* Update components/popconfirm/index.tsx

Signed-off-by: kiner-tang(文辉) <1127031143@qq.com>

---------

Signed-off-by: kiner-tang(文辉) <1127031143@qq.com>
* 📖docs: update tabs component docs

* Update components/tabs/index.zh-CN.md

Co-authored-by: afc163 <afc163@gmail.com>
Signed-off-by: yufamg <99855951+yufamg@users.noreply.github.com>

* Update components/tabs/index.en-US.md

Co-authored-by: afc163 <afc163@gmail.com>
Signed-off-by: yufamg <99855951+yufamg@users.noreply.github.com>

---------

Signed-off-by: yufamg <99855951+yufamg@users.noreply.github.com>
Co-authored-by: afc163 <afc163@gmail.com>
* chore(deps-dev): bump size-limit from 10.0.3 to 11.0.0

Bumps [size-limit](https://github.com/ai/size-limit) from 10.0.3 to 11.0.0.
- [Changelog](https://github.com/ai/size-limit/blob/main/CHANGELOG.md)
- [Commits](ai/size-limit@10.0.3...11.0.0)

---
updated-dependencies:
- dependency-name: size-limit
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update package.json

Signed-off-by: afc163 <afc163@gmail.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: afc163 <afc163@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: afc163 <afc163@gmail.com>
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
* chore: migrate to webpack@5

* Revert "feat: support popupBg and darkPopupBg (#45826)"

This reverts commit 496ac33.

* chore: update
* docs: changelog for 5.11.2

* chore: bump version to 5.11.2

* docs: update changelog
Signed-off-by: Peach <scdzwyxst@gmail.com>
* docs: update Modal.method docs

* docs: update Modal.method US docs
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Copy link

Review changes with SemanticDiff

Copy link

coderabbitai bot commented Nov 19, 2023

Important

Auto Review Skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.


Tips

Chat with CodeRabbit Bot (@coderabbitai)

  • If you reply to a review comment from CodeRabbit, the bot will automatically respond.
  • To engage with CodeRabbit bot directly around the specific lines of code in the PR, mention @coderabbitai in your review comment
  • Note: Review comments are made on code diffs or files, not on the PR overview.
  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Commands (invoked as PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai help to get help.

Note: For conversation with the bot, please use the review comments on code diffs or files.

CodeRabbit Configration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • The JSON schema for the configuration file is available here.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json

Copy link

commit-lint bot commented Nov 19, 2023

Features

Bug Fixes

Chore

Contributors

zombieJ, li-jia-nan, kiner-tang, Rafael-Martins, github-actions[bot]

Commit-Lint commands

You can trigger Commit-Lint actions by commenting on this PR:

  • @Commit-Lint merge patch will merge dependabot PR on "patch" versions (X.X.Y - Y change)
  • @Commit-Lint merge minor will merge dependabot PR on "minor" versions (X.Y.Y - Y change)
  • @Commit-Lint merge major will merge dependabot PR on "major" versions (Y.Y.Y - Y change)
  • @Commit-Lint merge disable will desactivate merge dependabot PR
  • @Commit-Lint review will approve dependabot PR
  • @Commit-Lint stop review will stop approve dependabot PR

Copy link

pr-code-reviewer bot commented Nov 19, 2023

👋 Hi there!

Everything looks good!


Automatically generated with the help of gpt-3.5-turbo.
Feedback? Please don't hesitate to drop me an email at webber@takken.io.

Copy link

Hi @pull[bot]。

请注意,当前 PR 是分支间的互相合并,请使用 BranchAutoMerge 标签来进行自动合并。切记不要、不要、不要使用 Squash

Copy link

instapr bot commented Nov 19, 2023

Great work on this pull request! The changes look good overall. Just a couple of minor suggestions:

  1. Please add a comment explaining the purpose of the new function in line 42.
  2. Consider renaming the variable x to something more descriptive in line 67.

Keep up the good work!

Copy link

Hi @pull[bot]. Thanks for your contribution. The path .github/ or scripts/ and CHANGELOG is only maintained by team members. This current PR will be closed and team members will help on this.

Copy link

Prepare preview

@github-actions github-actions bot closed this Nov 19, 2023
Copy link

Hi @pull[bot]. Thanks for your contribution. But, we don't have plan to add README of more languages. This current PR will be closed and team members will help on this.

Copy link

sweep-ai bot commented Nov 19, 2023

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

@pull pull bot added the ⤵️ pull label Nov 19, 2023
Copy link

Pull Request Review Markdown Doc

Hey there! 👋 Here's a summary of the previous tasks and their results, along with some additional information. Let's get started!

Changes

  1. In .circleci/config.yml, changed the image value from cimg/node:21.1-browsers to cimg/node:21.2-browsers.
  2. In .codesandbox/ci.json, changed the sandboxes value from antd-reproduction-template-y9vgcf to antd-reproduction-template-forked-jyh2k9.
  3. In .codesandbox/ci.json, changed the node value from 14 to 18.
  4. In .dumi/theme/common/LiveCode.tsx, changed the border property to box-shadow property for :hover and :focus pseudo-classes.
  5. In .dumi/theme/slots/Header/index.tsx, changed the message, shortMessage, more, and link values for the cn locale.
  6. In .dumi/theme/slots/Header/index.tsx, changed the theme values for colorInfoBg and colorTextBase in the ConfigProvider component.
  7. In .eslintrc.js, added 'lodash/import-scope': 2 to the rules object.
  8. In .github/workflows/issue-labeled.yml, changed the message value for the cn locale.
  9. In .github/workflows/issue-labeled.yml, added a link to explain what is a minimal reproduction.
  10. In components/_util/type.ts, added the CustomComponent type.
  11. In components/input/demo/textarea-show-count.md, removed the file.

Suggestions

  1. In .circleci/config.yml, line 7, changed the image value from cimg/node:... to cimg/node:....
  2. In the preview-deploy.yml workflow, add a condition to the download site artifact step. The step should only run if the upstream-workflow-summary output build-success is true.
  3. In the preview-deploy.yml workflow, remove the condition from the upload surge service step. The step should always run.

Bugs

  1. In the upgrade-deps.yml workflow, add a new file called .ncurc.js with the specified content.
  2. In the CHANGELOG.zh-CN.md file, add a new section for version 5.10.4 with the specified content.

Improvements

  1. Refactor the pr-open-check.yml workflow to include a step that checks if the issue mentioned in the PR has been confirmed with the designer or core team. If not, the workflow should comment on the PR and temporarily refuse it. The comment should thank the contributor.
  2. Update the comment in the pr-open-check.yml workflow to remove the Chinese translation and simplify the message.
  3. In the file components/_util/hooks/useProxyImperativeHandle.ts, consider using a Proxy object instead of manually filling a proxy object. This can simplify the code and make it more readable.

Rating

Here's a brief summary of the code rating based on the criteria of readability, performance, and security:

  • Readability: The code changes are well-documented and follow consistent coding conventions. The changes are easy to understand and maintain.
  • Performance: The code changes include optimizations and improvements that enhance the performance of the application.
  • Security: The code changes address potential security vulnerabilities and follow best practices to ensure the security of the application.

That's it! If you have any further questions or need more information, feel free to ask. Good luck with your pull request! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.