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

Added "middle" and "small" classes for the Card component (like Table) #13180

Merged
merged 1 commit into from
Dec 11, 2018

Conversation

ndbroadbent
Copy link
Contributor

@ndbroadbent ndbroadbent commented Nov 17, 2018

  • Make sure that you propose PR to right branch: bugfix for master, feature for branch feature.
  • Make sure that you follow antd's code convention.
  • Run npm run lint and fix those errors before submitting in order to keep consistent code style.
  • Rebase before creating a PR to keep commit history clear.
  • Add some descriptions and refer relative issues for you PR.

Extra checklist:

  • Update API docs for the component.
  • Update/Add demo to demonstrate new feature.
  • Update TypeScript definition for the component.
  • Add unit tests for the feature.

Fixes #13096

@codecov
Copy link

codecov bot commented Nov 17, 2018

Codecov Report

Merging #13180 into feature will decrease coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           feature   #13180      +/-   ##
===========================================
- Coverage    93.26%   93.25%   -0.02%     
===========================================
  Files          222      222              
  Lines         6195     6196       +1     
  Branches      1720     1721       +1     
===========================================
  Hits          5778     5778              
- Misses         409      410       +1     
  Partials         8        8
Impacted Files Coverage Δ
components/card/index.tsx 86.58% <100%> (+0.16%) ⬆️
components/_util/wave.tsx 85.29% <0%> (-2.95%) ⬇️
components/upload/UploadList.tsx 89.18% <0%> (+1.8%) ⬆️

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 7db8005...77c7287. Read the comment docs.

@netlify
Copy link

netlify bot commented Nov 17, 2018

Deploy preview for ant-design ready!

Built with commit 7c9f9e566b8b38be43d90be0c35631570c59fe57

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

@netlify
Copy link

netlify bot commented Nov 17, 2018

Deploy preview for ant-design ready!

Built with commit 77c7287

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

@ndbroadbent ndbroadbent force-pushed the card-small-middle branch 3 times, most recently from 22f6b8d to 7f11611 Compare November 17, 2018 20:58
@ndbroadbent
Copy link
Contributor Author

Thanks for your feedback @ilanus, I have updated the PR

components/card/index.tsx Outdated Show resolved Hide resolved
@ilanus ilanus requested a review from afc163 November 18, 2018 20:31
@ndbroadbent ndbroadbent force-pushed the card-small-middle branch 2 times, most recently from 7a39ad9 to 0b3ee13 Compare December 10, 2018 19:26
components/card/index.tsx Outdated Show resolved Hide resolved
@ndbroadbent ndbroadbent force-pushed the card-small-middle branch 2 times, most recently from 58da314 to 464632c Compare December 11, 2018 10:26
@afc163 afc163 merged commit cbed63c into ant-design:feature Dec 11, 2018
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.

3 participants