The Modal Window - when clicking on "Browse Interactive Videos" should have overflow: scroll. If not only 6 elements are shown.

Comments

patrickroma’s picture

Update: This only seems to be a problem on Safari (Mac). In Firefox (Mac) the scrollbar is visible.

Als’s picture

Title: Browse Interactive Videos » Modal window ("Browse Interactive Videos") not scrolling, hides content from user view
Category: feature » bug

I 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.

xurizaemon’s picture

Assigned: Unassigned » xurizaemon
Status: Active » Needs work

Instructions 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

xurizaemon’s picture

Status: Needs work » Needs review
StatusFileSize
new4.05 KB

Two 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.

xurizaemon’s picture

Review this one instead please.

xurizaemon’s picture

Just happened onto a running IE7 VM, so gave it a quick test there too - seems to work OK in IE7 also.

Als’s picture

I 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!

xurizaemon’s picture

Status: Needs review » Reviewed & tested by the community

RTBC based on feedback from Als, thanks. Will commit shortly.

xurizaemon’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)
Issue tags: -CSS, -upload, -browser compatibility, -file management

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

xurizaemon’s picture

Assigned: xurizaemon » Unassigned
Issue summary: View changes