Posted by aaron on February 5, 2008 at 6:33pm
| Project: | jCarousel |
| Version: | 5.x-1.2 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (won't fix) |
Issue Summary
allows for something like
<?php
jcarousel_add('rainbow', drupal_get_path('theme', 'rainbow') . '/jcarousel');
?>| Attachment | Size |
|---|---|
| jcarousel.new-paths.patch | 877 bytes |
Comments
#1
oops, diff'd the wrong direction...
#2
Of course. Why didn't I do that right away?
Thanks! :)
RTBC (if I ignore the one space that violates the coding guidelines)
#3
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.
#4
hmm... that version would probably add the base js more than once. not good.
here's a fixed patch
#5
this has been ready for months. what's the deal?
#6
It seems this patch is out of sync now. Sorry, aaron.
#7
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.