Skip to content

centered search box - rtl#38365

Merged
wilsonge merged 1 commit intojoomla:4.2-devfrom
brianteeman:center_rtl
Sep 5, 2022
Merged

centered search box - rtl#38365
wilsonge merged 1 commit intojoomla:4.2-devfrom
brianteeman:center_rtl

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

due to the daft RTL overrides we have in place the search boxes on the new module and new scheduled tasks pages are not centered in RTL.

without changing the daft RTL overrides (which really should be done) this PR fixes the problem by using the correct css for centering a block

New Module LTR

image

New Module RTL - BEFORE

image

New Module RTL - AFTER

image

due to the daft RTL overrides we have in place the search boxes on the new module and new scheduled tasks pages are not centered in RTL.

without changing the daft RTL overrides (which really should be done) this PR fixes the problem by using the correct css for centering a block
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Aug 3, 2022

I have tested this item ✅ successfully on bf60581


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

1 similar comment
@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 3, 2022

I have tested this item ✅ successfully on bf60581


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

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Aug 3, 2022

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 3, 2022
@Quy Quy added the PR-4.2-dev label Aug 3, 2022
@wilsonge wilsonge merged commit b2cb9f5 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 center_rtl branch September 5, 2022 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants