Closed (fixed)
Project:
Views carousel
Version:
6.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
8 Nov 2009 at 17:43 UTC
Updated:
26 Mar 2011 at 13:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
HunterElliott commentedActually, 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.
Comment #2
TimMachine commentedHas 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.
Comment #3
eft commentedThis may be related to IE's
tag limitation issue. See http://drupal.org/node/648254#comment-2338452Comment #4
HunterElliott commentedThe 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.
Comment #5
eft commented@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.
Comment #6
drewish commentedI'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
Comment #7
drewish commentedhere'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.
Comment #8
mfer commentedReviewing..... :)
Comment #9
mfer commentedI 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.
Comment #10
drewish commentedshouldn't the jquery.jcarousel.min.js file be deleted? since it's out of sync with the .js and .pack.js files?
Comment #11
mfer commented/me grumbles at CVS and removes the file.
@drewish thanks for catching this.
Comment #12
ljrweb commentedjust to confirm... that patch fixes the problem WITHOUT caching the css??
will this be released as a version upgrade soon?
thanks
Comment #13
mfer commented@ljrweb The update is in the release candidate available now. Barring something major happening there will be a release on Wednesday.