Hi,

I am using Drupal 5.3 and the dev version of jCarousel Module. I have installed jquery_update as per the README.

The carousel runs perfectly in IE7 and FF, but does not work at all in IE6. I have refered to the list I am trying to work with by the class of the enclosing div. The DOM part works so the HTML gets rewritten and all the list items have the correct classes attached. The actual part where the list is animated does not work, and all nodes that would appear in the carousel are show at the one time (ie. as a normal list).

Any hints or tips getting things working in IE6?

Thanks,

BlueSquare.

Comments

bluesquare’s picture

bluesquare commented

Ok,

So 7 hours (!) after I headed down this path I found a solution...

It seems in stripping the skins out the jCarousel Module one crucial css definition was lost.

.jcarousel-clip-horizontal

When i added this back into my site css file jCarousel started working as expected in ie6. .jcarousel-clip-vertical may also be important but I am not using it on my site. It may also be worth checking to see if any other css definitions that normally reside in the skins css file are important.

On a related note I found that in the main jcarousel.css file the height was being set to 75px for .jcarousel-item. This does not seem to get rewritten when jcarousel updates the DOM as I get a carousel clipped at 75px. I set this height for .jcarousel-item to 100% and this fixes the issue.

Cheers,

BlueSquare

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Category: bug » support
Status: Active » Fixed

Seems to me like you didn't properly refresh your browser, or that you have somehow messed up the installation. You're the only one with IE6 problems. Or perhaps you're not yet using the 1.1 release of this module?

bluesquare’s picture

bluesquare commented

Hi,

I was using the August 15th build of 5.x-1.x-dev as per this commit (http://drupal.org/cvs?commit=77506). The commit previous to that on the same day (#77505) removed all the skins which has css files with declarations required to get jcarousel working in ie6.

To get this version working at all in ie6 I had to add the following declarations to my global css file;

/* REQUIRED FOR JCAROUSEL TO WORK */

.jcarousel-clip-horizontal {
width: 100%;
}

.jcarousel-item {
height: 100%;
}

I have tried the latest build (#88020) but it gives me a script error in ie6...

Line: 232
Char: 7
Error: 'this.clip.offsetWidth' is null or not an object
Code: 0

and the jcarousel does not display properly.

Cheers,

BlueSquare.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Fixed » Active

In that case, I"ll have to try to reproduce this one more time.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Priority: Critical » Normal
Status: Active » Closed (works as designed)

1) The images in the styles are not GPL'ed, hence they had to be removed. For guidelines on how to build jCarousel styles, you should look at the jCarousel documentation. Hence this is not a bug or missing feature.

2) That's a bug in the jCarousel plugin itself, which is not maintained by me. I've updated it to a newer version too some time ago. I'm asuming this is fixed. If it's not, you should report it with the jCarousel plugin maintainer.

scott.bell’s picture

scott.bell commented

One thing that IE6 doesn't like is a comma after the last attribute. For instance.

$('#mydiv').jcarousel({
scroll: 5,
start: 1,
});

This will throw an error in IE6 because IE6 is all "well there's a comma so where's the next option!?". Remove the comma and it will work.

$('#mydiv').jcarousel({
scroll: 5,
start: 1
});