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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webadpro’s picture

Fixed some proper spacing.

wbobeirne’s picture

Fixed the following line:

$element[$delta]['#prefix'] = $description_html & $element[$delta]['#prefix'];

& 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.

jec006’s picture

Status: Needs review » Fixed

I 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...

webadpro’s picture

Sounds Good to me. Glad this got committed :)

Status: Fixed » Closed (fixed)

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