Download & Extend
Views Slideshow » Issues

Provide CSS classes to indicate the status of the pause button

Posted by pfrenssen on October 26, 2011 at 8:57pm
Project:Views Slideshow
Version:7.x-3.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

It would be nice for theming if there were CSS classes on the pause button that indicate the current active state (playing or paused). This would be handy if the buttons are themed with play/pause images.

Login or register to post comments

Comments

#1

Posted by pfrenssen on October 26, 2011 at 9:07pm
Title:Provide CSS classes for pause button that toggle on click» Provide CSS classes to indicate the status of the pause button

This simple patch adds the class 'playing' or 'paused' to the button. My javascript-fu is not exactly stellar, so I'd be glad if someone could give it a look.

AttachmentSize
1322780-1-views_slideshow-pause_button_classes.patch 1.57 KB

#2

Posted by Pomliane on December 9, 2011 at 4:48am
Status:active» needs review

#1353994: Add "paused" class to controls for theming marked as duplicate of this issue.

#3

Posted by ckng on December 29, 2011 at 9:25am
Status:needs review» reviewed & tested by the community

Tested and the code looks fine.

#4

Posted by scottrouse on January 24, 2012 at 8:51pm

I can also verify that this works. I'd love to see this in dev.

#5

Posted by ckrina on February 1, 2012 at 10:23am

Worked for me too. Thanks.

#6

Posted by robwithhair on February 9, 2012 at 4:54pm

Patch works for me. Would be good to see this in the release.

#7

Posted by cristinawithout on April 11, 2012 at 10:45pm
Status:reviewed & tested by the community» closed (fixed)

Appears there's a fix for this in the current dev using classes "views-slideshow-controls-text-status-pause" and "views-slideshow-controls-text-status-play".

#8

Posted by pfrenssen on May 6, 2012 at 9:52pm

This has been fixed in Commit 3de5104d9: Add a class to the pause/play control to know what the status is.