Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

First version of media plugin which supports searching Youtube videos and embedding Youtube and Vimeo videos. #1

Closed
wants to merge 82 commits into from

Conversation

izak
Copy link

@izak izak commented Jun 3, 2013

Please see the original pull request here: wysiwhat#25

agrippa and others added 30 commits February 2, 2013 11:32
…o-plugin branch anyway

Conflicts:
	src/plugins/oerpub/assorted/css/image.css
	src/plugins/oerpub/assorted/lib/assorted-plugin.coffee
	src/plugins/oerpub/assorted/lib/assorted-plugin.js
	src/plugins/oerpub/assorted/lib/image.coffee
	src/plugins/oerpub/assorted/lib/image.js
agrippa and others added 26 commits April 17, 2013 10:11
…input box won't highlight when url is correct
… out pointless radio buttons until search support for other sites is added.
This deals better with xhtml. innerHTML does not deal with it at all well.
@therealmarv
Copy link
Member

Who is gonna review this? This needs testing because a lot of code changed in between.

@kathi-fletcher
Copy link
Member

Don't merge this!

On Fri, Jul 5, 2013 at 4:40 PM, Marvin Reimer notifications@github.comwrote:

Who is gonna review this? This needs testing because a lot of code changed
in between.


Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-20540155
.

Katherine Fletcher, kathi.fletcher@gmail.com
kathi.fletcher@gmail.com
Twitter: kefletcher http://www.twitter.com/kefletcher Blog:
kefletcher.blogspot.com
kathi.fletcher@gmail.com

@philschatz
Copy link
Member

It also cannot be automatically merged anymore ; (

@TomWoodward TomWoodward closed this Oct 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants