Index: openacs-4/packages/attachments/www/attach.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/attachments/www/attach.adp,v
diff -u -r1.11 -r1.12
--- openacs-4/packages/attachments/www/attach.adp	9 Nov 2008 23:29:22 -0000	1.11
+++ openacs-4/packages/attachments/www/attach.adp	29 May 2009 16:16:10 -0000	1.12
@@ -2,91 +2,81 @@
 <property name="title">#attachments.lt_Attach_a_File_to_pret#</property>
 <property name="context">@context;noquote@</property>
 
-#attachments.lt_You_are_attaching_a_d#
+<p>#attachments.lt_You_are_attaching_a_d#</p>
 
-<p>
+<p>#attachments.lt_To_attach_a_file_alre#</p>
 
-<small>
-#attachments.lt_To_attach_a_file_alre#
-</small>
-
-<p>
-@fs_context_bar_html;noquote@
-<p>
 <if @write_permission_p@ eq 1>
-#attachments.attach_new#  &nbsp;&nbsp;     <a href="file-add?folder_id=@folder_id@&@passthrough_vars@">#attachments.File#</a>
+#attachments.attach_new#
+      &nbsp;
+      <a href="@file_add_url@">#attachments.File#</a>
       &nbsp;&nbsp;|&nbsp;&nbsp;
-      <a href="simple-add?folder_id=@folder_id@&@passthrough_vars@">#attachments.URL#</a>
+      <a href="@simple_add_url@">#attachments.URL#</a>
 </if>
-<p>
+
+
+<p class="attach-fs-bar">@fs_context_bar_html;noquote@</p>
+
 <if @contents:rowcount@ gt 0>
-  <table width="85%" cellpadding="5" cellspacing="0" class="table-display">
-    <tr class="table-header">
-      <td>&nbsp;</td>
-      <td>#attachments.Name#</td>
-      <td>#attachments.Action#</td>
-      <td>#attachments.Size#</td>
-      <td>#attachments.Type#</td>
-      <td>#attachments.Last_Modified#</td>
-    </tr>
+  <table width="95%" class="list-table">
+    <thead>
+      <tr class="list-header">
+        <th>#attachments.Type#</th>
+        <th>#attachments.Name#</th>
+        <th>#attachments.Action#</th>
+        <th>#attachments.Size#</th>
+        <th>#attachments.Last_Modified#</th>
+      </tr>
+    </thead>
+<tbody>
 <multiple name="contents">
 <if @contents.rownum@ odd>
     <tr class="odd">
 </if>
 <else>
     <tr class="even">
 </else>
-    <tr>
-<if @contents.type@ eq "folder">
-      <td><img src="graphics/folder.gif"></td>
+    <if @contents.type@ eq "folder">
       <td>
-        <a href="attach?folder_id=@contents.object_id@&@passthrough_vars@">@contents.name@</a>
-<if @contents.new_p@ and @contents.content_size@ gt 0>(&nbsp;new&nbsp;)</if>
+        <img src="graphics/folder.gif" alt="#file-storage.Folder#">
+        #file-storage.folder_type_pretty_name#
       </td>
+      <td>
+        <a href="@contents.name_url@">@contents.name@</a>
+      </td>
       <td>&nbsp;</td>
       <td>
         #attachments.lt_contentscontent_size_#<if @contents.content_size@ ne 1>s</if>
       </td>
-      <td>#file-storage.folder_type_pretty_name#</td>
       <td>@contents.last_modified@</td>
-</if>
-<else>
-<if @contents.type@ eq "url">
-      <td><img src="graphics/file.gif"></td>
+    </if>
+    <else>
       <td>
-      @contents.name@
-<if @contents.new_p@>(&nbsp;new&nbsp;)</if>
+        <img src="graphics/file.gif" alt="@contents.type@">
+        @contents.type@
       </td>
-      <td>
-        <small>[
-<a href="attach-2?item_id=@contents.object_id@&@passthrough_vars@">#attachments.Choose#</a>
-          ]</small>
-      </td>
-      <td>&nbsp;</td>
-      <td>@contents.type@</td>
+      <if @contents.type@ eq "url">
+        <td>@contents.name@</td>
+        <td>
+          <a href="@contents.action_url@" class="button">#attachments.Choose#</a>
+        </td>
+        <td>&nbsp;</td>
+      </if>
+      <else>
+        <td>
+          <if @contents.title@ eq nil>@contents.name@</if>
+          <else>@contents.title@</else>
+        </td>
+        <td>
+          <a href="@contents.action_url@" class="button">#attachments.Choose#</a>
+        </td>
+        <td>#attachments.lt_contentscontent_size__1#<if @contents.content_size@ ne 1>s</if></td>
+      </else>
       <td>@contents.last_modified@</td>
-</if>
-<else>
-      <td><img src="graphics/file.gif"></td>
-      <td>
-      <if @contents.title@ eq nil>@contents.name@</if>
-      <else>@contents.title@</else>
-<if @contents.new_p@>
-        (&nbsp;new&nbsp;)
-</if>
-      </td>
-      <td>
-        <small>[
-<a href="attach-2?item_id=@contents.object_id@&@passthrough_vars@">#attachments.Choose#</a>
-        ]</small>
-      </td>
-      <td>#attachments.lt_contentscontent_size__1#<if @contents.content_size@ ne 1>s</if></td>
-      <td>@contents.type@</td>
-      <td>@contents.last_modified@</td>
-</else>
-</else>
+    </else>
     </tr>
-</multiple>
+  </multiple>
+  </tbody>
   </table>
 </if>
 <else>