Skip to content

Install from web - rtl fix#38349

Merged
wilsonge merged 2 commits intojoomla:4.2-devfrom
brianteeman:webinstaller
Sep 5, 2022
Merged

Install from web - rtl fix#38349
wilsonge merged 2 commits intojoomla:4.2-devfrom
brianteeman:webinstaller

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

@brianteeman brianteeman commented Jul 31, 2022

Summary of Changes

Changes the css to use logical properties so that the sidebar is indented the same in RTL as it is in LTR

Testing Instructions

This is a css change so either run npm build:css or use a [re-built package

Before

LTR
image

RTL
image

After

LTR
image

RTL
image

Changes the css to use logical properties so that the sidebar is indented the same in RTL as it is in LTR
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.2-dev labels Jul 31, 2022
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Aug 3, 2022

I have tested this item ✅ successfully on a95ec55


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38349.

1 similar comment
@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 4, 2022

I have tested this item ✅ successfully on a95ec55


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38349.

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 4, 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38349.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 4, 2022
@wilsonge wilsonge merged commit b113c87 into joomla:4.2-dev Sep 5, 2022
@wilsonge
Copy link
Copy Markdown
Contributor

wilsonge commented Sep 5, 2022

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 5, 2022
@wilsonge wilsonge added this to the Joomla! 4.2.3 milestone Sep 5, 2022
@brianteeman
Copy link
Copy Markdown
Contributor Author

thanks

@brianteeman brianteeman deleted the webinstaller branch September 5, 2022 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants