Gotta be Great,.. to bad :(

CommentFileSizeAuthor
#1 viewscarousel.tar_.gz45.61 KBmfer

Comments

mfer’s picture

Assigned: Unassigned » mfer
Status: Active » Needs review
StatusFileSize
new45.61 KB

Here is my first pass at a drupal 6 views 2 version.

recidive’s picture

Hello, code looks good, though I have not tested it yet.

For version 6, I'd like to see it depend on jCarousel module. There is a patch being developed to port it to Drupal6.

recidive’s picture

Version: » 5.x-1.x-dev
Status: Needs review » Needs work

It's missing Ajax loading of items. It seems it should be easier in Views 2.

I'm doing some CVS gardening, and will commit you patch soon.

Thanks.

mfer’s picture

Yes, it is missing AJAX loading. That's not something I needed at the moment so I didn't write it.

I imagine it wouldn't be that hard to write.

drewish’s picture

Version: 5.x-1.x-dev » 6.x-1.x-dev
Category: feature » task
Status: Needs work » Active

since there's a updated code base how about an official release?

drewish’s picture

pretty please? this is the only module on my site without a supported release. i'm happy to help out with it in the queue.

robloach’s picture

There's a patch to make it use the jCarousel module here: #358213: Drupal 6 jCarousel. The benefit of using the jCarousel module instead of its own implementation are:

  1. Use packed JavaScript instead of uncompressed when JavaScript optimization is on
  2. Use of behaviors so AJAX loading may work
  3. Removes lots of code in Views Carousel
  4. Ability in a later patch to support the jCarousel JavaScript callbacks through Views Carousel (jCarousel has that support)
mfer’s picture

Version: 6.x-1.x-dev » 6.x-2.x-dev

I'm working on a D6-1.x release. Then I'll package up the D6-2.x release. Then, i'm going to work on some cool stuff for the D6-3.x release I really want to see.

quicksketch’s picture

Status: Active » Closed (fixed)

There's been a D6 version for a while now. See also #983844: Deprecate this module now that jCarousel includes it