This depends on #283978: RTL support for IE stylesheets.

I didn't understand what the -200 px margin in the sidebars are for, but I've added them just as-well. Maybe they are redundant?

CommentFileSizeAuthor
#5 rtl-flip-sidebars-375976-5.patch11.49 KBjohnalbin
#4 flip_sidebars_rtl_4.patch2.83 KBamitaibu
#3 flip_sidebars_rtl_3.patch2.82 KBamitaibu
#2 flip_sidebars_rtl_2.patch2.72 KBamitaibu
flip_sidebars_rtl.patch1.34 KBamitaibu

Comments

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented
Status: Needs review » Needs work

There is a problem with showing a single Sidebar, so setting to CNW.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented
Status: Needs work » Needs review
StatusFileSize
new flip_sidebars_rtl_2.patch2.72 KB

THe patch updated template.php according to the new names of the sidebars.
Patch is dependent on #375953: Change sidebar names to be RTL friendly

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented
StatusFileSize
new flip_sidebars_rtl_3.patch2.82 KB

Small fix

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented
StatusFileSize
new flip_sidebars_rtl_4.patch2.83 KB

in page preprocess add the right amount of sidebars.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Title: Replace sidebars on RTL » Flip sidebar positioning on RTL
Component: Code » layout.css
StatusFileSize
new rtl-flip-sidebars-375976-5.patch11.49 KB

Updated patch.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Status: Needs review » Fixed

Committed! :-)

Status: Fixed » Closed (fixed)
Issue tags: -RTL

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +RTL

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.