Index: lams_tool_spreadsheet/conf/language/lams/ApplicationResources_en_AU.properties
===================================================================
diff -u -re044a561b88b3eb68dc50846451a30d258108e0f -r943ef2ac69e4ce13ac802a4264ed9bb6864507f0
--- lams_tool_spreadsheet/conf/language/lams/ApplicationResources_en_AU.properties	(.../ApplicationResources_en_AU.properties)	(revision e044a561b88b3eb68dc50846451a30d258108e0f)
+++ lams_tool_spreadsheet/conf/language/lams/ApplicationResources_en_AU.properties	(.../ApplicationResources_en_AU.properties)	(revision 943ef2ac69e4ce13ac802a4264ed9bb6864507f0)
@@ -54,10 +54,10 @@
 label.learning.not.available  =Not Available
 label.monitoring.summary.learner  =Learner
 label.monitoring.summary.marked  =Marked?
-label.monitoring.summary.lock.when.finished  =Lock when finished: {0}
-label.monitoring.summary.individual.spreadsheets  =Learners are allowed to submit individual spreadsheets: {0}
-label.monitoring.summary.marking.enabled  =Spreadsheet marking is enabled: {0}
-label.monitoring.summary.notebook.reflection  =Notebook Reflection: {0}
+label.monitoring.summary.lock.when.finished  =Lock when finished
+label.monitoring.summary.individual.spreadsheets  =Learners are allowed to submit individual spreadsheets
+label.monitoring.summary.marking.enabled  =Spreadsheet marking is enabled
+label.monitoring.summary.notebook.reflection  =Notebook Reflection
 label.monitoring.summary.overall.summary  =Overall Summary
 label.monitoring.summary.confirm.completion  =Confirm Completion
 label.monitoring.summary.title.reflection  =Notebook Entry
@@ -167,7 +167,7 @@
 label.authoring.save.button  =Save
 monitoring.label.access.time  =Access time
 label.authoring.advance.lock.on.finished  =Lock when finished
-label.authoring.heading.advance  =Advanced
+label.authoring.heading.advance  =Advanced Settings
 message.step.of  =Step {0} of {1}
 label.up  =Move Up
 label.down  =Move down
Index: lams_tool_spreadsheet/src/java/org/lamsfoundation/lams/tool/spreadsheet/web/action/MonitoringAction.java
===================================================================
diff -u -re044a561b88b3eb68dc50846451a30d258108e0f -r943ef2ac69e4ce13ac802a4264ed9bb6864507f0
--- lams_tool_spreadsheet/src/java/org/lamsfoundation/lams/tool/spreadsheet/web/action/MonitoringAction.java	(.../MonitoringAction.java)	(revision e044a561b88b3eb68dc50846451a30d258108e0f)
+++ lams_tool_spreadsheet/src/java/org/lamsfoundation/lams/tool/spreadsheet/web/action/MonitoringAction.java	(.../MonitoringAction.java)	(revision 943ef2ac69e4ce13ac802a4264ed9bb6864507f0)
@@ -333,41 +333,7 @@
 	public ActionForward saveMark(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) {
 		MarkForm markForm = (MarkForm)form;
 		
-		
-//		ActionMessages errors = new ActionMessages();  
-//		//check whether the mark is validate
-//		String markStr = request.getParameter("marks");
-//		Long marks = null;
-//		try {
-//			marks = Long.parseLong(markStr);
-//		} catch (Exception e) {
-//			errors.add(ActionMessages.GLOBAL_MESSAGE,new ActionMessage("errors.mark.invalid.number"));
-//		}
-//		
-//		String comments = WebUtil.readStrParam(request,"comments",true);
-//		if(!errors.isEmpty()){
-//			submitFilesService = getSubmitFilesService();
-//			List report = new ArrayList<FileDetailsDTO>();
-//			FileDetailsDTO fileDetail = submitFilesService.getFileDetails(detailID);
-//			//echo back the input, even they are wrong.
-//			fileDetail.setComments(comments);
-//			fileDetail.setMarks(markStr);
-//			report.add(fileDetail);
-//			
-//			request.setAttribute("report",report);
-//			request.setAttribute("updateMode", updateMode);
-//			request.setAttribute(AttributeNames.PARAM_TOOL_SESSION_ID,sessionID);
-//			
-//			
-//			saveErrors(request,errors);
-//			return mapping.findForward("updateMark");
-//		}
-		
-		
-		
-		
 		ActionErrors errors = validateSpreadsheetMark(markForm);
-		
 		if(!errors.isEmpty()){
 			this.addErrors(request,errors);
 			return mapping.findForward("editMark");
Index: lams_tool_spreadsheet/web/pages/monitoring/parts/summarylist.jsp
===================================================================
diff -u -r56d116683b7554e8ad3677cd0e12866ac3feb834 -r943ef2ac69e4ce13ac802a4264ed9bb6864507f0
--- lams_tool_spreadsheet/web/pages/monitoring/parts/summarylist.jsp	(.../summarylist.jsp)	(revision 56d116683b7554e8ad3677cd0e12866ac3feb834)
+++ lams_tool_spreadsheet/web/pages/monitoring/parts/summarylist.jsp	(.../summarylist.jsp)	(revision 943ef2ac69e4ce13ac802a4264ed9bb6864507f0)
@@ -146,7 +146,7 @@
 </div>
 
 <%-- This script will works when a new resoruce Condition submit in order to refresh "TaskList List" panel. --%>
-<script lang="javascript">
+<script type="text/javascript">
 	if(window.top != null){
 		window.top.hideMessage();
 		var obj = window.top.document.getElementById('summariesArea');
Index: lams_tool_spreadsheet/web/pages/monitoring/summary.jsp
===================================================================
diff -u -r0713688db3433dbda8d0780e1b101342a90bcd9a -r943ef2ac69e4ce13ac802a4264ed9bb6864507f0
--- lams_tool_spreadsheet/web/pages/monitoring/summary.jsp	(.../summary.jsp)	(revision 0713688db3433dbda8d0780e1b101342a90bcd9a)
+++ lams_tool_spreadsheet/web/pages/monitoring/summary.jsp	(.../summary.jsp)	(revision 943ef2ac69e4ce13ac802a4264ed9bb6864507f0)
@@ -6,7 +6,8 @@
 	<lams:WebAppURL />
 </c:set>
 
-<script lang="javascript">
+<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/monitorToolSummaryAdvanced.js"></script>
+<script type="text/javascript">
 	function showMessage(url) {
 		var area=document.getElementById("marksInputArea");
 		if(area != null){
@@ -86,57 +87,88 @@
 	}	
 </script>
 
-<h1><fmt:message key="label.authoring.heading.advance"/></h1>
 
-<%-- Overall TaskList information  --%>
-<div class="monitoring-advanced">
+<h1>
 
-	<fmt:message key="label.monitoring.summary.lock.when.finished"><fmt:param>
-	<c:choose>
-		<c:when test="${spreadsheet.lockWhenFinished}">
-			<fmt:message key="label.on" />
-		</c:when>
-		<c:otherwise>
-			<fmt:message key="label.off" />
-		</c:otherwise>
-	</c:choose>	
-	</fmt:param></fmt:message>
-	<br/>
+	<img src="<lams:LAMSURL/>/images/tree_closed.gif" id="treeIcon" onclick="javascript:toggleAdvancedOptionsVisibility(document.getElementById('advancedDiv'), document.getElementById('treeIcon'), '<lams:LAMSURL/>');" />
 
-	<fmt:message key="label.monitoring.summary.individual.spreadsheets"><fmt:param>
-	<c:choose>
-		<c:when test="${spreadsheet.learnerAllowedToSave}">
-			<fmt:message key="label.on" />
-		</c:when>
-		<c:otherwise>
-			<fmt:message key="label.off" />
-		</c:otherwise>
-	</c:choose>	
-	</fmt:param></fmt:message>
-	<br/>
-	
-	<fmt:message key="label.monitoring.summary.marking.enabled"><fmt:param>
-	<c:choose>
-		<c:when test="${spreadsheet.markingEnabled}">
-			<fmt:message key="label.on" />
-		</c:when>
-		<c:otherwise>
-			<fmt:message key="label.off" />
-		</c:otherwise>
-	</c:choose>	
-	</fmt:param></fmt:message>
-	<br/>
-	
-	<fmt:message key="label.monitoring.summary.notebook.reflection"><fmt:param>
-	<c:choose>
-		<c:when test="${spreadsheet.reflectOnActivity}">
-			<fmt:message key="label.on" />
-		</c:when>
-		<c:otherwise>
-			<fmt:message key="label.off" />
-		</c:otherwise>
-	</c:choose>	
-	</fmt:param></fmt:message>
+	<a href="javascript:toggleAdvancedOptionsVisibility(document.getElementById('advancedDiv'), document.getElementById('treeIcon'),'<lams:LAMSURL/>');" >
+		<fmt:message key="label.authoring.heading.advance"/>
+	</a>
+</h1>
+<br />
+
+<%-- Overall TaskList information  --%>
+<div class="monitoring-advanced" id="advancedDiv" style="display:none">
+	<table class="alternative-color">
+		<tr>
+			<td>
+				<fmt:message key="label.monitoring.summary.lock.when.finished" />
+			</td>
+			
+			<td>
+				<c:choose>
+					<c:when test="${spreadsheet.lockWhenFinished}">
+						<fmt:message key="label.on" />
+					</c:when>
+					<c:otherwise>
+						<fmt:message key="label.off" />
+					</c:otherwise>
+				</c:choose>	
+			</td>	
+		</tr>
+		
+		<tr>
+			<td>
+				<fmt:message key="label.monitoring.summary.individual.spreadsheets" />
+			</td>
+			
+			<td>
+				<c:choose>
+					<c:when test="${spreadsheet.learnerAllowedToSave}">
+						<fmt:message key="label.on" />
+					</c:when>
+					<c:otherwise>
+						<fmt:message key="label.off" />
+					</c:otherwise>
+				</c:choose>
+			</td>	
+		</tr>
+		
+		<tr>
+			<td>
+				<fmt:message key="label.monitoring.summary.marking.enabled" />
+			</td>
+			
+			<td>
+				<c:choose>
+					<c:when test="${spreadsheet.markingEnabled}">
+						<fmt:message key="label.on" />
+					</c:when>
+					<c:otherwise>
+						<fmt:message key="label.off" />
+					</c:otherwise>
+				</c:choose>
+			</td>	
+		</tr>
+		
+		<tr>
+			<td>
+				<fmt:message key="label.monitoring.summary.notebook.reflection" />
+			</td>
+			
+			<td>
+				<c:choose>
+					<c:when test="${spreadsheet.reflectOnActivity}">
+						<fmt:message key="label.on" />
+					</c:when>
+					<c:otherwise>
+						<fmt:message key="label.off" />
+					</c:otherwise>
+				</c:choose>	
+			</td>	
+		</tr>
+	</table>
 </div>
 
 <%-- Summary list  --%>