Hello,
in advance congratulation for the module!
I have a problem with IE7, IE8 seems to work perfectly.
IE7 say me that there is an error at the end of the script, and don't scroll..
prevPage: ".prevPage",
nextPage: ".nextPage",
navi: ".navi",
easing: "swing",
}); <--- here

Result on the snapshot.

Thanks for your attenction.

Comments

dominik__w’s picture

Change in scrollable.module

  $js = '$(document).ready(function() {         
           // initialize scrollable  
           $("div.scrollable").scrollable({
             size: '. $options['size'] .',
             vertical: '. $options['vertical'] .',
             clickable: '. $options['clickable'] .',
             loop: '. $options['loop'] .',
             interval: '. $options['interval'] .',
             speed: '. $options['speed'] .',
             keyboard: '. $options['keyboard'] .',
             items: ".'. $options['items'] .'",
             prev: ".'. $options['prev'] .'",
             next: ".'. $options['next'] .'",
             prevPage: ".'. $options['prevPage'] .'",
             nextPage: ".'. $options['nextPage'] .'",
             navi: ".'. $options['navi'] .'",
             easing: "'. $options['easing'] .'",
           });     
         });';

to:

  $js = '$(document).ready(function() {         
           // initialize scrollable  
           $("div.scrollable").scrollable({
             size: '. $options['size'] .',
             vertical: '. $options['vertical'] .',
             clickable: '. $options['clickable'] .',
             loop: '. $options['loop'] .',
             interval: '. $options['interval'] .',
             speed: '. $options['speed'] .',
             keyboard: '. $options['keyboard'] .',
             items: ".'. $options['items'] .'",
             prev: ".'. $options['prev'] .'",
             next: ".'. $options['next'] .'",
             prevPage: ".'. $options['prevPage'] .'",
             nextPage: ".'. $options['nextPage'] .'",
             navi: ".'. $options['navi'] .'",
             easing: "'. $options['easing'] .'";
           })     
         });';
jcmarco’s picture

Status: Active » Reviewed & tested by the community
StatusFileSize
new436 bytes

RTC

Added other fix and code sytiling

jcmarco’s picture

Status: Reviewed & tested by the community » Needs review
StatusFileSize
new1.84 KB

Last final patch

bennos’s picture

patch worked in IE7 and 8

no status change, more feedback needed

mnlund’s picture

Confirmed

garywiz’s picture

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

texas-bronius’s picture

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

anrkaid’s picture

Status: Needs review » Reviewed & tested by the community

Patch works, please remove this comma from module.