Index: lams_monitoring/web/monitorLesson.jsp =================================================================== diff -u -r4607e31e2b78944be8fe4e2edc590cf3d1975dc6 -r35d33684e3444f4304b7c22aa87ae5503a75ab18 --- lams_monitoring/web/monitorLesson.jsp (.../monitorLesson.jsp) (revision 4607e31e2b78944be8fe4e2edc590cf3d1975dc6) +++ lams_monitoring/web/monitorLesson.jsp (.../monitorLesson.jsp) (revision 35d33684e3444f4304b7c22aa87ae5503a75ab18) @@ -42,7 +42,6 @@ // Use for handling popup windows var pWins = new Array(); - var iWins = 0; var isInternetExplorer = navigator.appName.indexOf("Microsoft") != -1; @@ -75,14 +74,18 @@ } function openPopUpFS(args){ - var params = args.split(","); + var params = args.split(","); var urlparams = args.split("&"); - var activityID = urlparams[1]; - + // Internet Explorer (7) doesn't like having the equals '=' sign in title param of + // the window.open call (bottom of openPopUp method) so we need to remove it + var activityAndID = urlparams[1]; // activityID=123... + var tmpArr = activityAndID.split("="); + var activityID = tmpArr[1]; + // assigned the args var url = params[0]; - var title = params[1] + "_" + activityID; + var title = params[1] + "_activityID_" + activityID; var h = params[2]; var w = params[3]; var resize = params[4]; @@ -99,28 +102,19 @@ } function openPopUp(args, title, h, w, resize, status, scrollbar, menubar, toolbar){ - // refocus code commented out as we want to replace contents due to tool's session issues. Code will be - // wanted again the future. - //if(thePopUp && thePopUp.open && !thePopUp.closed){ - // thePopUp.focus(); - - //}else{ - // thePopUp = window.open(args,title,"HEIGHT="+h+",WIDTH="+w+",resizable="+resize+",scrollbars=yes,status="+status+",menubar="+menubar+", toolbar="+toolbar); - //} var found = false; - for(var i=0; i