janjiss / conred

Embed your videos with ease

Conred

You often have to embed video from YouTube and Vimeo to your web site? Wouldn't it be nice if we had a gem that could make video embedding from URL's easy as pie? BOOM. Here you go.

Installation

Add this line to your application's Gemfile:

gem 'conred'

And then execute:

$ bundle

Or install it yourself as:

$ gem install conred

Usage

Iframe generator for YouTube and Vimeo videos:

c = Conred::Video.new(
	video_url: "http://www.youtube.com/watch?v=tNtW9pGFPTA&feature=plcp", 
	width: 285, 
	height: 185,
	error_message: "Video url is invalid"
)

Then you can get your ready embed code like this (Conred will recognize video provider by itself):

# In your Rails view you will need to append .html_safe
c.code.html_safe

You can also check if it is YouTube or Vimeo video like this:

c.youtube_video? #=> true
c.vimeo_video? #=> false

Or if it exists:

c.exist? # => true

You can also obtain the view count of the video like this:

c.view_count

Contributing

1. Fork it
2. Write your feature
3. Create tests for it (Important!)
4. Create new Pull Request
5. Be happy

Thank yous

Please stand up from your chair and applaud to these guys: https://github.com/janjiss/conred/graphs/contributors

conred's Issues

Adding custom classes and id to video iframe

Right now we have no possibility of adding class and id for video. This functionality should be added. I am thinking of a way of adding data atributes to. If somone has solutions for data attribute adding, you are welcome to help.

Add support for facebook videos

Add support for facebook videos

Change strings to haml partials

Right now iframe strings are hardcoded
<<-eos
<iframe id='vimeo_video' src='http://player.vimeo.com/video/#{vimeo_id}' width='#{@width}' height='#{@height}' frameborder='0' webkitAllowFullScreen mozallowfullscreen allowFullScreen>
</iframe>
eos

We should change this to haml, so it is better tested and it will look much cleaner

Refactor tests

Fix tests with lazy load. Right now for every video I use instance, thats not cool

Need to determine, if video is present or not

Right now, if youtube url format is ok, but it is a fake one, it shows iframe without video content. Can we determine somehow, is video present in youtube or vimeo?

Need to fix youtube links, that are coming from related video watching

Let's say we have link like this:
http://www.youtube.com/watch?NR=1&feature=endscreen&v=CJ_ovGKN7_4
It does not render video, because regex is not recognizing video url.

Gem relies on HAML

Should add HAML as a dependency. You get an error if you do not have it installed.

Changing the interface of Video class

Maybe we should consider passing hash with args for creating instance of Conred::Video, instead of simple arguments? It would be more clear (at least for me) if it looked like:

 Conred::Video(
    :url => video_url, :error_message => "Video url is not valid",
    :width => 350, :height => 350
 )

Would that be more prefferable, than current interface, where we just pass anonymous args?

The exist? method does not work for youtube videos

Hi!

The exist? method does not work properly for youtube videos. It always returns false, no matter if the link is valid. This is because gdata.youtube.com/feeds/api/videos/ is no longer available. I think you need to use YouTube Data API V3 for checking YT videos.

The name of the method to check whether video exists is 'c.exist?' rather than 'c.exists?'

The README or the code needs to be updated accordingly.