Active breakout teaser CSS class in views_slideshow_thumbnailhover contrib module

jcmarco - October 5, 2009 - 09:05
Project:Views Slideshow
Version:6.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

In order to help style the active breakout teaser, the javascript should add some class to identify the active teaser.

Right now there is a .views_slideshow_thumbnailhover_active_teaser but it doesn't cycle, it is only applied to the first thumbnail breakout teaser,
and in the jquery cycle plugin there is an .activeSlide class but it is not shown in the view.

#1

redndahead - October 10, 2009 - 20:22
Status:active» needs review

Here is a patch to fix the issue. Tell me if it works for you?

AttachmentSize
activeslide_thumbnailhover-596116-1.patch 1.55 KB

#2

jcmarco - October 13, 2009 - 15:48
Status:needs review» reviewed & tested by the community

Tested and working great.

But now I have the doubt about the need of still using in views_slideshow_thumbnailhover.theme.inc
the class: views_slideshow_thumbnailhover_active_teaser for the first element in function
theme_views_slideshow_thumbnailhover_breakout_teaser($item, $id, $count)

Would it be still needed? with just the activeSlide would be enough

#3

isaac77 - October 22, 2009 - 19:42

Patch works great for me, thanks! Was banging my ahead against a wall trying to figure out what i was doing wrong until I came across this post.

The demo site (http://drupalhub.org/videos) shows the correct behavior, which made me think it was a configuration rather than code issue. But I guess the demo is using this patch as well?

Thanks for a _great_ module!

#4

redndahead - November 16, 2009 - 04:57
Status:reviewed & tested by the community» fixed

Patch committed.

#5

System Message - November 30, 2009 - 05:00
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.