Forums | Mahara Community
Added Vimeo as External Video type
17 March 2011, 12:28
I added Vimeo as an embeddable type in the External Video block. It also has an option to retrieve default dimensions using the Vimeo API. (I looked into this for YouTube too, but no default size is reported.)
Please test it if you get a chance and report any issues.
Make sure to paste in the video's URL, not the embed code.
EDIT: updated download file.
17 March 2011, 20:42
Thanks for sharing that code. I have cleaned it up a bit and put it up on a git branch here to make it easier to review:
All I did was fix up some whitespace issues and split the changes into 3 different logical commits, which allowed me to cherry-pick this one onto master right away:
I haven't had time yet to review your patches fully, but I was wondering about this one:
I don't think we want to change the message there because pasting embed code into the URL box does work for all services except Vimeo. In fact, it might be easy to do it for Vimeo too.
By the way, if you want to get started with git and have any questions, let me know :)
20 March 2011, 8:25
Thanks for doing that Francois! I'm using svn here, but it would be useful to get up to speed on Git. I will take a look at Gitorious and no doubt ask you a few questions.
I have various other customisations I can contribute which may or may not be useful for others.
The problem with embed codes is that the default embed code for youtube doesn't work now - they have a new iframe code format which is different from the old object/embed one. Looking at it just now, it looks like it would be quite easy to adapt the External Video block to work with it.
Vimeo are also now using iframe code but that actually works ok as the required id can still be parsed out of it using the same regular expression.
So we should fix the Youtube embedding rather than change the help text.
27 April 2011, 10:59
this is great. How do I install this plugin/ extension? Thanks.