Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#1 | 1322780-1-views_slideshow-pause_button_classes.patch | 1.57 KB | pfrenssen |
Comments
Comment #1
pfrenssenThis 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.
Comment #2
Pomliane CreditAttribution: Pomliane commented#1353994: Add "paused" class to controls for theming marked as duplicate of this issue.
Comment #3
ckngTested and the code looks fine.
Comment #4
scottrouse CreditAttribution: scottrouse commentedI can also verify that this works. I'd love to see this in dev.
Comment #5
ckrinaWorked for me too. Thanks.
Comment #6
robwithhair CreditAttribution: robwithhair commentedPatch works for me. Would be good to see this in the release.
Comment #7
cwithout CreditAttribution: cwithout commentedAppears 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".
Comment #8
pfrenssenThis has been fixed in Commit 3de5104d9: Add a class to the pause/play control to know what the status is.
Comment #9
Jance CreditAttribution: Jance commentedI see this is working on the Dev side, but I was wondering if there was a possibility that this code could get pushed out to the production trac.