On Vimeo, we can specify if we want display portrait, title and byline informations on vimeo player.
i make a patch to add this to your module (on the next comment). Configuration is same as autoplay

Comments

goz’s picture

Status: Active » Needs review
StatusFileSize
new14.05 KB

Here is the patch to add options

jhubley’s picture

Status: Needs review » Reviewed & tested by the community

This patch works. Thanks, it was helpful.

rokr’s picture

Although i had some problems applying the patch it now works for me. It would be great to see this committed and maybe more options which i remember from original emfield/vimeo module (color, maybe player controls).

Thanks for the patch, rokr

KoCo’s picture

Oké first of all, former patch (#1) works and if it can be committed, go ahead.

Attached patch also adds the option for color choosing and uses Drupal's color picker.
This probably will not work everywhere and always, so please test.

Also coding isn't actually how it should be, but it's a start.

K

p.s. I'm currently working on a project without any version control, so I adapted former patch. Please forgive me ;-)

KoCo’s picture

Sorry added the wrong file

drewish’s picture

Seems like there's some overlap with #1443092: Add more video display settings, switch to universal player.. I think both add a lot of code for something that should be pretty simple. I think we should find a simpler way to handle the settings. Those accessor functions just seem noisy. Why not simply expose an array of settings?

drewish’s picture

Status: Reviewed & tested by the community » Needs work

i can't get this patch to apply at all. It keeps reporting it as "patch: **** malformed patch at line 30" i tried deleting the first few chunks to see if there was something specific to those but it just failed at later line numbers.

drewish’s picture

Status: Needs work » Closed (duplicate)

Just going to use the other patch as a starting point since it applies.