Reviewed & tested by the community
Project:
Scrollable Views
Version:
6.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
7 Jun 2009 at 15:20 UTC
Updated:
1 Apr 2011 at 05:49 UTC
Jump to comment: Most recent file
Comments
Comment #1
dominik__w commentedChange in scrollable.module
to:
Comment #2
jcmarco commentedRTC
Added other fix and code sytiling
Comment #3
jcmarco commentedLast final patch
Comment #4
bennos commentedpatch worked in IE7 and 8
no status change, more feedback needed
Comment #5
mnlund commentedConfirmed
Comment #6
garywiz commentedI just submitted a patch as part of #511938: Allow multiple views on the same page to user the scrollable style which completely upgrades the dev version to the newest 1.1.1 version of jQuery Tools scrollable and fixes all the calling sequences and adds new view options where appropriate. The code you're patching above is a very old version of the scrollable js module and I'm certain it has lots of problems. At least worth a look, it might be a better option. I have no affiliation with anybody who did any of this, but needed a rock solid version for our site.
Comment #7
texas-bronius commentedAny updates here? The latest stable is still May 2009 at this time. The fix above (adding a semicolon after easing) didn't do it for me, but removing that same comma after easing per #600574: Extra comma causes JS error in IE6 appears to have worked for me in IE7.
Comment #8
anrkaid commentedPatch works, please remove this comma from module.