-
-
Notifications
You must be signed in to change notification settings - Fork 49k
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
style: Alert rtl #28108
style: Alert rtl #28108
Conversation
Size Change: -220 B (0%) Total Size: 789 kB
ℹ️ View Unchanged
|
components/alert/style/rtl.less
Outdated
right: auto; | ||
left: @padding-md; | ||
margin-right: auto; | ||
margin-left: 37px - 16 - @font-size-base; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
为啥会有 37px - 数字的?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里主要是为了满足和上一版ui完全一致 算的
这里需要简单化处理吗? @afc163
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
他的意思是37px - 16
会是非法值,是不是笔误?
简化吧,建议按 index.less 里的样式完全重写一下。 |
现在用了 flex,甚至直接 reverse 一下就行。 |
components/alert/style/index.less
Outdated
@@ -111,7 +108,7 @@ | |||
} | |||
|
|||
&-with-description &-icon { | |||
margin-right: @alert-with-description-padding-vertical; | |||
margin-right: @margin-md; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个别动,担心有人覆盖过这个变量。
|
嗯,有 direction: rtl 就够了。 |
不使用 reverse 就现在这样吗 |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 905bcf6:
|
Codecov Report
@@ Coverage Diff @@
## master #28108 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 393 393
Lines 7388 7373 -15
Branches 2119 2112 -7
=========================================
- Hits 7388 7373 -15
Continue to review full report at Codecov.
|
Action 的位置左移了很多,请检查 https://www.argos-ci.com/ant-design/ant-design/builds/896 |
有一处错误, 已经更改了. |
* style: Alert rtl * update * icon space * update * update * fix
* style: Alert rtl * update * icon space * update * update * fix
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
📝 Changelog
rtl
样式。☑️ Self Check before Merge