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

Allow to use any CSS units for layout width #10479

Merged
merged 5 commits into from
May 11, 2018
Merged

Conversation

ApacheEx
Copy link
Contributor

@ApacheEx ApacheEx commented May 11, 2018

👋
From docs we can see that width of layout can be number or string. For me it means I can use 50%, 20em and so on. Currently, only pixels are supported.

This PR is going to fix it. It will allow to use any CSS units for layout width.

Resolves #9719 💯

@ant-design-bot
Copy link
Contributor

ant-design-bot commented May 11, 2018

Deploy preview for ant-design ready!

Built with commit b180cb6

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

@codecov
Copy link

codecov bot commented May 11, 2018

Codecov Report

Merging #10479 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #10479      +/-   ##
==========================================
- Coverage   86.48%   86.48%   -0.01%     
==========================================
  Files         196      196              
  Lines        4795     4794       -1     
  Branches     1339     1338       -1     
==========================================
- Hits         4147     4146       -1     
  Misses        645      645              
  Partials        3        3
Impacted Files Coverage Δ
components/layout/Sider.tsx 76.54% <100%> (-0.29%) ⬇️

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 dd7fb9d...b180cb6. Read the comment docs.

@afc163
Copy link
Member

afc163 commented May 11, 2018

Could you add a test case?

@afc163 afc163 merged commit 71abb96 into ant-design:master May 11, 2018
bors bot referenced this pull request in mozilla/delivery-console May 14, 2018
160: Update dependency antd to v3.5.2 r=rehandalal a=renovate[bot]

This Pull Request updates dependency [antd](https://github.com/ant-design/ant-design) from `v3.5.1` to `v3.5.2`



<details>
<summary>Release Notes</summary>

### [`v3.5.2`](https://github.com/ant-design/ant-design/releases/3.5.2)
[Compare Source](ant-design/ant-design@3.5.1...3.5.2)
- 🐞 Fixed `Cascader` displayRender z-index issue. [#&#8203;10433](`https://github.com/ant-design/ant-design/issues/10433`)
- 🐞 Fixed `Button` Types of property 'ref' are incompatible. [#&#8203;10405](`https://github.com/ant-design/ant-design/issues/10405`)
- 🐞 Fixed `Table` filter doesn't work on checkbox click. [#&#8203;10452](`https://github.com/ant-design/ant-design/issues/10452`)
- 🐞 Fixed the height of `Form` item when checks position is dithered. [#&#8203;10445](`https://github.com/ant-design/ant-design/issues/10445`)
- 🌟 Allow to use any CSS units for `Layout` width. [#&#8203;10479](`https://github.com/ant-design/ant-design/pull/10479`)


---


- 🐞 修复 `Cascader` 的 displayRender 问题。[#&#8203;10433](`https://github.com/ant-design/ant-design/issues/10433`)
- 🐞 修复 `Button` ref 不兼容问题。[#&#8203;10405](`https://github.com/ant-design/ant-design/issues/10405`)
- 🐞 修复 `Table` 过滤器在复选框上不起作用。[#&#8203;10452](`https://github.com/ant-design/ant-design/issues/10452`)
- 🐞 修复 `Form` 表单项校验位置高度时出现抖动问题。[#&#8203;10445](`https://github.com/ant-design/ant-design/issues/10445`)
- 🌟 设置 `Layout` 宽度时,允许使用任何的 CSS 单位。 [#&#8203;10479](`https://github.com/ant-design/ant-design/pull/10479`)

---

</details>




---

This PR has been generated by [Renovate Bot](https://renovatebot.com).

Co-authored-by: Renovate Bot <bot@renovateapp.com>
@potaty
Copy link

potaty commented May 19, 2019

I appreciated your time and effort! It helps me a lot!

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.

Layout.Sider does not support "style" prop (incorrect docs)
4 participants