Support RTL

shushu - May 14, 2009 - 09:27
Project:jCarousel
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs work
Description

When using RTL themes, jcarousel just does not work properly.
The simple solution I thought about it to replace left to right, so I hacked the js code (and the views carousel module used to make it work).

In addition, the relevat CSS files should be handled using additional -rtl.css files (especially the float property):

.jcarousel-list li,
.jcarousel-item {
    float: right;
}
.jcarousel-skin-custom .jcarousel-item-horizontal {
    margin-right: 1.5em;
    margin-left: 10px;
}

AttachmentSize
jquery.jcarousel.rtl_support.diff855 bytes

#1

shushu - May 14, 2009 - 09:35

Another patch I forgot to upload (adds the rtl option according to global language)

AttachmentSize
jcarousel.rtl_.diff 451 bytes

#2

heyyo - May 14, 2009 - 10:02

Could you help to port your fix to views carousel ?

thanks.

http://drupal.org/node/433078

#3

heyyo - May 15, 2009 - 08:37

Thanks for your patch, I finally use the last version of views carousel 6.x-2.x-dev. It's working perfectly, I hope your patch will be applyed for the next release...

#4

guyzalon - September 25, 2009 - 13:45

Just try to add this to style.css, it worked for me:

.jcarousel-container {
direction:ltr;
}

Good luck,
Guy.

#5

Rob Loach - September 25, 2009 - 16:00
Status:active» needs work

Changing jquery.jcarousel.js will mean we'll loose the modification when we update jCarousel.

What if in a CSS file we have a jcarousel.rtl.css with what guyzalon uploaded?

 
 

Drupal is a registered trademark of Dries Buytaert.