This is a great module and so much more simple then the media module for what i need. Just wondering however of how much trouble it would be to provide another size option to display for teasers and provide thumbnail support? Reason being i would like to create a view consisting of smaller thumbnails that link to the nodes. I know the media module allows for this but so much more complex to set up compared to your module.

Comments

jec006’s picture

Assigned: Unassigned » jec006

I think thats a great idea - will add to list of features for next release.

apmsooner’s picture

Thats great news. Thanks :)

JoshW’s picture

I dont know if it would be in your code or not but could you make it so the thumbnail image could open the video in a lightbox or something similar?

jec006’s picture

JoshW - I think that would be something you'd do in the theme or in an additional module.

jec006’s picture

Hey - I added the teaser dimensions support - I didn't add the thumbnail stuff (yet). - http://drupal.org/commitlog/commit/25834/3a13685501d2d3da693a82bdc0db106...

EndEd’s picture

+1 for the thumbnail support. Fantastic module btw

protools’s picture

sub, cool if to be more then one formater for Views for thumbnail and configurable preset

protools’s picture

how about integrate with imagCashe ? use views for catalog of video, and one page with 5 video preview load vary long time. Any solutions ?

Canadaka’s picture

+1 for the thumbnail support

JoshW’s picture

Agreed Imagecache would be nice!

enricpera’s picture

subscribe
cool if to be more then one formater for Views for thumbnail and configurable preset

jec006’s picture

in Re: Protools. What this module does is simply embed the video with an iframe - there is no caching that can/should be done with that content and imagecache (now image styles) does not handle this sorta thing.

We can leverage image styles to resize thumbnails, however - the load time is most likely caused by services (vimeo and youtube) not allowing you to load videos in parallel from the same location. The server load should be minimal (since its an iframe, which is handled by the browser). The load time is probably occurring at the browser and service level. Using thumbnails for lists would definitely assist with this, but i won't be able to help the load times for the videos themselves

protools’s picture

Canadaka’s picture

I added a super simple hack to my site that will display a thumbnail image instead of the teaser iframe.

video_embed_field.module line 354

if (!$teaser) return $template_start . $url_temp . $id . $template_end;
else {			
	$thumbnail = '<img src="http://img.youtube.com/vi/'.$id.'/0.jpg" width="'.$width.'" height="'.$height.'" />';
	return $thumbnail;
} //teaser end

But the image should use "image styles" to resize ans store a thumbnail locally I think. The code here could be used to create an image style from the external URL http://www.pepperpotdesign.co.uk/blog/rendering-drupal-7-image-styles-ou...
But the filename would have to be stored in a field.

dgastudio’s picture

+1 for video thumb!

can we use this somehow?

http://drupal.org/project/imagecache_external

<?php
  print theme('imagecache_external', array('path' => 'http://larsjung.de/h5ai/sample/screenshots/h5ai-v0.7-icons.png', 'style_name'=> 'thumbnail'));
?>
Canadaka’s picture

good find, I installed their dev and it works.
return theme('imagecache_external', array('path' => 'http://img.youtube.com/vi/'.$id.'/0.jpg', 'style_name'=> 'thumbnail'));
I guess the style_name should be a variable configurable in the video embed settings page, or maybe the display widget.

Once the first imagecache is created, it doesn't re-create it each time right? Does it check it if exists and if it does use the local one from now on?

jec006’s picture

Hey Guys,

My thought is that the best way to do this and be able to handle multiple movie providers is to download the thumbnail and save it to the file system. We will then be able to use image styles (imagecache) to improve performance and size it the way we want. We can then delete the file whenever a video is removed (or changed).

One of the guys that I work with has made some process on producing a patch towards this end - so hopefully we will have a new release soon with this functionality.

jec006’s picture

Status: Active » Fixed

I'm marking this item as fixed because the teaser was added - please see other issues related to thumbnail support ect.

Thumbnail / image support has been added in addition in the latest series of commits and should be in alpha5

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.