Closed (fixed)
Project:
Views Slideshow
Version:
7.x-3.x-dev
Component:
Cycle
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Dec 2011 at 22:32 UTC
Updated:
11 Jul 2013 at 04:51 UTC
Jump to comment: Most recent file
Comments
Comment #1
marty2081 commentedI work on the same project as Baris. If we disable the option " Start On Last Slide Viewed" the pager of the first slide correctly gets a class "active" on page load. So the issue must be somewhere in there.
Comment #2
pandabrand commentedI am having this issue as well but it's not related to the "Start On Last Slide Viewed" option. For me it's tied to using advanced options when I add more than one advanced option,(I am using the before and after option) the active class stops appearing. I don't know how or if this is tied to this in any way but on the JS where the option are displayed it looks like this.
when i only have one option the advanced_option_choices and advanced_options_entry are both empty and active class works when I add a second,(doesn't matter if it's before or after) it will end up in those two and the active class goes away. Maybe it's correlation but it happens consistently for me. Anybody know what's happening there?
Comment #3
pandabrand commentedActually it seems to be with the 'before' portion of advanced options, it works fine with 'after' but not before and i can't see what is wrong with it and it's really screwing up my site, I kind of need this to work. anyone?
Comment #4
mustanggb commentedSuper annoying bug, I'd say this is a major priority
Comment #5
mustanggb commentedClarifying issue
Comment #6
Geijutsuka commentedI'm having the same issue at http://thinktank.arizona.edu/testing-page-2 (will try #4 patch asap)
Except in my case, the .active class doesn't get applied when I tell the slideshow to start with a random slide (done by showing the "View Transition Advanced Options" and checking the Random box or adding "random = true" in the "jQuery Cycle Custom Options" section).
Comment #7
Geijutsuka commentedI applied the patch to a new install of 7.x-3.x-dev and the latest D7. It still looks as if the active class isn't getting applied to the correct initial slide.
Comment #8
mustanggb commentedCan you give the steps to reproduce?
Comment #9
Geijutsuka commentedI followed this tutorial to set up the show with an image pager:
http://www.brightwebsitedesign.com/how-to-install-views-slideshow-module...
The only difference is that I checked the Random box under View Transition Advanced Options.
Comment #10
mustanggb commentedHere is an updated version that fixes the pager active class for the first slide when random order is in use.
I have a feeling it needs jquery.cycle.all.js version 2.9999 or later, however I used version 2.9999.5 for my testing.
Comment #11
Geijutsuka commentedThe patch works like a charm! Just like you said though, use jQuery Cycle 2.9999 or later. Mine is working on jQuery 2.9999.5.
Comment #12
mustanggb commentedHave you committed this to the git repository?
If not, then you should instead use RTBC
Comment #13
Geijutsuka commentedSorry about that akamustang. Thank you for clarifying the status (I'm not quite sure how to commit to the git repository...).
Comment #14
mustanggb commentedThat is for a module maintainer to do.
Comment #15
TimG1 commentedAnother confirmation. #10 worked for me. Patched on 7.x-3.x-dev (2012-Apr-25)
-Tim
Comment #16
wangqizhong commentedThanks a lot to BarisW, akamustang and everyone else for the great work on the patches, testing, reviews and reporting.
It's more time this got committed and I went ahead and had this rolled at:
7.x-3.x: 5528a45
Marking this as fixed.
Please let me know if you would have any other questions, comments, issues or concerns on any of these changes, I would be glad to provide more information.
Thanks again to everyone for the help and great work on this issue.
Cheers!
Comment #17
wangqizhong commented