MDL-14492 - Provide feedback to user about redirect delay in loadsco.php. Add sanity checking for scorm_resize() as it tries to set -ve heights if user specifies a small area. Thanks francois.
This commit is contained in:
+23
-7
@@ -14,6 +14,7 @@ function scorm_get_element_style(obj, prop, cssProp) {
|
||||
}
|
||||
|
||||
function scorm_resize (cwidth, cheight) {
|
||||
|
||||
var winwidth = 0, winheight = 0;
|
||||
if( typeof( window.innerWidth ) == 'number' ) {
|
||||
//Non-IE
|
||||
@@ -60,21 +61,29 @@ function scorm_resize (cwidth, cheight) {
|
||||
scormtopheight+
|
||||
topmargin +
|
||||
bottommargin+10; // +10 to save a minor vertical scroll always present!
|
||||
|
||||
|
||||
var totalheighttoc = totalheight+imsnavheight;
|
||||
// override total height with configured height if it is smaller
|
||||
// override total height with configured height if it is defined
|
||||
if (cheight > 0) {
|
||||
winheight = cheight;
|
||||
}
|
||||
var finalheighttoc = winheight - totalheighttoc;
|
||||
if (finalheighttoc <= 0) {
|
||||
finalheighttoc = winheight;
|
||||
}
|
||||
var finalheight = winheight - totalheight;
|
||||
if (finalheight <= 0) {
|
||||
finalheight = winheight;
|
||||
}
|
||||
var toctree = document.getElementById('toctree');
|
||||
if (toctree != null){
|
||||
var toctreeHeight = toctree.offsetHeight;
|
||||
document.getElementById('toctree').style.height = (winheight - totalheighttoc) + 'px';
|
||||
document.getElementById('toctree').style.height = finalheighttoc + 'px';
|
||||
var scoframe2 = document.getElementById('scoframe1');
|
||||
document.getElementById('scormobject').style.height = (winheight - totalheight) + 'px';
|
||||
document.getElementById('scormobject').style.height = finalheight + 'px';
|
||||
}else{
|
||||
var scoframe2 = document.getElementById('scoframe1');
|
||||
document.getElementById('scormobject').style.height = (winheight - totalheight) + 'px';
|
||||
document.getElementById('scormobject').style.height = finalheight + 'px';
|
||||
}
|
||||
|
||||
// resize the content container too to move the footer below the SCORM content
|
||||
@@ -82,7 +91,14 @@ function scorm_resize (cwidth, cheight) {
|
||||
if (contenti3) {
|
||||
contenti3.style.height = (winheight - totalheight + 30) + 'px';
|
||||
} else {
|
||||
document.getElementById('content').style.height = (winheight - totalheight + 30) + 'px';
|
||||
document.getElementById('content').style.height = (finalheight + 30) + 'px';
|
||||
}
|
||||
// resize the content container too to move the footer below the SCORM content
|
||||
var contenti3 = document.getElementById('content-i3');
|
||||
if (contenti3) {
|
||||
contenti3.style.height = (finalheight + 30) + 'px';
|
||||
} else {
|
||||
document.getElementById('content').style.height = (finalheight + 30) + 'px';
|
||||
}
|
||||
}
|
||||
-->
|
||||
-->
|
||||
Reference in New Issue
Block a user