Closed (fixed)
Project:
Media: Vimeo
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Jul 2011 at 08:07 UTC
Updated:
30 Sep 2012 at 21:41 UTC
Jump to comment: Most recent file
Comments
Comment #1
kreynen commentedI'll take a look when I get a chance, but you're already deep enough into the code you should take a few minutes and try to submit a patch. If you are using the dev snapshot, you might as well start using git. Makes creating patches much easier...
http://drupal.org/node/707484
Comment #2
sillygwailoTake a look at this comment in #1051090: Revamp file view modes: migrate media_small to teaser, media_large to full, media_preview to preview; deprecate link & original. Does that enable a small preview in the Media: Vimeo module?
Comment #3
drewish commentedIs this still a problem? I hadn't noticed it myself.
Comment #4
yurg commentedre: #3
Yes, it's a problem still: Large and Original works fine. My configuration:
Drupal 7.14
Media 7.x-2.0-unstable5
Media: Vimeo 7.x-1.0-beta5
Views 7.x-3.3
Comment #5
h3rj4n commentedI created the patch which should set some default values of the small style. You should check this url: admin/config/media/file-types/manage/video/display/media_small. If that is also set to 'visible' than everything should work fine.
Comment #6
yurg commented@h3rj4n
Tested and works just fine.
Thank you!
Comment #7
RobW commentedI'm expanding this issue to revising all of the view mode defaults. We actually shouldn't be using small, which will be removed when #1051090: Revamp file view modes: migrate media_small to teaser, media_large to full, media_preview to preview; deprecate link & original goes through. There need to be defaults for teaser and full though.
Comment #8
RobW commentedCopied working code from the YouTube sister issue here: #1320308: Add display settings for "full", "teaser", and "preview" file view mode for media_youtube. Committed to the M:V 7.x-2.x branch: http://drupal.org/node/1229630.