I would like the ability to choose, per field instance/display mode, if the video should be played on a loop.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bxtaylor’s picture

Status: Active » Needs review
FileSize
3.59 KB

And for good measure, I'll add a patch to start us off.

dasginganinja’s picture

I just tested the module out and it works as intended. The code looks fine as well and all additions are well within the d.o coding standards.

dasginganinja’s picture

Status: Needs review » Reviewed & tested by the community

  • guschilds committed 5759176 on 7.x-1.x authored by bxtaylor
    Issue #2444699 by bxtaylor, dasginganinja: Added a loop option to the...
  • guschilds committed cf44d2c on 7.x-1.x
    Issue #2444699: Added a loop option to the display field formatter for...
guschilds’s picture

Version: 7.x-1.x-dev » 8.x-1.x-dev
Status: Reviewed & tested by the community » Needs work

Thanks for the patch. I have committed it to the 7.x-1.x branch and it is part of the 7.x-1.6 release.

In an effort to keep features synced between the 7.x-1.x and 8.x-1.x branches, I've marked this as "Needs work" and changed the version to "8.x-1.x-dev". It can be closed once this feature has been added there as well.

This is a good one for someone with no Drupal 8 experience to start getting their hands dirty!

  • guschilds committed 6687f33 on 8.x-1.x
    Issue #2444699: Added a loop option to the display field formatter.
    
guschilds’s picture

Status: Needs work » Fixed

I've ported the feature to the 8.x-1.x branch. Thanks again for the patch and testing!

Status: Fixed » Closed (fixed)

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