Skip to content

perf(module:tabs): need add .ant-tabs-tab class reduce css computing … #7936

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

Merged
merged 1 commit into from
Jun 14, 2023

Conversation

doxingg
Copy link
Contributor

@doxingg doxingg commented May 6, 2023

…time consuming(#7935)

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #7935

What is the new behavior?

添加tabs组件路由联动中a标签的原本的父级div的class作为父级class,减少css需要计算时的范围,从而提升性能。

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@doxingg doxingg requested a review from hsuanxyz as a code owner May 6, 2023 11:12
@codecov
Copy link

codecov bot commented May 6, 2023

Codecov Report

Merging #7936 (9a945b0) into master (59143d2) will not change coverage.
The diff coverage is n/a.

❗ Current head 9a945b0 differs from pull request most recent head 0fdbf42. Consider uploading reports for the commit 0fdbf42 to get more accurate results

@@           Coverage Diff           @@
##           master    #7936   +/-   ##
=======================================
  Coverage   91.78%   91.78%           
=======================================
  Files         509      509           
  Lines       17444    17444           
  Branches     2683     2772   +89     
=======================================
  Hits        16011    16011           
  Misses       1134     1134           
  Partials      299      299           

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.

2 participants