The Modal Window - when clicking on "Browse Interactive Videos" should have overflow: scroll. If not only 6 elements are shown.
The Modal Window - when clicking on "Browse Interactive Videos" should have overflow: scroll. If not only 6 elements are shown.
Comments
Comment #1
patrickroma commentedUpdate: This only seems to be a problem on Safari (Mac). In Firefox (Mac) the scrollbar is visible.
Comment #2
AlsI confirm this problem exits in IE 8 for Windows, too.
Editing title to reflect the problem, and switching category to "bug report".
In Firefox 3 the vertical scrollbar appears, probably provided by the browser itself as it detects additional content there.
I tried playing with the "scrolling" parameter in this row, with no success for IE (file kaltura.module, row 379):
$js .= PHP_EOL .'window.top.document.getElementById("kaltura_modal_iframe").scrolling = "";';Also tried adding "overflow: scroll" to the relevant classes in kaltura.css, with no result.
Comment #3
xurizaemonInstructions to replicate (able to do so in Chrome, Safari, IE8) -
1. Enable Kaltura as CCK and dependencies
2. Add a Kaltura CCK field to any content type
3. Have enough media (>6) to require scrolling or paging to view all entries
4. Add a node of the content type with Kaltura CCK field
5. Click "Add Kaltura Media" button
6. Modal dialog will not show all Kaltura entries
Comment #4
xurizaemonTwo related issues here.
In WebKit browsers, only a CSS issue is apparent; you can't scroll to view any media past the first few, but you can use the pager (which will show you the first several of each block of twenty Kaltura media).
In IE8 (probably earlier too), a JS error prevented the modal dialog being resized as intended, so when browsing media the dialog was only 380px high; no pager visible and even less media available as a result.
Patch fixes both issues; tested in Safari 5, Chrome 11, Firefox 4 OSX, IE8 Windows.
Comment #5
xurizaemonReview this one instead please.
Comment #6
xurizaemonJust happened onto a running IE7 VM, so gave it a quick test there too - seems to work OK in IE7 also.
Comment #7
AlsI confirm the patch #5 solves the problem in IE 8/XP and IE 9/Vista.
Firefox 3/Vista keeps working fine.
Thanks grobot, please commit this small but fundamental fix to the next release!
Comment #8
xurizaemonRTBC based on feedback from Als, thanks. Will commit shortly.
Comment #9
xurizaemoncommitted
Comment #12
xurizaemon