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
[4.0][RTL] CSS logical properties initial #28589
Conversation
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.
LGTM on code review
@infograf768 can you test this please. If this works it will make maintaining RTL much easier. |
I have tested this item ✅ successfully on eb0d8bb This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28589. |
My tests were done with Firefox Macintosh |
Thank you @infograf768 |
Think the toolbar cleanup you did conflicted here. If you fix the conflicts happy to get this in |
Sorted. Once this is merged I will continue a cleanup across admin css. |
Thanks! |
This css should have been removed a long time ago when @ciar4n changed the css in joomla#28589
Pull Request for Issue #28546 .
Also removed some redundant rules ( #28586 & #28585).
Summary of Changes
Implements some logical properties to reduce on RTL CSS.
Testing Instructions
Apply this patch and run
node build.js --compile-css
to update SCSS. Check RTL styling.