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

ConfigProvider support prefixCls #13389

Merged
merged 70 commits into from
Dec 5, 2018
Merged

ConfigProvider support prefixCls #13389

merged 70 commits into from
Dec 5, 2018

Conversation

zombieJ
Copy link
Member

@zombieJ zombieJ commented Dec 2, 2018

do not merge


Note

  • To support old version of React. Some sub item render use this.prefixCls to save current rendered prefixCls.
  • Divider prefix is ant. Change to ant-divider to keep all the prefixCls same pattern.
  • Menu pass rootPrefixCls in MenuItem & SubMenu. Use ConfigProvider also.

Not change

  • Dropdown.Button origin prefixCls not change button under group prefixCls.
  • Notification.xxx don't have context.
  • Message.xxx don't have context.
  • Modal.xxx don't have context.
  • UploadList of Upload not change prefixCls when prefixCls set.

@netlify
Copy link

netlify bot commented Dec 2, 2018

Deploy preview for ant-design ready!

Built with commit 0bc93a3

https://deploy-preview-13389--ant-design.netlify.com

@codecov
Copy link

codecov bot commented Dec 2, 2018

Codecov Report

Merging #13389 into feature will decrease coverage by 0.23%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           feature   #13389      +/-   ##
===========================================
- Coverage    92.78%   92.55%   -0.24%     
===========================================
  Files          222      222              
  Lines         5810     5830      +20     
  Branches      1680     1681       +1     
===========================================
+ Hits          5391     5396       +5     
- Misses         412      429      +17     
+ Partials         7        5       -2
Impacted Files Coverage Δ
components/dropdown/dropdown-button.tsx 100% <100%> (ø) ⬆️
components/time-picker/index.tsx 82.81% <100%> (ø) ⬆️
components/anchor/Anchor.tsx 91.12% <100%> (+0.29%) ⬆️
components/tooltip/index.tsx 96.25% <100%> (ø) ⬆️
components/auto-complete/index.tsx 88.09% <100%> (+0.91%) ⬆️
components/anchor/AnchorLink.tsx 100% <100%> (ø) ⬆️
components/config-provider/index.tsx 100% <100%> (+50%) ⬆️
components/dropdown/dropdown.tsx 96.77% <100%> (ø) ⬆️
components/alert/index.tsx 92.98% <100%> (+0.25%) ⬆️
components/tree-select/index.tsx 86.36% <100%> (ø) ⬆️
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 288e308...b6c1f1c. Read the comment docs.

@codecov
Copy link

codecov bot commented Dec 2, 2018

Codecov Report

Merging #13389 into feature will increase coverage by 0.42%.
The diff coverage is 98.6%.

Impacted file tree graph

@@             Coverage Diff             @@
##           feature   #13389      +/-   ##
===========================================
+ Coverage    92.78%   93.21%   +0.42%     
===========================================
  Files          222      221       -1     
  Lines         5810     6070     +260     
  Branches      1680     1672       -8     
===========================================
+ Hits          5391     5658     +267     
+ Misses         414      407       -7     
  Partials         5        5
Impacted Files Coverage Δ
components/skeleton/Title.tsx 100% <ø> (ø) ⬆️
components/skeleton/Paragraph.tsx 100% <ø> (ø) ⬆️
components/skeleton/Avatar.tsx 100% <ø> (ø) ⬆️
components/pagination/Pagination.tsx 100% <100%> (ø) ⬆️
components/back-top/index.tsx 92.06% <100%> (+0.53%) ⬆️
components/radio/radio.tsx 100% <100%> (ø) ⬆️
components/tree/Tree.tsx 96.55% <100%> (+0.55%) ⬆️
components/anchor/AnchorLink.tsx 100% <100%> (ø) ⬆️
components/badge/index.tsx 100% <100%> (ø) ⬆️
components/config-provider/index.tsx 100% <100%> (+50%) ⬆️
... and 78 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bdba406...0bc93a3. Read the comment docs.

@zombieJ zombieJ changed the title [WIP] ConfigProvider support prefixCls ConfigProvider support prefixCls Dec 5, 2018
@zombieJ
Copy link
Member Author

zombieJ commented Dec 5, 2018

写的 👁 都要弹出来了,文件很多,大家直接看看预览对现有的样式有没有影响。没有就合了,以后有 ConfigProvider 的 issue 再单独处理。

@afc163
Copy link
Member

afc163 commented Dec 5, 2018

image

@afc163
Copy link
Member

afc163 commented Dec 5, 2018

貌似没啥问题。

@chenshuai2144
Copy link
Contributor

review 的眼睛也很疼

@yesmeck
Copy link
Member

yesmeck commented Dec 5, 2018

这么多提交,要 squash 么?

@zombieJ
Copy link
Member Author

zombieJ commented Dec 5, 2018

嗯,会 squash 掉。改动文件比较多,尽快合进去,免得一直要处理冲突。

@zombieJ zombieJ merged commit 1a0a06f into feature Dec 5, 2018
@zombieJ zombieJ deleted the prefixCls branch December 5, 2018 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants