Index: lams_tool_sbmt/src/java/org/lamsfoundation/lams/tool/sbmt/web/MonitoringAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_sbmt/src/java/org/lamsfoundation/lams/tool/sbmt/web/MonitoringAction.java,v
diff -u -r1.40 -r1.41
--- lams_tool_sbmt/src/java/org/lamsfoundation/lams/tool/sbmt/web/MonitoringAction.java	2 May 2008 07:35:22 -0000	1.40
+++ lams_tool_sbmt/src/java/org/lamsfoundation/lams/tool/sbmt/web/MonitoringAction.java	5 May 2008 06:44:42 -0000	1.41
@@ -139,7 +139,7 @@
 		request.setAttribute(SbmtConstants.PAGE_EDITABLE, persistContent.isContentInUse());
 		
 		DynaActionForm smbtMonitoringForm = (DynaActionForm) form;
-		smbtMonitoringForm.set("currentTab", WebUtil.readStrParam(request, AttributeNames.PARAM_CURRENT_TAB,true));
+//		smbtMonitoringForm.set("currentTab", WebUtil.readStrParam(request, AttributeNames.PARAM_CURRENT_TAB,true));
 		
 		return mapping.findForward("success");
     }
Index: lams_tool_sbmt/web/authoring/instructions.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_sbmt/web/authoring/Attic/instructions.jsp,v
diff -u -r1.18 -r1.19
--- lams_tool_sbmt/web/authoring/instructions.jsp	18 Oct 2006 23:47:31 -0000	1.18
+++ lams_tool_sbmt/web/authoring/instructions.jsp	5 May 2008 06:44:42 -0000	1.19
@@ -71,7 +71,7 @@
 			<div class="field-name">
 				<fmt:message key="label.authoring.online.instruction"></fmt:message>
 			</div>
-			<html:textarea property="onlineInstruction" cols="75" rows="3"></html:textarea>
+			<lams:STRUTS-textarea property="onlineInstruction" rows="3" cols="75" />
 		</td>
 	</tr>
 	<tr>
@@ -109,7 +109,7 @@
 			<div class="field-name-alternative-color">
 				<fmt:message key="label.authoring.offline.instruction"></fmt:message>
 			</div>
-			<html:textarea property="offlineInstruction" cols="75" rows="3"></html:textarea>
+			<lams:STRUTS-textarea property="offlineInstruction" rows="3" cols="75" />
 		</td>
 	</tr>
 	<tr>
Index: lams_tool_sbmt/web/monitoring/parts/instructions.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_sbmt/web/monitoring/parts/Attic/instructions.jsp,v
diff -u -r1.2 -r1.3
--- lams_tool_sbmt/web/monitoring/parts/instructions.jsp	23 Aug 2006 05:42:37 -0000	1.2
+++ lams_tool_sbmt/web/monitoring/parts/instructions.jsp	5 May 2008 06:44:42 -0000	1.3
@@ -1,83 +1,94 @@
 <%@ include file="/common/taglibs.jsp"%>
 
-<table cellpadding="0" border="0">
-	<!-- Instructions Row -->
+<table cellspacing="0">
+	
 	<tr>
-		<td class="field-name" width="30%">
-			<fmt:message key="label.authoring.online.instruction" />
-			:
+		<td colspan="3">
+			<div class="field-name">
+				<fmt:message key="label.authoring.online.instruction" /> :
+			</div>	
+			<div style="padding-left: 20px;">
+				<c:out value="${authoring.onlineInstruction}" escapeXml="false" />
+			</div>
 		</td>
-		<td>
-			<c:out value="${authoring.onlineInstruction}" escapeXml="false" />
-		</td>
 	</tr>
+	
+	<c:if test="${not empty authoring.onlineFiles}">
+		<tr>
+			<td  colspan="3" class="field-name-alternative-color">
+				<fmt:message key="monitoring.instructions.attachments" />
+			</td>
+		</tr>		
+	</c:if>
+		
+	<c:forEach var="file" items="${authoring.onlineFiles}">
+		<tr >
+			<td width="7%" style="padding-left: 30px;">			
+				<c:out value="${file.name}" />
+			</td>
+			
+			<td width="3%" align="left">
+				<c:set var="viewURL">
+					<html:rewrite page="/download/?uuid=${file.uuID}&preferDownload=false" />
+				</c:set>
+				<a href="javascript:launchInstructionsPopup('<c:out value='${viewURL}' escapeXml='false'/>')" class="button"> <fmt:message key="label.view" /> </a> &nbsp;
+			</td>
+			
+			<td width="5%" align="left">
+				<c:set var="downloadURL">
+					<html:rewrite page="/download/?uuid=${file.uuID}&versionID=${file.versionID}&preferDownload=true" />
+				</c:set>
+				<a href="<c:out value='${downloadURL}' escapeXml='false'/>" class="button"> <fmt:message key="label.download" /> </a>
+			</td>
+		</tr>
+	</c:forEach>
+
+	
 	<tr>
-		<td class="field-name-alternative-color">
-			<fmt:message key="label.authoring.online.filelist" />
-			:
+		<td colspan="3">
+			<hr />
 		</td>
-		<td></td>
 	</tr>
+	
 	<tr>
-		<td colspan="2">
-			<ul>
-			<c:forEach var="file" items="${authoring.onlineFiles}">
-				<li>
-					<c:out value="${file.name}" />
-					<html:link href="javascript:launchInstructionsPopup('download/?uuid=${file.uuID}&preferDownload=false')">
-						<fmt:message key="label.view" />
-					</html:link>
-					&nbsp;&nbsp;
-					<c:set var="downloadURL">
-							<html:rewrite page="/download/?uuid=${file.uuID}&versionID=${file.versionID}&preferDownload=true" />
-					</c:set>
-					<html:link href="${downloadURL}">
-						<fmt:message key="label.download" />
-					</html:link>
-				</li>
-			</c:forEach>
-			</ul>
+		<td colspan="3">
+			<div class="field-name">
+				<fmt:message key="label.authoring.offline.instruction" /> :
+			</div>	
+			<div style="padding-left: 20px;">
+				<c:out value="${authoring.offlineInstruction}" escapeXml="false" />
+			</div>
 		</td>
 	</tr>
-	<tr>
-		<td colspan="2">
-		<hr size="1" style="width:550px"/>
-		</td>
-	</tr>
-	<tr>
-		<td class="field-name" width="30%">
-			<fmt:message key="label.authoring.offline.instruction" />
-			:
-		</td>
-		<td>
-			<c:out value="${authoring.offlineInstruction}" escapeXml="false" />
-		</td>
-	</tr>
-	<tr>
-		<td class="field-name-alternative-color">
-			<fmt:message key="label.authoring.offline.filelist" />
-			:
-		</td>
-		<td></td>		
-	</tr>
-	<tr>
-		<td colspan="2">
-			<ul>
-				<c:forEach var="file" items="${authoring.offlineFiles}">
-					<li>
-						<c:out value="${file.name}" />
-						<html:link href="javascript:launchInstructionsPopup('download/?uuid=${file.uuID}&preferDownload=false')">
-							<fmt:message key="label.view" />
-						</html:link>&nbsp;&nbsp;
-						<c:set var="downloadURL">
-								<html:rewrite page="/download/?uuid=${file.uuID}&versionID=${file.versionID}&preferDownload=true" />
-						</c:set>
-						<html:link href="${downloadURL}">
-							<fmt:message key="label.download" />
-						</html:link>
-					</li>
-				</c:forEach>
-			</ul>
-		</td>
-	</tr>
+	
+	<c:if test="${not empty authoring.offlineFiles}">
+		<tr>
+			<td  colspan="3" class="field-name-alternative-color">
+				<fmt:message key="monitoring.instructions.attachments" />
+			</td>
+		</tr>
+	</c:if>		
+		
+	<c:forEach var="file" items="${authoring.offlineFiles}">
+		<tr >
+			<td width="7%" style="padding-left: 30px;">			
+				<c:out value="${file.name}" />
+			</td>
+			
+			<td width="3%" align="left">
+				<c:set var="viewURL">
+					<html:rewrite page="/download/?uuid=${file.uuID}&preferDownload=false" />
+				</c:set>
+				<a href="javascript:launchInstructionsPopup('<c:out value='${viewURL}' escapeXml='false'/>')" class="button"> <fmt:message key="label.view" /> </a> &nbsp;
+			</td>
+			
+			<td width="5%" align="left">
+				<c:set var="downloadURL">
+					<html:rewrite page="/download/?uuid=${file.uuID}&versionID=${file.versionID}&preferDownload=true" />
+				</c:set>
+				<a href="<c:out value='${downloadURL}' escapeXml='false'/>" class="button"> <fmt:message key="label.download" /> </a>
+			</td>
+		</tr>
+	</c:forEach>
+	
 </table>