Hi,
views carousel suddenly stopped working in internet explorer, it works well in firefox. I'm not sure what exactly has been changed.
You can see the problem on the front page of my production site: http://kidsgame.co.il
:(
Please also find attached image,
Thanks,
Guy.

CommentFileSizeAuthor
#7 viewscarousel_js_update.patch24.94 KBdrewish
viewsC.jpg33.58 KBguyzalon

Comments

HunterElliott’s picture

Version: 6.x-2.x-dev » 6.x-1.x-dev

Actually, it stopped working in the 6.x-1.x-dev version on the 6th, too. It works(-ed) in FF and Safari fine... but just totally disappeared in IE8 for some reason when it had been working fine up until the 6th. This makes me wonder if there was an IE/Windows update that caused the problem.

TimMachine’s picture

Has anyone figured out a workaround to this problem?

Update: This is working in IE7 and IE8. My issue was a permissions problem with the photo field.

eft’s picture

This may be related to IE's

tag limitation issue. See http://drupal.org/node/648254#comment-2338452
HunterElliott’s picture

The solution I found to this is to enable CSS caching in the Performance section of the site. I have no idea *why* this would work, only that it does.

eft’s picture

@HunterElliott - Because it consolidates a lot of individual css files into one, thereby eliminating the issue with IE, which ignores more than 30 css files.

drewish’s picture

I've run into an issue with IE where the visible() method is incorrectly reporting the list is hidden. I just looked and the plugin seems to have evolved quite a bit since then: http://github.com/jsor/jcarousel/blob/master/lib/jquery.jcarousel.js

drewish’s picture

Status: Active » Needs review
StatusFileSize
new24.94 KB

here's a patch that updates the js to 0.2.3 from the author's github: http://github.com/jsor/jcarousel/downloads

they've switched from minified to packed but i preserved the .min filename.

mfer’s picture

Reviewing..... :)

mfer’s picture

Status: Needs review » Fixed

I updated to 0.2.3. This includes switching to the .pack.js file naming, updating the unpacked version, and switching the drupal_add_js call to the .pack file.

drewish’s picture

Status: Fixed » Needs work

shouldn't the jquery.jcarousel.min.js file be deleted? since it's out of sync with the .js and .pack.js files?

mfer’s picture

Status: Needs work » Fixed

/me grumbles at CVS and removes the file.

@drewish thanks for catching this.

ljrweb’s picture

just to confirm... that patch fixes the problem WITHOUT caching the css??

will this be released as a version upgrade soon?

thanks

mfer’s picture

@ljrweb The update is in the release candidate available now. Barring something major happening there will be a release on Wednesday.

Status: Fixed » Closed (fixed)

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