I've created *-rtl.css files for cck, based on 2.1 CSS files.
They were tested only on my dev version of Tendu theme, so this code needs review.
Attached a zip file with the original folder hierarchy, original CSS files with /*LTR*/ comments on the relevant lines, *-rtl.css files

CommentFileSizeAuthor
#3 cck-rtl-support-tom-kicksass.patch2.62 KBelcuco
cck-rtl.zip8.83 KBtombigel

Comments

tombigel’s picture

tombigel commented
Status: Active » Needs review
yhager’s picture

yhager commented
Status: Needs review » Needs work

Can you roll it as a patch, so people can actually review it?

elcuco’s picture

elcuco commented
Status: Needs work » Needs review
StatusFileSize
new cck-rtl-support-tom-kicksass.patch2.62 KB
yhager’s picture

yhager commented

Patch applies, but I am not sure what it fixes. Looks rather trivial though.

elcuco’s picture

elcuco commented

Tom, what does these CSS fix...? Do you have a list?

For all I care, this looks great on my setup as well, it works here.

tombigel’s picture

tombigel commented

To tell you the truth, I don't remember... I guess all kinds of left/right padding issues and the "move" icons locations in "manage fields" and "display fields"

yched’s picture

yched commented
Status: Needs review » Fixed

Committed to D6 branch, and foreported to HEAD for D7.
Created #402044: Field RTL CSS tweaks for the parts that now live in D7 core.

Thanks !

Status: Fixed » Closed (fixed)

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