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

Fix: tree component vertical alignment of text #11473

Closed
wants to merge 3 commits into from
Closed

Fix: tree component vertical alignment of text #11473

wants to merge 3 commits into from

Conversation

ilanus
Copy link
Contributor

@ilanus ilanus commented Jul 26, 2018

closes #11458

tree-preview

@ant-design-bot
Copy link
Contributor

ant-design-bot commented Jul 26, 2018

Deploy preview for ant-design ready!

Built with commit 6a0d06e

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

@yesmeck
Copy link
Member

yesmeck commented Jul 27, 2018

Setting vertical-align to middle seems not a solution.

@codecov
Copy link

codecov bot commented Jul 27, 2018

Codecov Report

Merging #11473 into master will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11473      +/-   ##
==========================================
+ Coverage   91.84%   91.89%   +0.04%     
==========================================
  Files         199      199              
  Lines        5030     5060      +30     
  Branches     1409     1415       +6     
==========================================
+ Hits         4620     4650      +30     
+ Misses        406      404       -2     
- Partials        4        6       +2
Impacted Files Coverage Δ
components/time-picker/index.tsx 74.5% <0%> (ø) ⬆️
components/transfer/index.tsx 87.41% <0%> (ø) ⬆️
components/pagination/Pagination.tsx 100% <0%> (ø) ⬆️
components/select/index.tsx 100% <0%> (ø) ⬆️

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 ee1c726...6a0d06e. Read the comment docs.

@yesmeck yesmeck requested a review from zombieJ July 27, 2018 10:06
@zombieJ
Copy link
Member

zombieJ commented Jul 30, 2018

Thanks for your contribution. We've adjust css of this file.

@zombieJ zombieJ closed this Jul 30, 2018
@ilanus ilanus changed the title fix: tree component vertical alignment of text Fix: tree component vertical alignment of text Oct 23, 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.

Tree text vertical-align not middle
4 participants