Hi, I made a patch for modifying ooyala_player, theme_ooyala_player and template_preprocess_ooyala_thickbox_player in order to be able to specify custom width and height other than what's globally configured.

It would be nice to have configurable video sizes form administrative interface, meanwhile this will help to embed custom sized players.

Also there was a small issue, in theme_ooyala_player was calling to variable_Get capitalized which is wrong.

Thanks!
Javier

CommentFileSizeAuthor
#2 ooyala_height_width.patch2.12 KBquicksketch
ooyala-configurable_size.patch1.96 KBjavi-er

Comments

heronog’s picture

heronog commented

thanks! I like this, I'll test it later

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Fixed
StatusFileSize
new ooyala_height_width.patch2.12 KB

This seems pretty much like an API cleanup. I've committed this patch with a few changes, like code style formatting and variable names. Otherwise looked good! Committed.

Status: Fixed » Closed (fixed)

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