-
-
Notifications
You must be signed in to change notification settings - Fork 50.8k
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: add message rtl #22513
style: add message rtl #22513
Conversation
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 6247b44:
|
Codecov Report
@@ Coverage Diff @@
## master #22513 +/- ##
=======================================
Coverage 98.28% 98.28%
=======================================
Files 361 361
Lines 7212 7215 +3
Branches 1971 1934 -37
=======================================
+ Hits 7088 7091 +3
Misses 124 124
Continue to review full report at Codecov.
|
@xrkffgg thanks for your great effort on RTL of antd. |
Agree with @saeedrahimi |
I agree and I know this is not a good solution. |
message.config({
rtl: true,
}) How about global config for |
0d3d3a9
to
6c4120c
Compare
I have changed, how about this? |
🤔 This is a ...
🔗 Related issue link
close #22473
💡 Background and solution
用
ConfigConsumer
嵌套 Message,试了半天没成功,所以用了这种方式。📝 Changelog
☑️ Self Check before Merge
View rendered components/message/demo/other.md
View rendered components/message/index.en-US.md
View rendered components/message/index.zh-CN.md