When a carousel is configured to wrap continuously, I don't see a use case in which hiding the buttons is actually desirable. The point of wrapping is precisely to have a fixed length slideshow whatever the number of slides.

CommentFileSizeAuthor
#1 1488890-jcarousel-hide-buttons-continuous.patch683 bytesdamien tournoud

Comments

damien tournoud’s picture

damien tournoud commented
Status: Active » Needs review
StatusFileSize
new 1488890-jcarousel-hide-buttons-continuous.patch683 bytes

Here is a suggested patch. We also should use the buttons() API method instead of messing up with jcarousel classes directly.

Macronomicus’s picture

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

Sweet, that totally fixed my issue of having a 100% width circular carousel in a responsive template design, the prev/next buttons would not load on wide screens if even a tiny sliver of the last carousel item was visible.

This patch seems a logical fit being it ties in with the circular setting, Cheers!

markpavlitski’s picture

markpavlitski commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed - http://drupalcode.org/project/jcarousel.git/commit/cdfdc1d

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.