The Video.js takes on a range of display needs for the Video module, but some users might need reference images as placeholders. If this module is to provide that, the default image in sites/all/modules/video/wysiwyg_plugins/video/images/video-place-holder.gif is not a reference to any known aspect ratio, so it would represent a distortion of someone's source.

For for what it's worth, I'm attaching some few alternate images that are based on 240, HD, NTSC, and PAL. (Note that there is only one reference for True 240 because the other pixel widths don't produce a whole pixel derivative.)

CommentFileSizeAuthor
video-place-holder_640x512_PAL.png5.05 KBmichael.k
video-place-holder_640x480_NTSC.png5.11 KBmichael.k
video-place-holder_640x360_HD.png4.89 KBmichael.k
video-place-holder_480x384_PAL.png4.72 KBmichael.k
video-place-holder_480x360_NTSC.png4.68 KBmichael.k
video-place-holder_480x270_HD.png4.56 KBmichael.k
video-place-holder_480x200_T240.png4.45 KBmichael.k
video-place-holder_400x320_PAL.png4.51 KBmichael.k
video-place-holder_400x300_NTSC.png4.49 KBmichael.k
video-place-holder_400x225_HD.png4.37 KBmichael.k
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jorrit’s picture

Jorrit CreditAttribution: Jorrit commented
Component: Miscellaneous » WYSIWYG integration
Status: Active » Postponed

Thanks for your input. Work on the WYSIWYG integration part keeps getting postponed because more important issues come up. When I start working on the WYSIWYG part I'll put this in too.

brycefisherfleig’s picture

brycefisherfleig CreditAttribution: brycefisherfleig commented
Issue summary: View changes

Looks like this would be easy issue to tackle in the near future, and it appears to be still be relevant. I'll keep this issue as "postponed" since there are still several high priority issues in front of this.

I'd be happy to review this as patch right away however.