Using the colorbox integration, I get both vertical and horizontal scrollbars in Webkit and IE. On some computers, I am getting them on FF as well. Here is an example of what I am talking about:
http://fouronefourproductions.com/videotest/
And views: http://fouronefourproductions.com/videotest/video
I have tried several different colorbox themes, all kinds of different sizes in Video Embed Field > Settings. Always with the same result. Scrollbars inside Colorbox on everything but Firefox. I've tested on Mac and PC. if the scrollbars weren't there, the video would be slightly cut off.
I get the following error irregularly:
Strict warning: Only variables should be passed by reference in template_preprocess_video_embed_field_embed_code() (line 355 of /home8/anthoob7/public_html/videotest/sites/all/modules/video_embed_field/video_embed_field.module).
Any help in eliminating these scrollbars would be appreciated. I realize I can simple set overflow: visible
to hide the scrollbars, but sometimes I'm loading content that NEEDS to scroll. This seems to be limited to this module's integration. I haven't encountered this issue using image styles or colorbox node module yet.
Comment | File | Size | Author |
---|---|---|---|
#2 | 1999226-colorbox_scrollbars-2.patch | 831 bytes | a.ross |
#1 | scroll.jpg | 43.47 KB | iacono |
Comments
Comment #1
iacono CreditAttribution: iacono commentedNot sure about the error but this is actually a setting in colorbox.
admin/config/media/colorbox
second to last option to will be Scrollbar settings:
Comment #2
a.ross CreditAttribution: a.ross commentedThat's because the iframe displays as inline by default, which means it has a line-height. Effectively the iframe become higher than its
height
property. Settingline-height: 0;
ordisplay: block;
fixes it.Patch...
Comment #3
b_willems CreditAttribution: b_willems commentedHi working on this as well.
With colorbox 7.x - 2.4 and the 1.x plugin, I don't get scrollbars in the config.
any suggestions for a patch on 2.0-beta5? 1st time using this module, not familiar with the code.
Comment #4
a.ross CreditAttribution: a.ross commentedTry to apply my patch
Comment #5
plopescHello
After some other changes in code, I can't reproduce this bug in 7.x-2.x-dev. So marking as fixed.
Please, feel free to reopen it if your problems persists.
Thank you for your interest in this module.
Regards