I have the following:
[slideshow: 2, rotate=1, blend=1, layout=default, height=808, width=595, fx=scrollRight, dir=|sites/default/files/slideshow/picts/|yes| ]

I CANNOT get it to auto-rotate. Any ideas?

Comments

neofactor’s picture

Any one? Really would like to get this resolved so I can have the rotation working.

Thanks.

Tyrael’s picture

try to update the files from the cvs repository.
it works for me since then.

Tyrael

neofactor’s picture

I tried with CVS version and no change. Still no auto-rotate.
What is does you code look like?

neofactor’s picture

Anyone else? Can you post me your code and I will try it.

elvis2’s picture

I can confirm that auto rotate is not working on my end either. The .js files seem to be available in the head tags, the images are in the html as well. Just no rotation. I have tried jquery_plugin 6.x-1.4 and jquery_plugin 6.x-1.3 with no luck. I did make sure no caching was taking place within performance settings and cleared the cache just in case.

AltaVida’s picture

Same here.

Tried debugging with uncompressed version of jquery.cycle. Although JS certainly is not my strong point I'd note that the advance() function from jquery.cycle doesn't get called automatically.

Clicking the next/previous links does fire it correctly, and the slide does then change.

I'll post more if I can figure something out... but like I said I'm no Javascript/JQuery guru.

brmassa’s picture

Guys,

i will check it out and try to fix it as soon i get some free time.

regards,

massa

rimma’s picture

yes, it does not for drupal 6. But if you delete rotate , it works. Just I donot know how much time it rotates.

neofactor’s picture

Looking at my config settings... I noticed colon vs equal signs...

so I changed the above to this and it WORKED!

slideshow: 2 ,rotate:4, blend:1, layout=default, height=808, width=595, fx=scrollRight, dir=|sites/default/files/slideshow/picts/|yes|

I am not sure it they ALL need to be a : instead of an = sign.
The above gives me rotation and at my defined time.

I will play with the rest of the line to see if the : is the correct deliminator.

elvis2’s picture

Confirmation, what neofactor mentioned, #9, does make my slideshow work...

@neofactor, the only parts affected are blend and rotate, meaning they need the ":" instead of "=". See below.

The problem is not the js but function _slideshow_creator_tagconversion(). In slideshow_creator.inc,v 1.1.2.28 this code needs to be changed (me thinks at least):


          // For retro compatibility, we must filter some results
          // to simulate the previous behaviour
          if ($list[trim($attribute[0])]) {
            $slides['#'. trim($attribute[0])] = check_plain(trim($attribute[1]));
          }
          else {
            $value = check_plain(trim($attribute[1]));
            if (is_numeric($value)) {
              $value = (int) $value;
            }
            if (trim($attribute[0]) == 'blend') {
              $slides['js']['speed'] = $slides['js']['blend'];
              unset($slides['js']['blend']);
            }
            elseif (trim($attribute[0]) == 'rotate') {
              $slides['js']['timeout'] = $slides['js']['rotate'];
              unset($slides['js']['rotate']);
            }
            $slides['js'][trim($attribute[0])] = $value;
          }

I am sure other code in this function needs to be changed a bit too. I am confused why though "blend" and "rotate" are being split with ":", I didn't find any code specifically splitting some parts of the tags with ":", except for the "[slideshow:" part of the tag.

Thoughts...

brmassa’s picture

Assigned: Unassigned » brmassa

Guys,

sorry about that. i will fix it soon. Its probably a error i inserted when i start to support both SsC tag and direct JS input (which uses ":").

regards,

massa

neofactor’s picture

Cool.... thanks for adding it to your to-do list. You rock!

duntuk’s picture

@ neofactor :

thanks works perfectly now with colon ( : ) instead of equal ( = )...

** documentation (README.txt) should be updated **

duntuk’s picture

whoops... sorta works... specifying, e.g. rotate:6 doesn't actually take 6 seconds.... it seems the value here is ignored and a default value of around 3 seconds is used...

brmassa’s picture

Al B.,

just note that the colon sign is a bug that i will fix. it was suppose to be only the equal sign.

regards,

massa

jsm174’s picture

You could also use the real jQuery Cycle parameters. Timeout and speed are in millis so multiply by 1000.

[slideshow:2,timeout=5000,speed=2000,random=1,width=90,height=90, layout=none, dir=|sites/default/files/upnp/|]
brmassa’s picture

Status: Active » Fixed

Guys,

blend was created before im starting to use plugin. blend was supposed to create a fadein/fadeout. Now that i use the plugin, you should, as much as possible, start to use "timeout" and "speed", because it will be converted.

the code is now fixed too.

regards,

massa

brmassa’s picture

Guys,

blend was created before im starting to use plugin. blend was supposed to create a fadein/fadeout. Now that i use the plugin, you should, as much as possible, start to use "timeout" and "speed", because it will be converted.

the code is now fixed too.

regards,

massa

gaurav_m’s picture

thanks man ...you rocked a big problem solved ;)

Status: Fixed » Closed (fixed)

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