Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Hi,
I needed to have the description on top of the video rather than on the bottom so i have made some changes to the code.
Please review the patch, and if possible commit.
Thanks
Comment | File | Size | Author |
---|---|---|---|
#2 | video_embed_field-description-position-1679452-2.patch | 3.02 KB | wbobeirne |
#1 | description_positionv2.patch | 2.97 KB | webadpro |
description_position.patch | 2.88 KB | webadpro | |
Comments
Comment #1
webadpro CreditAttribution: webadpro commentedFixed some proper spacing.
Comment #2
wbobeirne CreditAttribution: wbobeirne commentedFixed the following line:
& is the pass by reference operator, . is the add string operator. Also made some of the settings arrays look nicer, and simplified the #prefix/#suffix stuff for the description position.
Comment #3
jec006 CreditAttribution: jec006 commentedI sort of reproduced this patch here: http://drupal.org/commitlog/commit/25834/e82c984ac8dff0c8ca4aa16621cafe8...
With some various cleanup and bug fixes. I also committed some additional cleanup and fixed a syntax error i introduced here: http://drupal.org/commitlog/commit/25834/9882486224a0ab97f0841560f41c819...
Comment #4
webadpro CreditAttribution: webadpro commentedSounds Good to me. Glad this got committed :)