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;
}
CommentFileSizeAuthor
#1 jcarousel.rtl_.diff451 bytesshushu
jquery.jcarousel.rtl_support.diff855 bytesshushu

Comments

shushu’s picture

shushu commented
StatusFileSize
new jcarousel.rtl_.diff451 bytes

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

heyyo’s picture

heyyo commented

Could you help to port your fix to views carousel ?

thanks.

http://drupal.org/node/433078

heyyo’s picture

heyyo commented

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...

guyzalon’s picture

guyzalon commented

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

.jcarousel-container {
direction:ltr;
}

Good luck,
Guy.

robloach’s picture

robloach
he/him
commented
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?

ilw’s picture

ilw commented

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

.jcarousel-container {
direction:ltr;
}

Good luck,
Guy."

Thanks!

That worked for me...

amirtaiar’s picture

amirtaiar commented

#4 solve the issue for me - Thank you!