-
-
Notifications
You must be signed in to change notification settings - Fork 219
💄 style(ui): Adopt directional icons with RTL languages (#261) #262
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
base: master
Are you sure you want to change the base?
💄 style(ui): Adopt directional icons with RTL languages (#261) #262
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
👍 @alirezaImani-f4L3e |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #262 +/- ##
=========================================
+ Coverage 0.00% 1.54% +1.54%
=========================================
Files 330 490 +160
Lines 22325 18046 -4279
Branches 330 513 +183
=========================================
+ Hits 0 278 +278
+ Misses 21995 17478 -4517
+ Partials 330 290 -40 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I think a better way is to add an internal rtl control in the ActionIcon and Icon Component |
|
I think it shouldn’t be managed in ActionIcon because it’s dependent on the icon being used, but ActionIcon shouldn’t be aware of the Icon content. |
|
is there a problem with pr ? if you would not merge it , please fix this problem . |
91110db to
bdd17cf
Compare
commit: |
💻 变更类型 | Change Type
🔀 变更说明 | Description of Change
I have added a useDirection hook to provide app direction using Antd direction config .
We will choose Icons direction using useDirection hook .