Download & Extend
Video.js (HTML5 Video Player) » Issues

Setting video width and height

Posted by Mark Vincent Verallo on September 6, 2011 at 2:04pm
Project:Video.js (HTML5 Video Player)
Version:7.x-2.2
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Is there a way to set video width and height like the one in the videojs embed builder?

Login or register to post comments

Comments

#1

Posted by jasondecamp on January 4, 2012 at 4:10pm

subscribe

I'm looking for a solution that works on a per video basis, not a global height and width setting (which is located in the module configuration area).

#2

Posted by Jorrit on February 4, 2012 at 7:30pm
Version:7.x-1.3» 7.x-1.x-dev
Category:support request» feature request
Status:active» postponed

Maybe in a later version.

#3

Posted by mogtofu33 on August 24, 2012 at 5:10pm
Version:7.x-1.x-dev» 7.x-2.2
Component:Miscellaneous» Code
Status:postponed» patch (to be ported)

Hi,

Thanks for this module, i need to set different width and height by field for different content type, so here is a patch to add this feature.

Regards.
Mog

AttachmentSizeStatusTest resultOperations
videojs-add-field-size-settings-1270884.patch2.56 KBIgnored: Check issue status.NoneNone

#4

Posted by michael.k on August 24, 2012 at 7:10pm

@mogtofu33 you appear to be using 2.40 aspect ratio in your patch. I'm curious about your source format. Are you cropping from HD and then outputting a new file for upload, or are you transcoding directly from your source files?

#5

Posted by mogtofu33 on August 24, 2012 at 7:21pm

I just use default values already set by videojs in admin settings.
Currently i am not using this module in production, i will probably have some user feedback when website will be launched...

#6

Posted by michael.k on August 24, 2012 at 7:42pm

I see, thanks for the clarification.

@Jorrit then I'd suggest using a default value of 360 (instead of 264) for 'videojs_height' in videojs.theme.inc and videojs.admin.inc.

#7

Posted by Jorrit on August 25, 2012 at 10:12am
Status:patch (to be ported)» fixed

I have changed the default resolution and committed this patch. I have changed the patch such that it is possible to leave the field-specific form empty and get the default values. I also added some form validation. I hope you like the changes.

#8

Posted by System Message on September 8, 2012 at 10:21am
Status:fixed» closed (fixed)

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