Index: lams_tool_sbmt/src/java/org/lamsfoundation/lams/tool/sbmt/form/AuthoringForm.java
===================================================================
diff -u -r17b9effc3c7ea57333aa800fff6d511689c32329 -rb3f41153ea9815528aed7730b3760cc11f48bf75
--- lams_tool_sbmt/src/java/org/lamsfoundation/lams/tool/sbmt/form/AuthoringForm.java	(.../AuthoringForm.java)	(revision 17b9effc3c7ea57333aa800fff6d511689c32329)
+++ lams_tool_sbmt/src/java/org/lamsfoundation/lams/tool/sbmt/form/AuthoringForm.java	(.../AuthoringForm.java)	(revision b3f41153ea9815528aed7730b3760cc11f48bf75)
@@ -1,22 +1,19 @@
 package org.lamsfoundation.lams.tool.sbmt.form;
 
-import java.lang.reflect.InvocationTargetException;
 import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Set;
 
 import javax.servlet.http.HttpServletRequest;
 
-import org.apache.commons.beanutils.PropertyUtils;
 import org.apache.commons.lang.StringUtils;
 import org.apache.struts.action.ActionMapping;
 import org.apache.struts.upload.FormFile;
 import org.apache.struts.validator.ValidatorForm;
 import org.lamsfoundation.lams.contentrepository.client.IToolContentHandler;
 import org.lamsfoundation.lams.tool.sbmt.InstructionFiles;
 import org.lamsfoundation.lams.tool.sbmt.SubmitFilesContent;
-import org.lamsfoundation.lams.tool.sbmt.dto.DTOException;
 
 /**
  * 
@@ -26,6 +23,8 @@
 public class AuthoringForm extends ValidatorForm {
 
 	private Long toolContentID;
+	private String contentFolderID;
+	
 	//control fields
 	private String sessionMapID;
 	private String currentTab;
@@ -172,4 +171,10 @@
 	public void setTitle(String title) {
 		this.title = title;
 	}
+	public String getContentFolderID() {
+		return contentFolderID;
+	}
+	public void setContentFolderID(String contentFolderID) {
+		this.contentFolderID = contentFolderID;
+	}
 }
Index: lams_tool_sbmt/src/java/org/lamsfoundation/lams/tool/sbmt/web/AuthoringAction.java
===================================================================
diff -u -rc261f143be0756641bb0423662c0175aeb91ed7b -rb3f41153ea9815528aed7730b3760cc11f48bf75
--- lams_tool_sbmt/src/java/org/lamsfoundation/lams/tool/sbmt/web/AuthoringAction.java	(.../AuthoringAction.java)	(revision c261f143be0756641bb0423662c0175aeb91ed7b)
+++ lams_tool_sbmt/src/java/org/lamsfoundation/lams/tool/sbmt/web/AuthoringAction.java	(.../AuthoringAction.java)	(revision b3f41153ea9815528aed7730b3760cc11f48bf75)
@@ -92,6 +92,7 @@
 			HttpServletRequest request, HttpServletResponse response) {
 
 		ToolAccessMode mode = SbmtWebUtils.getAccessMode(request);
+		
 		//when first time open flash icon on authoring page: mode will be null 
 		if(mode == null)
 			mode = ToolAccessMode.AUTHOR;
@@ -100,6 +101,7 @@
 		request.getSession().setAttribute(sessionMap.getSessionID(), sessionMap);
 		
 		Long contentID = new Long(WebUtil.readLongParam(request,AttributeNames.PARAM_TOOL_CONTENT_ID));
+		String contentFolderID = WebUtil.readStrParam(request, AttributeNames.PARAM_CONTENT_FOLDER_ID);
 		//get back the upload file list and display them on page
 		submitFilesService = getService();
 		
@@ -137,6 +139,7 @@
 		authForm.setSessionMapID(sessionMap.getSessionID());
 		//current tab
 		authForm.setCurrentTab(request.getParameter("currentTab"));
+		authForm.setContentFolderID(contentFolderID);
 		
 		return mapping.findForward("success");
 	}
Index: lams_tool_sbmt/web/WEB-INF/fckeditor/tlds/FCKeditor.tld
===================================================================
diff -u -rd2314c176ba54c65b43815825e21b4d36b423a9b -rb3f41153ea9815528aed7730b3760cc11f48bf75
--- lams_tool_sbmt/web/WEB-INF/fckeditor/tlds/FCKeditor.tld	(.../FCKeditor.tld)	(revision d2314c176ba54c65b43815825e21b4d36b423a9b)
+++ lams_tool_sbmt/web/WEB-INF/fckeditor/tlds/FCKeditor.tld	(.../FCKeditor.tld)	(revision b3f41153ea9815528aed7730b3760cc11f48bf75)
@@ -14,6 +14,7 @@
 		<attribute>
 			<name>id</name>
 			<required>true</required>
+			<rtexprvalue>true</rtexprvalue>
   		</attribute>
 		<attribute>
 			<name>basePath</name>
Index: lams_tool_sbmt/web/WEB-INF/lams.tld
===================================================================
diff -u -r3598d5bbce42e145fc46a33d70a7bd035b71d8bc -rb3f41153ea9815528aed7730b3760cc11f48bf75
--- lams_tool_sbmt/web/WEB-INF/lams.tld	(.../lams.tld)	(revision 3598d5bbce42e145fc46a33d70a7bd035b71d8bc)
+++ lams_tool_sbmt/web/WEB-INF/lams.tld	(.../lams.tld)	(revision b3f41153ea9815528aed7730b3760cc11f48bf75)
@@ -287,14 +287,10 @@
        <path>/WEB-INF/tags/TabBody.tag</path>
    	</tag-file>
    	<tag-file>
-       <name>HTMLEditor</name>
-       <path>/WEB-INF/tags/HTMLEditor.tag</path>
+       <name>FCKEditor</name>
+       <path>/WEB-INF/tags/FCKEditor.tag</path>
    	</tag-file>
    	<tag-file>
-       <name>SetEditor</name>
-       <path>/WEB-INF/tags/SetEditor.tag</path>
-   	</tag-file>
-   	<tag-file>
        <name>AuthoringButton</name>
        <path>/WEB-INF/tags/AuthoringButton.tag</path>
    	</tag-file>   	
@@ -310,14 +306,8 @@
 	    <name>ExportPortOutput</name>
 		<path>/WEB-INF/tags/ExportPortOutput.tag</path>
    	</tag-file>  
-   	<tag-file>
+	<tag-file>
 	    <name>Date</name>
 		<path>/WEB-INF/tags/Date.tag</path>
    	</tag-file>  
-	<tag-file>
-	    <name>DefineLater</name>
-		<path>/WEB-INF/tags/DefineLater.tag</path>
-   	</tag-file>  
-   	
-   	
 </taglib>
Index: lams_tool_sbmt/web/WEB-INF/tags/AuthoringButton.tag
===================================================================
diff -u -r17b9effc3c7ea57333aa800fff6d511689c32329 -rb3f41153ea9815528aed7730b3760cc11f48bf75
--- lams_tool_sbmt/web/WEB-INF/tags/AuthoringButton.tag	(.../AuthoringButton.tag)	(revision 17b9effc3c7ea57333aa800fff6d511689c32329)
+++ lams_tool_sbmt/web/WEB-INF/tags/AuthoringButton.tag	(.../AuthoringButton.tag)	(revision b3f41153ea9815528aed7730b3760cc11f48bf75)
@@ -37,6 +37,7 @@
 <%@ attribute name="formID" required="true" rtexprvalue="true" %>
 <%@ attribute name="toolSignature" required="true" rtexprvalue="true" %>
 <%@ attribute name="toolContentID" required="true" rtexprvalue="true" %>
+<%@ attribute name="contentFolderID" required="true" rtexprvalue="true" %>
 <%@ attribute name="clearSessionActionUrl" required="true" rtexprvalue="true" %>
 
 <%-- Optional attribute --%>
@@ -64,7 +65,7 @@
 <!-- begin tab content -->
 <script type="text/javascript">
 	if(<c:choose><c:when test="${LAMS_AUTHORING_SUCCESS_FLAG == true}">true</c:when><c:otherwise>false</c:otherwise></c:choose>){
-       	location.href="<c:url value='${clearSessionActionUrl}?action=confirm&mode=${accessMode}&signature=${toolSignature}&toolContentID=${toolContentID}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}'/>";
+       	location.href="<c:url value='${clearSessionActionUrl}?action=confirm&mode=${accessMode}&signature=${toolSignature}&toolContentID=${toolContentID}&defineLater=${defineLater}&customiseSessionID=${customiseSessionID}&contentFolderID=${contentFolderID}'/>";
 	}
     function doSubmit_Form_Only() {
     	document.getElementById("${formID}").submit();
@@ -81,7 +82,7 @@
 		}
     }  				
 </script>				
-<p align="right">
+<p style="text-align: right">
 	<html:link href="javascript:doSubmit_Form_Only();" property="submit" styleClass="button">
 		<fmt:message key="${saveButtonLabelKey}" /> 
 	</html:link>
Index: lams_tool_sbmt/web/WEB-INF/tags/FCKEditor.tag
===================================================================
diff -u
--- lams_tool_sbmt/web/WEB-INF/tags/FCKEditor.tag	(revision 0)
+++ lams_tool_sbmt/web/WEB-INF/tags/FCKEditor.tag	(revision b3f41153ea9815528aed7730b3760cc11f48bf75)
@@ -0,0 +1,57 @@
+<%@ taglib uri="tags-core" prefix="c"%>
+<%@ taglib uri="tags-lams" prefix="lams"%>
+<%@ taglib uri="fck-editor" prefix="fck"%>
+
+<%@ attribute name="id" required="true" rtexprvalue="true"%>
+<%@ attribute name="value" required="true" rtexprvalue="true"%>
+<%@ attribute name="toolbarSet" required="false" rtexprvalue="true"%>
+<%@ attribute name="contentFolderID" required="true" rtexprvalue="true"%>
+
+<c:set var="language">
+	<lams:user property="localeLanguage" />
+</c:set>
+
+<c:if test="${empty toolbarSet}">
+	<c:set var="toolbarSet" value="Default" />
+</c:if>
+
+<c:set var="basePath" value="/lams/fckeditor/" />
+
+<!-- 
+<script type="text/javascript">
+	var oFCKeditor=new FCKeditor('${id}');
+	oFCKeditor.BasePath= "${basePath}";
+	oFCKeditor.ToolbarSet= "${toolbarSet}";
+	oFCKeditor.Value= '${value}';
+
+	oFCKeditor.Config["ImageBrowserURL"]= "${basePath}editor/filemanager/browser/default/browser.html?Type=Image&amp;Connector=connectors/jsp/connector&amp;CurrentFolder=/${contentFolderID}/";
+	oFCKeditor.Config["LinkBrowserURL"]=  "${basePath}editor/filemanager/browser/default/browser.html?Connector=connectors/jsp/connector&amp;CurrentFolder=/${contentFolderID}/";
+	oFCKeditor.Config["FlashBrowserURL"]= "${basePath}editor/filemanager/browser/default/browser.html?Type=Flash&amp;Connector=connectors/jsp/connector&amp;CurrentFolder=/${contentFolderID}/";
+	oFCKeditor.Config["ImageUploadURL"]=  "${basePath}editor/filemanager/upload/simpleuploader?Type=Image&CurrentFolder=/${contentFolderID}/";
+	oFCKeditor.Config["LinkUploadURL"]=   "${basePath}editor/filemanager/upload/simpleuploader?Type=File&CurrentFolder=/${contentFolderID}/";
+	oFCKeditor.Config["FlashUploadURL"]=  "${basePath}editor/filemanager/upload/simpleuploader?Type=Flash&CurrentFolder=/${contentFolderID}/";
+
+	oFCKeditor.Config["AutoDetectLanguage"]= false ;
+	oFCKeditor.Config["DefaultLanguage"]= "${language}" ;
+
+	oFCKeditor.Create();
+</script>
+-->
+
+<fck:editor id="${id}"
+	basePath="${basePath}"
+	toolbarSet="${toolbarSet}"
+	imageBrowserURL="${basePath}editor/filemanager/browser/default/browser.html?Type=Image&Connector=connectors/jsp/connector&CurrentFolder=/${contentFolderID}/"
+	imageUploadURL="${basePath}editor/filemanager/upload/simpleuploader?Type=Image&CurrentFolder=/${contentFolderID}/"
+	linkBrowserURL="${basePath}editor/filemanager/browser/default/browser.html?Connector=connectors/jsp/connector&CurrentFolder=/${contentFolderID}/"
+	linkUploadURL="${basePath}editor/filemanager/upload/simpleuploader?Type=File&CurrentFolder=/${contentFolderID}/"
+	flashBrowserURL="${basePath}editor/filemanager/browser/default/browser.html?Type=Flash&Connector=connectors/jsp/connector&CurrentFolder=/${contentFolderID}/"
+	flashUploadURL="${basePath}editor/filemanager/upload/simpleuploader?Type=Flash&CurrentFolder=/${contentFolderID}/"
+	autoDetectLanguage="false"
+	defaultLanguage="${language}">
+		${value}
+</fck:editor>
+
+
+
+
Fisheye: Tag b3f41153ea9815528aed7730b3760cc11f48bf75 refers to a dead (removed) revision in file `lams_tool_sbmt/web/WEB-INF/tags/HTMLEditor.tag'.
Fisheye: No comparison available.  Pass `N' to diff?
Fisheye: Tag b3f41153ea9815528aed7730b3760cc11f48bf75 refers to a dead (removed) revision in file `lams_tool_sbmt/web/WEB-INF/tags/SetEditor.tag'.
Fisheye: No comparison available.  Pass `N' to diff?
Index: lams_tool_sbmt/web/authoring/authoring.jsp
===================================================================
diff -u -r671e0f31315ba99ec13f82789869ce2a01e0ea04 -rb3f41153ea9815528aed7730b3760cc11f48bf75
--- lams_tool_sbmt/web/authoring/authoring.jsp	(.../authoring.jsp)	(revision 671e0f31315ba99ec13f82789869ce2a01e0ea04)
+++ lams_tool_sbmt/web/authoring/authoring.jsp	(.../authoring.jsp)	(revision b3f41153ea9815528aed7730b3760cc11f48bf75)
@@ -1,5 +1,5 @@
-<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
-        "http://www.w3.org/TR/html4/strict.dtd">
+<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
+            "http://www.w3.org/TR/html4/loose.dtd">
 
 <%@include file="/common/taglibs.jsp"%>
 <c:set var="lams">
@@ -35,10 +35,6 @@
             else
                 selectTab(1); //select the default tab;
             
-            initEditor("Title");
-            initEditor("Instructions");
-            initEditor("OnlineInstruction");
-            initEditor("OfflineInstruction");
         }     
         
         function doSelectTab(tabId) {
@@ -85,6 +81,7 @@
 				<input type="hidden" name="mode" value="author">
 				<html:hidden property="sessionMapID" />
 				<html:hidden property="toolContentID" />
+				<html:hidden property="contentFolderID" />
 				<html:hidden property="currentTab" styleId="currentTab" />
 				<html:hidden property="dispatch" value="updateContent" />
 
@@ -102,9 +99,8 @@
 
 				<lams:AuthoringButton formID="authoringForm" clearSessionActionUrl="/clearsession.do" 
 					toolSignature="<%=SbmtConstants.TOOL_SIGNATURE%>" toolContentID="${formBean.toolContentID}" 
-					customiseSessionID="${formBean.sessionMapID}" />
-
-				<lams:HTMLEditor />
+					customiseSessionID="${formBean.sessionMapID}" 
+					contentFolderID="abc"/>
 			</html:form>
 		</div>
 		<div id="footer"></div>
Index: lams_tool_sbmt/web/authoring/basic.jsp
===================================================================
diff -u -r17b9effc3c7ea57333aa800fff6d511689c32329 -rb3f41153ea9815528aed7730b3760cc11f48bf75
--- lams_tool_sbmt/web/authoring/basic.jsp	(.../basic.jsp)	(revision 17b9effc3c7ea57333aa800fff6d511689c32329)
+++ lams_tool_sbmt/web/authoring/basic.jsp	(.../basic.jsp)	(revision b3f41153ea9815528aed7730b3760cc11f48bf75)
@@ -1,16 +1,26 @@
 <%@include file="/common/taglibs.jsp"%>
-<c:set var="formBean" value="<%= request.getAttribute(org.apache.struts.taglib.html.Constants.BEAN_KEY) %>" />
+<c:set var="formBean"
+	value="<%= request.getAttribute(org.apache.struts.taglib.html.Constants.BEAN_KEY) %>" />
 
 <!-- Basic Tab Content -->
 <table cellpadding="0">
 	<tr>
 		<td>
-			<lams:SetEditor id="Title" text="${formBean.title}" small="true" key="label.authoring.basic.title" />
+			<div class="field-name" style="text-align: left;">
+				<fmt:message key="label.authoring.basic.title"></fmt:message>
+			</div>
+			<html:text property="title" style="width: 100%;"></html:text>
 		</td>
 	</tr>
 	<tr>
 		<td>
-			<lams:SetEditor id="Instructions" text="${formBean.instructions}" key="label.authoring.basic.instruction" />
+			<div class="field-name" style="text-align: left;">
+				<fmt:message key="label.authoring.basic.instruction"></fmt:message>
+			</div>
+			<lams:FCKEditor id="instructions"
+				value="${formBean.instructions}"
+				contentFolderID="${formBean.contentFolderID}">
+			</lams:FCKEditor>
 		</td>
 	</tr>
 </table>
Index: lams_tool_sbmt/web/authoring/instructions.jsp
===================================================================
diff -u -r671e0f31315ba99ec13f82789869ce2a01e0ea04 -rb3f41153ea9815528aed7730b3760cc11f48bf75
--- lams_tool_sbmt/web/authoring/instructions.jsp	(.../instructions.jsp)	(revision 671e0f31315ba99ec13f82789869ce2a01e0ea04)
+++ lams_tool_sbmt/web/authoring/instructions.jsp	(.../instructions.jsp)	(revision b3f41153ea9815528aed7730b3760cc11f48bf75)
@@ -1,8 +1,9 @@
-<%@ include file="/common/taglibs.jsp" %>
-<c:set var="formBean" value="<%= request.getAttribute(org.apache.struts.taglib.html.Constants.BEAN_KEY) %>" />
-<c:set var="sessionMapID" value="${formBean.sessionMapID}"/>
+<%@ include file="/common/taglibs.jsp"%>
+<c:set var="formBean"
+	value="<%= request.getAttribute(org.apache.struts.taglib.html.Constants.BEAN_KEY) %>" />
+<c:set var="sessionMapID" value="${formBean.sessionMapID}" />
 
-<script lang="javascript">
+<script type="text/javascript">
 <!-- Common Javascript functions for LAMS -->
 	var offlineArea = "offlinefileArea";
 	var onlineArea = "onlinefileArea";
@@ -52,65 +53,82 @@
 		hideBusy(onlineArea);
 	}
 </script>
-	<!-- Instruction Tab Content -->
-	<table class="forms">
-		<!--hidden field contentID passed by flash-->
-		<tr>
-		</tr>
-		<!-- Instructions Row -->
-		<tr>
-			<td colspan="2">
-				<lams:SetEditor id="onlineInstruction" small="true" text="${formBean.onlineInstruction}" key="label.authoring.online.instruction" />
-			</td>
-		</tr>
-		<tr>
-			<td colspan="2">
-				<c:set var="fileTypeFlag" value="<%=IToolContentHandler.TYPE_ONLINE %>"/>
-				<div id="onlinefileArea">
-					<%@ include file="parts/instructionfilelist.jsp" %>
-				</div>
-			</td>
-		</tr>
-		<tr>
-			<td nowrap="nowrap" class="field-name">
-				<fmt:message key="label.authoring.online.file" />:
-			</td>
-			<td>
-				<html:file property="onlineFile">
-					<fmt:message key="label.authoring.choosefile.button" />
-				</html:file>
-				<a href="#" onclick="javascript:doSubmit('uploadOnline');" class="button">
-					<fmt:message key="label.authoring.upload.online.button" />
-				</a>
-			</td>
-		</tr>	
-		<!-- Offline Instructions -->
-		<tr>
-			<td colspan="2">
-			<lams:SetEditor id="offlineInstruction" small="true"  text="${formBean.offlineInstruction}" key="label.authoring.offline.instruction"/>
-			</td>
-		</tr>
-		<tr>
-			<td colspan="2">
-				<c:set var="fileTypeFlag" value="<%=IToolContentHandler.TYPE_OFFLINE %>"/>
-				<div id="offlinefileArea">
-					<%@ include file="parts/instructionfilelist.jsp" %>
-				</div>
-			</td>
-		</tr>
-		<tr>
-			<td nowrap="nowrap" valign="top" class="field-name">
-				<fmt:message key="label.authoring.offline.file" />:
-			</td>
-			<td valign="top">
-				<html:file property="offlineFile">
-					<fmt:message key="label.authoring.choosefile.button" />
-				</html:file>
-				<a href="#" onclick="javascript:doSubmit('uploadOffline');" class="button">
-					<fmt:message key="label.authoring.upload.offline.button" />
-				</a>
-				<p>&nbsp;</p>
-			</td>
-		</tr>			
-		</table>
-		
\ No newline at end of file
+<!-- Instruction Tab Content -->
+<table class="forms">
+	<!-- Instructions Row -->
+	<tr>
+		<td>
+			<div class="field-name" style="text-align: left;">
+				<fmt:message key="label.authoring.online.instruction"></fmt:message>
+			</div>
+			<html:textarea property="onlineInstruction" cols="80" rows="3"></html:textarea>
+		</td>
+	</tr>
+	<tr>
+		<td>
+			<c:set var="fileTypeFlag"
+				value="<%=IToolContentHandler.TYPE_ONLINE %>" />
+			<div id="onlinefileArea">
+				<%@ include file="parts/instructionfilelist.jsp"%>
+			</div>
+		</td>
+	</tr>
+	<tr>
+		<td class="field-name">
+			<div class="field-name" style="text-align: left">
+				<fmt:message key="label.authoring.online.file" />
+			</div>
+		</td>
+	</tr>
+	<tr>
+		<td>
+			<html:file property="onlineFile">
+				<fmt:message key="label.authoring.choosefile.button" />
+			</html:file>
+			<a href="#" onclick="javascript:doSubmit('uploadOnline');"
+				class="button"> <fmt:message
+					key="label.authoring.upload.online.button" /> </a>
+		</td>
+	</tr>
+</table>
+
+<hr />
+
+<table>
+	<!-- Offline Instructions -->
+	<tr>
+		<td>
+			<div class="field-name-alternative-color" style="text-align: left;">
+				<fmt:message key="label.authoring.offline.instruction"></fmt:message>
+			</div>
+			<html:textarea property="offlineInstruction" cols="80" rows="3"></html:textarea>
+		</td>
+	</tr>
+	<tr>
+		<td>
+			<c:set var="fileTypeFlag"
+				value="<%=IToolContentHandler.TYPE_OFFLINE %>" />
+			<div id="offlinefileArea">
+				<%@ include file="parts/instructionfilelist.jsp"%>
+			</div>
+		</td>
+	</tr>
+	<tr>
+		<td>
+			<div class="field-name-alternative-color" style="text-align: left">
+				<fmt:message key="label.authoring.offline.file" />
+			</div>
+		</td>
+	</tr>
+	<tr>
+		<td>
+			<html:file property="offlineFile">
+				<fmt:message key="label.authoring.choosefile.button" />
+			</html:file>
+			<a href="#" onclick="javascript:doSubmit('uploadOffline');"
+				class="button"> <fmt:message
+					key="label.authoring.upload.offline.button" /> </a>
+		</td>
+	</tr>
+</table>
+
Index: lams_tool_sbmt/web/authoring/parts/instructionfilelist.jsp
===================================================================
diff -u -r671e0f31315ba99ec13f82789869ce2a01e0ea04 -rb3f41153ea9815528aed7730b3760cc11f48bf75
--- lams_tool_sbmt/web/authoring/parts/instructionfilelist.jsp	(.../instructionfilelist.jsp)	(revision 671e0f31315ba99ec13f82789869ce2a01e0ea04)
+++ lams_tool_sbmt/web/authoring/parts/instructionfilelist.jsp	(.../instructionfilelist.jsp)	(revision b3f41153ea9815528aed7730b3760cc11f48bf75)
@@ -1,66 +1,68 @@
-<%@ include file="/common/taglibs.jsp" %>
-<%@ page import=" org.lamsfoundation.lams.contentrepository.client.IToolContentHandler"%>
-<c:set var="CONS_OFFLINE" value="<%=IToolContentHandler.TYPE_OFFLINE %>"/>
+<%@ include file="/common/taglibs.jsp"%>
+<%@ page
+	import=" org.lamsfoundation.lams.contentrepository.client.IToolContentHandler"%>
+<c:set var="CONS_OFFLINE" value="<%=IToolContentHandler.TYPE_OFFLINE %>" />
 <c:choose>
 	<c:when test="${fileTypeFlag==CONS_OFFLINE}">
-		<c:set var="targetFileType" value="<%=IToolContentHandler.TYPE_OFFLINE %>"/>
+		<c:set var="targetFileType"
+			value="<%=IToolContentHandler.TYPE_OFFLINE %>" />
 	</c:when>
 	<c:otherwise>
-		<c:set var="targetFileType" value="<%=IToolContentHandler.TYPE_ONLINE %>"/>
+		<c:set var="targetFileType"
+			value="<%=IToolContentHandler.TYPE_ONLINE %>" />
 	</c:otherwise>
 </c:choose>
 
-<c:set var="sessionMap" value="${sessionScope[sessionMapID]}"/>
-<c:set var="fileList" value="${sessionMap.attachmentList}"/>
+<c:set var="sessionMap" value="${sessionScope[sessionMapID]}" />
+<c:set var="fileList" value="${sessionMap.attachmentList}" />
 
 <%-- check whehter has target file type --%>
 <c:forEach var="file" items="${fileList}">
 	<c:if test="${targetFileType == file.type}">
-		<c:set var="hasFile" value="${true}"/>
+		<c:set var="hasFile" value="${true}" />
 	</c:if>
 </c:forEach>
 
 <%-- Display target file type --%>
 <c:if test="${hasFile}">
-	<table style="align:left;width:600px" border="0" >
+	<ul>
 		<c:forEach var="file" items="${fileList}">
+
 			<c:if test="${targetFileType == file.type}">
-				<tr>
-					<td>
-						<c:out value="${file.name}"/>
-					</td>
-					<td width="80px" align="center">
-						<c:set var="viewURL">
-							<html:rewrite page="/download/?uuid=${file.uuID}&versionID=${file.versionID}&preferDownload=false"/>
-						</c:set>
-						<html:link href="javascript:launchPopup('${viewURL}','offlinefile')" styleClass="button">
-							<fmt:message key="label.view"/>
-						</html:link>
-					</td>
-					<td width="100px" align="center">
-						<c:set var="downloadURL">
-								<html:rewrite page="/download/?uuid=${file.uuID}&versionID=${file.versionID}&preferDownload=true"/>
-						</c:set>
-						<html:link href="${downloadURL}"  styleClass="button">
-							<fmt:message key="label.download"/>
-						</html:link>
-					</td>
-					<td width="80px" align="center">
-						<c:choose>
-							<c:when test="${fileTypeFlag==CONS_OFFLINE}">
-								<html:link href="#" onclick="deleteOfflineFile(${file.uuID},${file.versionID})" styleClass="button">
-									<fmt:message key="label.authoring.offline.delete"/>
-								</html:link>
-							</c:when>
-							<c:otherwise>
-								<html:link href="#" onclick="deleteOnlineFile(${file.uuID},${file.versionID})" styleClass="button">
-									<fmt:message key="label.authoring.online.delete"/>
-								</html:link>
-							</c:otherwise>
-						</c:choose>
-					</td>				
-				</tr>			
+				<li>
+					<c:out value="${file.name}" />
+					<c:set var="viewURL">
+						<html:rewrite
+							page="/download/?uuid=${file.uuID}&versionID=${file.versionID}&preferDownload=false" />
+					</c:set>
+					<html:link
+						href="javascript:launchPopup('${viewURL}','offlinefile')">
+						<fmt:message key="label.view" />
+					</html:link>
+					<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>
+					<c:choose>
+						<c:when test="${fileTypeFlag==CONS_OFFLINE}">
+							<html:link href="#"
+								onclick="deleteOfflineFile(${file.uuID},${file.versionID})">
+								<fmt:message key="label.authoring.offline.delete" />
+							</html:link>
+						</c:when>
+						<c:otherwise>
+							<html:link href="#"
+								onclick="deleteOnlineFile(${file.uuID},${file.versionID})">
+								<fmt:message key="label.authoring.online.delete" />
+							</html:link>
+						</c:otherwise>
+					</c:choose>
+				</li>
 			</c:if>
+
 		</c:forEach>
-	</table>
-</c:if>
\ No newline at end of file
+	</ul>
+</c:if>