Download & Extend
Content Construction Kit (CCK) » Issues

RTL CSS files

Posted by tombigel on January 18, 2009 at 8:58pm
Project:Content Construction Kit (CCK)
Version:6.x-2.1
Component:General
Category:task
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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

AttachmentSize
cck-rtl.zip8.83 KB
Login or register to post comments

Comments

#1

Posted by tombigel on January 18, 2009 at 8:59pm
Status:active» needs review

#2

Posted by yhager on March 1, 2009 at 8:43am
Status:needs review» needs work

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

#3

Posted by elcuco on March 1, 2009 at 9:00am
Status:needs work» needs review
AttachmentSize
cck-rtl-support-tom-kicksass.patch 2.62 KB

#4

Posted by yhager on March 5, 2009 at 5:15pm

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

#5

Posted by elcuco on March 8, 2009 at 7:20am

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.

#6

Posted by tombigel on March 8, 2009 at 8:40am

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"

#7

Posted by yched on March 14, 2009 at 7:03pm
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 !

#8

Posted by System Message on March 28, 2009 at 7:10pm
Status:fixed» closed (fixed)

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