Index: lams_tool_lamc/web/authoring/InstructionsContent.jsp
===================================================================
diff -u -r8b5fd331984013fa4afe645a4753ae7562b0dc84 -r9ad7805407672e72dfc83fb0ba49ccafb0d43d6f
--- lams_tool_lamc/web/authoring/InstructionsContent.jsp	(.../InstructionsContent.jsp)	(revision 8b5fd331984013fa4afe645a4753ae7562b0dc84)
+++ lams_tool_lamc/web/authoring/InstructionsContent.jsp	(.../InstructionsContent.jsp)	(revision 9ad7805407672e72dfc83fb0ba49ccafb0d43d6f)
@@ -20,179 +20,140 @@
   http://www.gnu.org/licenses/gpl.txt
 --%>
 
-<%@ include file="/common/taglibs.jsp" %>
+<%@ include file="/common/taglibs.jsp"%>
 
-		<table class="forms">
 
-				<tr>
-					<td NOWRAP colspan=2 valign=top>
-						<div class="field-name" style="text-align: left;">
-							<fmt:message key="label.onlineInstructions.col"></fmt:message>
-						</div>
-						<html:textarea property="onlineInstructions" rows="3" cols="80"></html:textarea>
-					</td>
-				</tr>
-				
-				
-				<tr>
-				<td NOWRAP colspan=2 valign=top>
-				<table class="forms">
-					<tr><td align=center>
 
-							<table  width="100%" align=center  border="0" cellspacing="0" cellpadding="0">
-								<tr>
-									<td>
-										<table width="70%" align="left">
 
-									<c:forEach var="attachment" items="${mcGeneralAuthoringDTO.attachmentList}">
-											<c:if test="${attachment.fileOnline == true }"> 			
-								            	<bean:define id="view">/download/?uuid=<bean:write name="attachment" property="uuid"/>&preferDownload=false</bean:define>
-												<bean:define id="download">/download/?uuid=<bean:write name="attachment" property="uuid"/>&preferDownload=true</bean:define>
-						                        <bean:define id="uuid" name="attachment" property="uuid" />
-						                        
-						                        <tr>
-									            	<td> <bean:write name="attachment" property="fileName"/>  </td>
-										            <td>
-											        	<table>
-												        	<tr>
-												            	<td>
-												                	<a href='javascript:launchInstructionsPopup("<html:rewrite page='<%=view%>'/>")' class="button">
-												                   		<fmt:message key="label.view" />
-												                    </a>
-																	&nbsp&nbsp
-													            	<html:link page="<%=download%>" styleClass="button">
-													                	<fmt:message key="label.download" />
-													                </html:link>
-																	&nbsp&nbsp   
-													            	<html:link page="/authoring.do?dispatch=deleteFile&httpSessionID=${mcGeneralAuthoringDTO.httpSessionID}&toolContentID=${mcGeneralAuthoringDTO.toolContentID}&contentFolderID=${mcGeneralAuthoringDTO.contentFolderID}&activeModule=${mcGeneralAuthoringDTO.activeModule}&defaultContentIdStr=${mcGeneralAuthoringDTO.defaultContentIdStr}&sln=${mcGeneralAuthoringDTO.sln}&questionsSequenced=${mcGeneralAuthoringDTO.questionsSequenced}&retries=${mcGeneralAuthoringDTO.retries}&reflect=${mcGeneralAuthoringDTO.reflect}&reflectionSubject=${mcGeneralAuthoringDTO.reflectionSubject}&passmark=${mcGeneralAuthoringDTO.passMarkValue}"
-													                         	paramId="uuid" paramName="attachment" paramProperty="uuid"
-													                         	onclick="javascript:return confirm('Are you sure you want to delete this file?')"
-													                         	target="_self" styleClass="button">
-													                	<fmt:message key="label.delete" />
-													                </html:link> 
-													            </td>
-												           	</tr>
-											            </table>
-										           	</td>
-									   	     	</tr>
-											</c:if> 											   	     	
-										</c:forEach>
-										</table>
-								 	</td>
-								</tr>
-							</table>
 
+<table>
 
+	<tr>
+		<td colspan="2">
+			<div class="field-name">
+				<fmt:message key="label.onlineInstructions.col"></fmt:message>
+			</div>
+			<html:textarea property="onlineInstructions" rows="3" cols="75"></html:textarea>
+		</td>
+	</tr>
 
-							 	</td>
-							</tr>
-				</table>
-					</td> 				
-				</tr>
-				
-				
-				<tr> 
-					<td class="field-name"> 
-	          				<fmt:message key="label.onlineFiles" />
-          			</td>
-          			<td NOWRAP> 
-						<html:file  property="theOnlineFile"></html:file>
-					 	<html:submit property="submitOnlineFile" 
-                                     styleClass="linkbutton" 
-                                     onclick="submitMethod('addNewFile');">
-								<fmt:message key="label.upload"/>
-						</html:submit>
-					</td> 
-				
-				</tr>
 
+	<tr>
+		<td>
+			<ul>
+				<c:forEach var="attachment"
+					items="${mcGeneralAuthoringDTO.attachmentList}">
+					<c:if test="${attachment.fileOnline == true }">
+						<li>
+							<bean:define id="view">/download/?uuid=<bean:write
+									name="attachment" property="uuid" />&preferDownload=false</bean:define>
+							<bean:define id="download">/download/?uuid=<bean:write
+									name="attachment" property="uuid" />&preferDownload=true</bean:define>
+							<bean:define id="uuid" name="attachment" property="uuid" />
 
-				<tr>
-					<td NOWRAP colspan=2 valign=top>
-						<div class="field-name" style="text-align: left;">
-							<fmt:message key="label.offlineInstructions.col"></fmt:message>
-						</div>
-						<html:textarea property="offlineInstructions" rows="3" cols="80"></html:textarea>
-					</td>
-				</tr>
 
+							<bean:write name="attachment" property="fileName" />
 
-				<tr>
-					<td NOWRAP colspan=2 valign=top>
-				<table class="forms">
-					<tr><td align=center>
-							<table  width="100%" align=center  border="0" cellspacing="0" cellpadding="0">
-								<tr>
-									<td>
-										<table width="70%" align="left">
+							<a
+								href='javascript:launchInstructionsPopup("<html:rewrite page='<%=view%>'/>")'>
+								<fmt:message key="label.view" /> </a> &nbsp;
+							<html:link page="<%=download%>">
+								<fmt:message key="label.download" />
+							</html:link>
+							&nbsp;
+							<html:link
+								page="/authoring.do?dispatch=deleteFile&amp;httpSessionID=${mcGeneralAuthoringDTO.httpSessionID}&amp;toolContentID=${mcGeneralAuthoringDTO.toolContentID}&amp;contentFolderID=${mcGeneralAuthoringDTO.contentFolderID}&amp;activeModule=${mcGeneralAuthoringDTO.activeModule}&amp;defaultContentIdStr=${mcGeneralAuthoringDTO.defaultContentIdStr}&amp;sln=${mcGeneralAuthoringDTO.sln}&amp;questionsSequenced=${mcGeneralAuthoringDTO.questionsSequenced}&amp;retries=${mcGeneralAuthoringDTO.retries}&amp;reflect=${mcGeneralAuthoringDTO.reflect}&amp;reflectionSubject=${mcGeneralAuthoringDTO.reflectionSubject}&amp;passmark=${mcGeneralAuthoringDTO.passMarkValue}"
+								paramId="uuid" paramName="attachment" paramProperty="uuid"
+								onclick="javascript:return confirm('Are you sure you want to delete this file?')"
+								target="_self">
+								<fmt:message key="label.delete" />
+							</html:link>
+						</li>
+					</c:if>
+				</c:forEach>
+			</ul>
 
-									<c:forEach var="attachment" items="${mcGeneralAuthoringDTO.attachmentList}">
-		
-											<c:if test="${attachment.fileOnline == false}"> 			
-								            	<bean:define id="view">/download/?uuid=<bean:write name="attachment" property="uuid"/>&preferDownload=false</bean:define>
-												<bean:define id="download">/download/?uuid=<bean:write name="attachment" property="uuid"/>&preferDownload=true</bean:define>
-						                        <bean:define id="uuid" name="attachment" property="uuid" />
-						                        
-						                        <tr>
-									            	<td> <bean:write name="attachment" property="fileName"/>  </td>
-										            <td>
-											        	<table>
-												        	<tr>
-												            	<td>
-												                	<a href='javascript:launchInstructionsPopup("<html:rewrite page='<%=view%>'/>")' class="button">
-												                   		<fmt:message key="label.view" />
-												                    </a>
-																	&nbsp&nbsp
-													            	<html:link page="<%=download%>" styleClass="button">
-													                	<fmt:message key="label.download" />
-													                </html:link>
-																	&nbsp&nbsp
-													            	<html:link page="/authoring.do?dispatch=deleteFile&httpSessionID=${mcGeneralAuthoringDTO.httpSessionID}&toolContentID=${mcGeneralAuthoringDTO.toolContentID}&contentFolderID=${mcGeneralAuthoringDTO.contentFolderID}&activeModule=${mcGeneralAuthoringDTO.activeModule}&defaultContentIdStr=${mcGeneralAuthoringDTO.defaultContentIdStr}&sln=${mcGeneralAuthoringDTO.sln}&questionsSequenced=${mcGeneralAuthoringDTO.questionsSequenced}&retries=${mcGeneralAuthoringDTO.retries}&reflect=${mcGeneralAuthoringDTO.reflect}&reflectionSubject=${mcGeneralAuthoringDTO.reflectionSubject}&passmark=${mcGeneralAuthoringDTO.passMarkValue}"
-													            	paramId="uuid" paramName="attachment" paramProperty="uuid"
-													                         	onclick="javascript:return confirm('Are you sure you want to delete this file?')"
-													                         	target="_self" styleClass="button">
-													                	<fmt:message key="label.delete" />
-													                </html:link> 
-													            </td>
-												           	</tr>
-											            </table>
-										           	</td>
-									   	     	</tr>
-										</c:if> 											   	     	
-											</c:forEach>
-										</table>
-								 	</td>
-								</tr>
-							</table>
+		</td>
+	</tr>
 
-							 	</td>
-							</tr>
-				</table>
-					</td> 				
-				</tr>
 
+	<tr>
+		<td>
+			<div class="field-name">
+				<fmt:message key="label.onlineFiles" />
+			</div>
+			<html:file property="theOnlineFile"></html:file>
+			<html:submit property="submitOnlineFile" styleClass="button"
+				onclick="submitMethod('addNewFile');">
+				<fmt:message key="label.upload" />
+			</html:submit>
+		</td>
 
+	</tr>
 
-				<tr> 
-					<td class="field-name"> 
-          				<fmt:message key="label.offlineFiles" />
-          			</td>
-          			<td NOWRAP> 
-						<html:file  property="theOfflineFile"></html:file>
-					 	<html:submit property="submitOfflineFile" 
-                                     styleClass="linkbutton" 
-                                     onclick="submitMethod('addNewFile');">
-								<fmt:message key="label.upload"/>
-						</html:submit>
-					</td> 
-				</tr>          		
-			</table>	  	
-	
+	<tr>
+		<td>
+			<hr />
+		</td>
+	</tr>
 
+	<tr>
+		<td colspan="2">
+			<div class="field-name-alternative-color">
+				<fmt:message key="label.offlineInstructions.col"></fmt:message>
+			</div>
+			<html:textarea property="offlineInstructions" rows="3" cols="75"></html:textarea>
+		</td>
+	</tr>
 
-	
-	
+	<tr>
+		<td>
+			<ul>
+				<c:forEach var="attachment"
+					items="${mcGeneralAuthoringDTO.attachmentList}">
 
+					<c:if test="${attachment.fileOnline == false}">
+						<li>
+							<bean:define id="view">/download/?uuid=<bean:write
+									name="attachment" property="uuid" />&preferDownload=false</bean:define>
+							<bean:define id="download">/download/?uuid=<bean:write
+									name="attachment" property="uuid" />&preferDownload=true</bean:define>
+							<bean:define id="uuid" name="attachment" property="uuid" />
 
+							<bean:write name="attachment" property="fileName" />
 
+							<a
+								href='javascript:launchInstructionsPopup("<html:rewrite page='<%=view%>'/>")'>
+								<fmt:message key="label.view" /> </a>
 
+							<html:link page="<%=download%>">
+								<fmt:message key="label.download" />
+							</html:link>
+
+							<html:link
+								page="/authoring.do?dispatch=deleteFile&amp;httpSessionID=${mcGeneralAuthoringDTO.httpSessionID}&amp;toolContentID=${mcGeneralAuthoringDTO.toolContentID}&amp;contentFolderID=${mcGeneralAuthoringDTO.contentFolderID}&amp;activeModule=${mcGeneralAuthoringDTO.activeModule}&amp;defaultContentIdStr=${mcGeneralAuthoringDTO.defaultContentIdStr}&amp;sln=${mcGeneralAuthoringDTO.sln}&amp;questionsSequenced=${mcGeneralAuthoringDTO.questionsSequenced}&amp;retries=${mcGeneralAuthoringDTO.retries}&amp;reflect=${mcGeneralAuthoringDTO.reflect}&amp;reflectionSubject=${mcGeneralAuthoringDTO.reflectionSubject}&amp;passmark=${mcGeneralAuthoringDTO.passMarkValue}"
+								paramId="uuid" paramName="attachment" paramProperty="uuid"
+								onclick="javascript:return confirm('Are you sure you want to delete this file?')"
+								target="_self">
+								<fmt:message key="label.delete" />
+							</html:link>
+						</li>
+					</c:if>
+				</c:forEach>
+			</ul>
+		</td>
+	</tr>
+
+	<tr>
+		<td>
+			<div class="field-name-alternative-color">
+				<fmt:message key="label.offlineFiles" />
+			</div>
+			<html:file property="theOfflineFile"></html:file>
+			<html:submit property="submitOfflineFile" styleClass="button"
+				onclick="submitMethod('addNewFile');">
+				<fmt:message key="label.upload" />
+			</html:submit>
+		</td>
+	</tr>
+</table>