From http://drupal.org/node/324777#comment-3307046

_____________________replace this____________________
// Add the skin CSS if it hasn't been added yet.
if (!isset($jcarousel_added['skin'][$skin])) {
if ($skin == 'tango' || $skin == 'ie7') {
drupal_add_css("$jcarousel/jcarousel/skins/$skin/skin.css");
}
elseif (!empty($skin_path)) {
drupal_add_css($skin_path, 'theme');
}
$jcarousel_added['skin'][$skin] = TRUE;
}
_____________________with this____________________
// Add the skin CSS if it hasn't been added yet.
if (!isset($jcarousel_added['skin'][$skin])) {
if ($skin == 'tango' || $skin == 'ie7') {
drupal_add_css("$jcarousel/jcarousel/skins/$skin/skin.css");
}
elseif (!empty($skin_path)) {
drupal_add_css($skin_path, 'theme');
}
else {
drupal_add_css("$jcarousel/jcarousel/skins/$skin/skin.css");
}
$jcarousel_added['skin'][$skin] = TRUE;
}

CommentFileSizeAuthor
#6 viewscarousel.module.980346-6.patch553 bytesjefftrnr

Comments

AdrianB’s picture

First, shouldn't this be in the jcarousel queue?

Second, I've posted a patch for jcarousel, it's not the same change as above, but I think it solves the same problem.

Unfortunately there has been no response from jcarousel maintainers so far.

quicksketch’s picture

This has been fixed in jCarousel 2.0: See #426774: jCarousel Skin Registry (hook_jcarousel_skins).

1kenthomas’s picture

Assigned: Unassigned » 1kenthomas

^^ We often have issues with matters which depend on coordinating two modules or modifying one module to accommodate the needs of another. Just wanted to get this one on the board. Self-assigning and will follow up in the jcarousel queue.

kbell’s picture

What's the status on this issue? Any progress?
Thanks,
K

quicksketch’s picture

Views Carousel is abandoned in preference of the jCarousel 2.x module. There won't ever be any further progress on this in the Views Carousel issue queue or module code.

jefftrnr’s picture

StatusFileSize
new553 bytes

here's a patch which adds the custom skin option and points it to the default theme/skins/custom/ folder