allows for something like

jcarousel_add('rainbow', drupal_get_path('theme', 'rainbow') . '/jcarousel');
CommentFileSizeAuthor
#4 jcarousel.jq-with-paths-once.patch859 bytesaaron
#3 jcarousel.new-paths-static.patch861 bytesaaron
#1 jcarousel.new-paths2.patch877 bytesaaron
jcarousel.new-paths.patch877 bytesaaron

Comments

aaron’s picture

aaron commented
StatusFileSize
new jcarousel.new-paths2.patch877 bytes

oops, diff'd the wrong direction...

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Of course. Why didn't I do that right away?

Thanks! :)

RTBC (if I ignore the one space that violates the coding guidelines)

aaron’s picture

aaron commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new jcarousel.new-paths-static.patch861 bytes

here's another version that will also allow multiple skins to be plastered on a page, if one were so inclined. the version in cvs only allows a single skin to be applied.

aaron’s picture

aaron commented
StatusFileSize
new jcarousel.jq-with-paths-once.patch859 bytes

hmm... that version would probably add the base js more than once. not good.

here's a fixed patch

aaron’s picture

aaron commented
Status: Needs review » Reviewed & tested by the community

this has been ready for months. what's the deal?

robloach’s picture

robloach
he/him
commented
Status: Reviewed & tested by the community » Needs work

It seems this patch is out of sync now. Sorry, aaron.

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs work » Closed (won't fix)

Rather than using custom skin paths, we now have a skin registry in 2.x (see #426774: jCarousel Skin Registry (hook_jcarousel_skins)), but if you prefer you could simply drupal_add_css() your own skin file manually and then use a matching skin name when you call jcarousel_add().

Support has ended for the 5.x version, so I'm moving this to won't fix.