How about moving the global Slide Duration setting out of the admin pages and into a new field on the spotlight FPP itself, so that each spotlight can get its own setting for slide duration?

Patch incoming.

CommentFileSizeAuthor
#1 panopoly-widgets-2330667-1-spotlight-duration.patch9.85 KBmcrittenden

Comments

mcrittenden’s picture

mcrittenden commented
Status: Active » Needs review
StatusFileSize
new panopoly-widgets-2330667-1-spotlight-duration.patch9.85 KB
mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Status: Needs review » Reviewed & tested by the community

Works as suggested. Works out really well when you don't provide access to the Panopoly settings page.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Assigned: Unassigned » dsnopek

@mcrittenden: Thanks! This looks mostly good, except that you have to revert the panopoly_widgets Feature in order for the duration field to appear and start working - this needs a hook_update() function. I'll add one and do a little more testing, but will probably commit this in a moment.

We should also add testing of the duration field to our Behat tests. I don't think we should bother with actually watching the rotation, but we should verify that the field is there and settable: #2334043: Check that Spotlight 'Slide Duration' field is present and settable in Behat tests

  • dsnopek committed 4a22ed2 on 7.x-1.x 
    Update Panopoly Widgets for #2330667 by mcrittenden: Added per-spotlight...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Reviewed & tested by the community » Fixed

Commited!

Status: Fixed » Closed (fixed)

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