Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java,v
diff -u -r1.41 -r1.42
--- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java	14 Jun 2006 06:02:58 -0000	1.41
+++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java	5 Jul 2006 01:45:43 -0000	1.42
@@ -520,6 +520,12 @@
 		}
 		StringBuffer sb = new StringBuffer();
 		iter = leftAttachments.iterator();
+		
+		boolean hasAtts = iter.hasNext();
+		
+		if (hasAtts) {
+			sb.append("<ul>");
+		}		
 		while(iter.hasNext()){
 			Attachment file = (Attachment) iter.next();
 			sb.append("<li>").append(file.getFileName()).append("\r\n");
@@ -550,6 +556,9 @@
 				sb.append(this.getResources(request).getMessage("label.authoring.online.delete"));
 			sb.append("</a></li>\r\n");
 		}
+		if (hasAtts) {
+			sb.append("<ul>");
+		}		
 		try {
 			PrintWriter out = response.getWriter();
 			out.print(sb.toString());
Index: lams_tool_forum/web/jsps/authoring/instructions.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/authoring/Attic/instructions.jsp,v
diff -u -r1.11 -r1.12
--- lams_tool_forum/web/jsps/authoring/instructions.jsp	29 Jun 2006 08:49:23 -0000	1.11
+++ lams_tool_forum/web/jsps/authoring/instructions.jsp	5 Jul 2006 01:45:43 -0000	1.12
@@ -2,114 +2,118 @@
 
 <!--   Instruction Tab Content -->
 
-<table>
+<table cellpadding="0">
 	<!-- Instructions Row -->
 	<tr>
-		<td colspan="2">
-			<lams:SetEditor id="forum.onlineInstructions" small="true" text="${forumForm.forum.onlineInstructions}"  key="label.authoring.online.instruction" />
-		</td>
-	</tr>
-	<tr>
-		<td></td>
 		<td>
-
-			<div id="onlinefile">
-				<c:forEach var="file" items="${forumForm.onlineFileList}">
-					<li>
-						<c:out value="${file.fileName}" />
-						<c:set var="viewURL">
-							<html:rewrite page="/download/?uuid=${file.fileUuid}&preferDownload=false" />
-						</c:set>
-						<a href="javascript:launchInstructionsPopup('<c:out value='${viewURL}' escapeXml='false'/>')">
-							<fmt:message key="label.view" />
-						</a>
-						&nbsp;
-						<c:set var="downloadURL">
-							<html:rewrite page="/download/?uuid=${file.fileUuid}&versionID=${file.fileVersionId}&preferDownload=true" />
-						</c:set>
-						<a href="<c:out value='${downloadURL}' escapeXml='false'/>">
-							<fmt:message key="label.download" />
-						</a>
-						&nbsp;
-						<c:set var="deleteonline">
-							<html:rewrite page="/authoring/deleteOnline.do?toolContentID=${forumForm.toolContentID}&uuID=${file.fileUuid}&versionID=${file.fileVersionId}" />
-						</c:set>
-						<html:link href="javascript:loadDoc('${deleteonline}','onlinefile')">
-							<fmt:message key="label.authoring.online.delete" />
-						</html:link>
-					</li>
-				</c:forEach>
-			</div>
+			<lams:SetEditor id="forum.onlineInstructions" small="true" text="${forumForm.forum.onlineInstructions}" key="label.authoring.online.instruction" />
 		</td>
 	</tr>
+
+	<c:if test="${not empty forumForm.onlineFileList}">
+		<tr>
+			<td>
+				<div id="onlinefile">
+					<ul>
+						<c:forEach var="file" items="${forumForm.onlineFileList}">
+							<li>
+								<c:out value="${file.fileName}" />
+								<c:set var="viewURL">
+									<html:rewrite page="/download/?uuid=${file.fileUuid}&preferDownload=false" />
+								</c:set>
+								<a href="javascript:launchInstructionsPopup('<c:out value='${viewURL}' escapeXml='false'/>')"> <fmt:message key="label.view" /> </a> &nbsp;
+								<c:set var="downloadURL">
+									<html:rewrite page="/download/?uuid=${file.fileUuid}&versionID=${file.fileVersionId}&preferDownload=true" />
+								</c:set>
+								<a href="<c:out value='${downloadURL}' escapeXml='false'/>"> <fmt:message key="label.download" /> </a> &nbsp;
+								<c:set var="deleteonline">
+									<html:rewrite page="/authoring/deleteOnline.do?toolContentID=${forumForm.toolContentID}&uuID=${file.fileUuid}&versionID=${file.fileVersionId}" />
+								</c:set>
+								<html:link href="javascript:loadDoc('${deleteonline}','onlinefile')">
+									<fmt:message key="label.authoring.online.delete" />
+								</html:link>
+							</li>
+						</c:forEach>
+					</ul>
+				</div>
+			</td>
+		</tr>
+	</c:if>
 	<tr>
-		<td  class="field-name">
+		<td class="field-name">
 			<fmt:message key="label.authoring.online.file" />
-			:
 		</td>
+	</tr>
+	<tr>
 		<td>
 			<html:file property="onlineFile">
 				<fmt:message key="label.authoring.choosefile.button" />
 			</html:file>
 			<c:set var="uploadonline">
 				<html:rewrite page="/authoring/uploadOnline.do" />
 			</c:set>
-			<a href="#" onclick="doSubmit('${uploadonline}')" class="button">
-				<fmt:message key="label.authoring.upload.online.button" />
-			</a>
+			<a href="#" onclick="doSubmit('${uploadonline}')" class="button"> <fmt:message key="label.authoring.upload.online.button" /> </a>
 		</td>
 	</tr>
+</table>
+
+<hr />
+
+<table>
 	<!-- Offline Instructions -->
 	<tr>
-		<td colspan="2">
-			<lams:SetEditor id="forum.offlineInstructions" small="true"  text="${forumForm.forum.offlineInstructions}" key="label.authoring.offline.instruction" />
-		</td>
-	</tr>
-	<tr>
-		<td></td>
 		<td>
-			<div id="offlinefile">
-				<c:forEach var="file" items="${forumForm.offlineFileList}">
-					<li>
-						<c:out value="${file.fileName}" />
-						<c:set var="viewURL">
-							<html:rewrite page="/download/?uuid=${file.fileUuid}&preferDownload=false" />
-						</c:set>
-						<html:link href="javascript:launchInstructionsPopup('${viewURL}')">
-							<fmt:message key="label.view" />
-						</html:link>
-						<c:set var="downloadURL">
-							<html:rewrite page="/download/?uuid=${file.fileUuid}&versionID=${file.fileVersionId}&preferDownload=true" />
-						</c:set>
-						<html:link href="${downloadURL}">
-							<fmt:message key="label.download" />
-						</html:link>
-						<c:set var="deleteoffline">
-							<html:rewrite page="/authoring/deleteOffline.do?toolContentID=${forumForm.toolContentID}&uuID=${file.fileUuid}&versionID=${file.fileVersionId}" />
-						</c:set>
-						<html:link href="javascript:loadDoc('${deleteoffline}','offlinefile')">
-							<fmt:message key="label.authoring.offline.delete" />
-						</html:link>
-					</li>
-				</c:forEach>
-			</div>
+			<lams:SetEditor id="forum.offlineInstructions" small="true" text="${forumForm.forum.offlineInstructions}" key="label.authoring.offline.instruction" alt="true" />
 		</td>
 	</tr>
+	<c:if test="${not empty forumForm.offlineFileList}">
+		<tr>
+			<td>
+				<div id="offlinefile">
+					<ul>
+						<c:forEach var="file" items="${forumForm.offlineFileList}">
+							<li>
+								<c:out value="${file.fileName}" />
+								<c:set var="viewURL">
+									<html:rewrite page="/download/?uuid=${file.fileUuid}&preferDownload=false" />
+								</c:set>
+								<html:link href="javascript:launchInstructionsPopup('${viewURL}')">
+									<fmt:message key="label.view" />
+								</html:link>
+								<c:set var="downloadURL">
+									<html:rewrite page="/download/?uuid=${file.fileUuid}&versionID=${file.fileVersionId}&preferDownload=true" />
+								</c:set>
+								<html:link href="${downloadURL}">
+									<fmt:message key="label.download" />
+								</html:link>
+								<c:set var="deleteoffline">
+									<html:rewrite page="/authoring/deleteOffline.do?toolContentID=${forumForm.toolContentID}&uuID=${file.fileUuid}&versionID=${file.fileVersionId}" />
+								</c:set>
+								<html:link href="javascript:loadDoc('${deleteoffline}','offlinefile')">
+									<fmt:message key="label.authoring.offline.delete" />
+								</html:link>
+							</li>
+						</c:forEach>
+					</ul>
+				</div>
+			</td>
+		</tr>
+	</c:if>
+
 	<tr>
-		<td class="field-name">
+		<td class="field-name-alternative-color">
 			<fmt:message key="label.authoring.offline.file" />
-			:
 		</td>
+	</tr>
+	<tr>
 		<td>
 			<html:file property="offlineFile">
 				<fmt:message key="label.authoring.choosefile.button" />
 			</html:file>
 			<c:set var="uploadoffline">
 				<html:rewrite page="/authoring/uploadOffline.do" />
 			</c:set>
-			<a href="#" onclick="doSubmit('${uploadoffline}')" class="button">
-				<fmt:message key="label.authoring.upload.offline.button" />
-			</a>
+			<a href="#" onclick="doSubmit('${uploadoffline}')" class="button"> <fmt:message key="label.authoring.upload.offline.button" /> </a>
 		</td>
 	</tr>
 </table>