Index: lams_central/web/css/progressBar.css
===================================================================
diff -u -r9438790401b957cdd70a4d6ab2a76da70176c6eb -re2ca795a36c3539c53c0845757e87b0792bf9aa5
--- lams_central/web/css/progressBar.css	(.../progressBar.css)	(revision 9438790401b957cdd70a4d6ab2a76da70176c6eb)
+++ lams_central/web/css/progressBar.css	(.../progressBar.css)	(revision e2ca795a36c3539c53c0845757e87b0792bf9aa5)
@@ -35,4 +35,14 @@
 
 div.optionalActivity td:hover {
 	background-color: rgb(246, 238, 191);
+}
+
+div#progressBarDialog {
+	display: none;
+}
+
+div#progressBarDialog iframe {
+	width: 100%;
+	height: 100%;
+	border: none;
 }
\ No newline at end of file
Index: lams_central/web/includes/javascript/progressBar.js
===================================================================
diff -u -r1b117caf4135f53248542cbc97d71aac448f3de9 -re2ca795a36c3539c53c0845757e87b0792bf9aa5
--- lams_central/web/includes/javascript/progressBar.js	(.../progressBar.js)	(revision 1b117caf4135f53248542cbc97d71aac448f3de9)
+++ lams_central/web/includes/javascript/progressBar.js	(.../progressBar.js)	(revision e2ca795a36c3539c53c0845757e87b0792bf9aa5)
@@ -1,8 +1,10 @@
 // ------- GLOBAL VARIABLES ----------
 // IMPORTANT: set following variables on the page which imports this JS file
-// var isHorizontalBar = false;
-// var hasContentFrame = true;
-// above settings are default for Learner page
+var isHorizontalBar = isHorizontalBar || false;
+var hasContentFrame = hasContentFrame || true;
+var hasDialog = hasDialog || false;
+var presenceEnabled = presenceEnabled || false;
+var REVIEW_ACTIVITY_TITLE = REVIEW_ACTIVITY_TITLE || 'Review activity';
 
 // colors used in shapes
 // dark red
@@ -45,7 +47,35 @@
 
 // just a short cut to openPopUp function
 function openActivity(url) {
-	openPopUp(url, "LearnerActivity", 600, 800, "yes");
+	if (hasDialog) {
+		var dialog = $('#progressBarDialog');
+		if (!dialog.hasClass('ui-dialog')) {
+			dialog.dialog({
+				'autoOpen' : false,
+				'height' : 600,
+				'width' : 800,
+				'modal' : true,
+				'resizable' : false,
+				'hide' : 'fold',
+				'title' : REVIEW_ACTIVITY_TITLE,
+				'open' : function() {
+					// load contents after opening the dialog
+					$('iframe', this).load(function(){
+						// remove finish button
+						// so user can only close dialog with X button
+						$(this).contents().find('#finishButton, #finish').remove();
+					}).attr('src', url);
+				},
+				'beforeClose' : function(){
+					$('iframe', this).off('load').attr('src', null);
+				}
+			});
+		}
+		
+		dialog.dialog('open');
+	} else {
+		openPopUp(url, "LearnerActivity", 600, 800, "yes");
+	}
 }
 
 // loads a new activity to main content frame; alternative to opening in pop up
@@ -391,7 +421,7 @@
 
 		var isSupportActivity = activity instanceof SupportActivity;
 		var dblclick = activity.url ? function() {
-			// open pop up if it is a support or completed activity
+			// open pop up or dialog if it is a support or completed activity
 			if (isSupportActivity
 					|| activity.status == 1
 					|| (!hasContentFrame && activity.status <= 2)) {
Index: lams_learning/conf/language/lams/ApplicationResources.properties
===================================================================
diff -u -r8aded0b18a5a7712622ae56f38923fa297449977 -re2ca795a36c3539c53c0845757e87b0792bf9aa5
--- lams_learning/conf/language/lams/ApplicationResources.properties	(.../ApplicationResources.properties)	(revision 8aded0b18a5a7712622ae56f38923fa297449977)
+++ lams_learning/conf/language/lams/ApplicationResources.properties	(.../ApplicationResources.properties)	(revision e2ca795a36c3539c53c0845757e87b0792bf9aa5)
@@ -120,6 +120,7 @@
 label.learner.progress.activity.attempted.tooltip  =You have attempted but not yet finished this activity
 label.learner.progress.activity.tostart.tooltip  =You need to complete the activities before this activity to access it
 label.learner.progress.activity.support.tooltip  =Double click to participate in this support activity
+label.learner.progress.review.activity =Review activity
 button.exit  =Exit
 button.export  =Export
 button.save  =Save
Index: lams_learning/web/main.jsp
===================================================================
diff -u -r9438790401b957cdd70a4d6ab2a76da70176c6eb -re2ca795a36c3539c53c0845757e87b0792bf9aa5
--- lams_learning/web/main.jsp	(.../main.jsp)	(revision 9438790401b957cdd70a4d6ab2a76da70176c6eb)
+++ lams_learning/web/main.jsp	(.../main.jsp)	(revision e2ca795a36c3539c53c0845757e87b0792bf9aa5)
@@ -41,7 +41,6 @@
 	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/jquery.form.js"></script>
 	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/jquery.layout.js"></script>
 	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/raphael/raphael.js"></script>
-	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/progressBar.js"></script>
 	<script type="text/javascript" src="includes/javascript/main.js"></script>
 	<script type="text/javascript">
 		var LAMS_URL = '<lams:LAMSURL/>';
@@ -59,9 +58,11 @@
 		var parentURL = "${notifyCloseURL}";
 		var lessonId = '${param.lessonID}';
 		var progressPanelEnabled = '<lams:Configuration key="LearnerCollapsProgressPanel" />' != 'false';
+		// settings for progress bar
 		var presenceEnabled = '${param.presenceEnabledPatch}' != 'false';
 		var isHorizontalBar = false;
 		var hasContentFrame = true;
+		var hasDialog = false;
 		var bars = {
 			'learnerMainBar' : {
 				'containerId' : 'progressBarDiv'
@@ -103,6 +104,8 @@
 			}
 		});
 	</script>
+	<!-- Some settings need to be done in the script first and only then this file can be included -->
+	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/progressBar.js"></script>
 </lams:head>
 
 <body class="stripes">
Index: lams_monitoring/web/monitor.jsp
===================================================================
diff -u -rd1b23130f9980a55d8df855d834238415ba50aa2 -re2ca795a36c3539c53c0845757e87b0792bf9aa5
--- lams_monitoring/web/monitor.jsp	(.../monitor.jsp)	(revision d1b23130f9980a55d8df855d834238415ba50aa2)
+++ lams_monitoring/web/monitor.jsp	(.../monitor.jsp)	(revision e2ca795a36c3539c53c0845757e87b0792bf9aa5)
@@ -16,17 +16,18 @@
 	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/jquery-ui.js"></script>
 	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/jquery-ui.timepicker.js"></script>
 	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/raphael/raphael.js"></script>
-	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/progressBar.js"></script>
 	<script type="text/javascript" src="includes/javascript/monitorLesson.js"></script>
 	<script type="text/javascript">
 		var userId = '<lams:user property="userID"/>';
 		var lessonId = ${lesson.lessonID};
 		var ldId = ${lesson.learningDesignID};
 		var lessonStateId = ${lesson.lessonStateID};
 		var createDateTimeStr = '${lesson.createDateTimeStr}';
+		// settings for progress bar
 		var isHorizontalBar = true;
 		var hasContentFrame = false;
 		var presenceEnabled =  false;
+		var hasDialog = false;
 		
 		var LAMS_URL = '<lams:LAMSURL/>';
 		
@@ -84,6 +85,8 @@
 			refreshMonitor();
 		});
 	</script>
+	<!-- Some settings need to be done in the script first and only then this file can be included -->
+	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/progressBar.js"></script>
 </lams:head>
 <body>
 <div id="tabs">