I recently filed a patch that is now in the source tree to add 24 pixels to the flowplayer height in order to compensate for the height of the flowplayer controls.

I manually updated the flowplayer module to use flowplayer 3.2.5 which now overlays the controls on top of the video image. This negates the need for the extra 24 pixels in height. Rather than have the video source chase this, I decided to make it an admin option so the user can specify whatever adjustment is needed for her version of flowplayer.

Feel free to hack away.

CommentFileSizeAuthor
video_flowplayer_height.patch1.55 KBbkat

Comments

Jorrit’s picture

Jorrit commented
Status: Active » Postponed (maintainer needs more info)

I think the Flowplayer API module should do this. Is this still a problem with the latest versions of the flowplayer module and the video module?

bkat’s picture

bkat commented

The current version of video still hardcodes 24 pixels of height to add to flowplayer.

Jorrit’s picture

Jorrit commented
Status: Postponed (maintainer needs more info) » Fixed

I have added an option to specify the amount of pixels added to the Flowplayer height. Set this to zero or empty to add no pixels. You can find this option at the bottom of the Players tab. The default is 24 pixels because I do not want to break existing installations. Let me know if this is sufficient.

Status: Fixed » Closed (fixed)

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

bkat’s picture

bkat commented

This is basically identical to the patch I submitted and works just fine for me.