Active class not being applied?

@inteja, did you try my patch in #1450846: jCarousel is never initialized properly with Omega theme? I have encountered several javascript issues with the current state of the module.

I'm running into the same problem. It looks like the class views_slideshow_active_pager_field_item gets stuck on the first item in the pager and is not being removed and rewritten to the next active pager item. This pager widget is a great module, specially with its ease of use with Views Slideshow, but I really need to be able to style the active pager item. I'm not a Javascript guy but have been trying to backwards engineer the views_slideshow_jcarousel.theme.inc file to fix the function _views_slideshow_jcarousel_preprocess_pager_item with no luck so far. Anyone else with thoughts or ideas on getting the active class to change with the currently active pager item?

Anyone make headway on this one? I'm experiencing the same issue as #4.

Found a solution for #4. If you go into the settings for your Slideshow in Views, under "jQuery Cycle Custom Options" select "pager" and input ul.jcarousel-list (no quotes or apostrophes) or whatever the selector is for your pager. This activated the activeSlide class for the active list item in the pager.

Hi rmd-14,

I can't find how did you do that, should I download the json and that stuff to activate the custom options? At the bottom when a select pager there is not input field to put that selector.

Regards,

Julian Mancera

Besides #1450846: jCarousel is never initialized properly with Omega theme, there are a couple other significant errors in the views_slideshow_jcarousel.js file, including the missing active class on this issue.

The third issue is that somehow a section of the code has been duplicated.

Here's a patch that a) removes the duplicate code, and b) fixes the active class issue.

@jwilson3 I am unable to apply this patch

#6 does add an activeslide class to the li in my case

Update: #6 DID work for me. (I hadn't clicked the "update" link previously.) I was able to apply a style like:

ul.jcarousel-list .activeSlide {
  opacity: 0.5;
  filter: alpha(opacity=50);
}

to see that it was working.

I tried the patch from #8 and it didn't solve the active pager issue for me. Note that you have to edit the patch directory structure to apply to your own site unless you are using the exact directory structure in the patch:

www/sites/all/modules/contrib/views_slideshow_jcarousel/

if you are applying the patch from within the:

views_slideshow_jcarousel

directory then you can delete all occurrences of:

www/sites/all/modules/contrib/views_slideshow_jcarousel/

from the patch file and then apply with:

patch -p1 < [patchfile]

#6 worked great for me. Thanks!

I did confirm that both approaches (#6 and #8) work, but the patch in comment #8 made by jwilson3 solves also issues related to the multiple triggering of duplicated funcions in the js, maybe the patch can be better rerolled, but for now it's enough.

thanks jwilson3

Just rerolling #8 to conform with drupal patching guidelines. This can now be applied with patch -p1 < [patchfile] directly.

Works OK, thanks. Applied without problem and active item is now highlighted.

patch #14 works for me

Thanks for catching this duplicated code, and adding an active class! I've committed this patch. Its always nice to see a negative lines of code fix :) keep it up!