Support for configurable height and width
javierreartes - November 19, 2009 - 02:24
| Project: | Ooyala |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
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
| Attachment | Size |
|---|---|
| ooyala-configurable_size.patch | 1.96 KB |
#1
thanks! I like this, I'll test it later
#2
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.
#3
Automatically closed -- issue fixed for 2 weeks with no activity.