I worked with this module and had a lot of issues to incorporate imageFlow with views on IE7. Basically the work around for me was not to use the settings with in the view but to alter Java script to be able to see the flow on IE7. Have anyone else had the same issue?

Comments

struesda’s picture

struesda commented

Unfortunately I seem to have arrived at the same problem. In IE7 the views settings for Views Slideshow are completely missing.
In Firefox they are partially there - but neither one is rendering anything in the page view.

I'm just starting to debug into it - in hopes of finding something and not having to hack JS.

struesda’s picture

struesda commented

I found that Using views_slideshow_imageflow-6.1.x-2.x-dev with views_slideshow-6.x-2.1 works.
(I thought I had tried it before - but one final attempt and it is working.)

Now I'm working on getting it to work with non-standard opacity and reflection settings.

I'm trying to change the javascript file in the module - but the changes are not being rendered to the page when I refresh it - but I'm pretty sure I'm just missing some drupal js cache setting or something - I don't think it is an issue with the module anymore.

FreddieK’s picture

FreddieK commented

#subscribing.

ivan zugec’s picture

ivan zugec commented

Were you guys getting any JavaScript errors?

On IE7 I was getting a "Expected identifier, string or number" error. I was getting this error when I was viewing the imageflow view, not editing the view.

I fixed it by removing a trailing comma from line 45 in views_slideshow_imageflow.js. On version 6.x-2.0-alpha1.
From this:

      var _settings = {
        ImageFlowID: id,
        reflections: false,
        imagesHeight: flow['imagesHeight'],
        aspectRatio: flow['aspectRatio'],
        imageCursor: flow['imageCursor'],
        startID: flow['startID'],
        slider: flow['slider'],
        sliderCursor: flow['sliderCursor'],
        captions: flow['captions'],
        imageFocusM : flow['imageFocusM'],
        scrollbarP : flow['scrollbarP'],
        imageFocusMax : flow['imageFocusMax'],
        opacity : true,
        opacityArray : [10, 3, 2, 1, 0],
        buttons : flow['buttons'],
      };

To this:

      var _settings = {
        ImageFlowID: id,
        reflections: false,
        imagesHeight: flow['imagesHeight'],
        aspectRatio: flow['aspectRatio'],
        imageCursor: flow['imageCursor'],
        startID: flow['startID'],
        slider: flow['slider'],
        sliderCursor: flow['sliderCursor'],
        captions: flow['captions'],
        imageFocusM : flow['imageFocusM'],
        scrollbarP : flow['scrollbarP'],
        imageFocusMax : flow['imageFocusMax'],
        opacity : true,
        opacityArray : [10, 3, 2, 1, 0],
        buttons : flow['buttons'] //NOTICE: No trailing comma
      };
mikebell_’s picture

mikebell_ commented

Seeing this issue in the latest dev release as well, fix in #4 works.