From 82b85c9a3f83b81876612aac57407c698bcb2e0d Mon Sep 17 00:00:00 2001
From: "lpsolit%gmail.com" <lpsolit%gmail.com>
Date: Sun, 21 Jan 2007 14:11:14 +0000
Subject: [PATCH] =?UTF-8?q?Bug=20365247:=20New=20UI=20for=20the=20attachme?=
 =?UTF-8?q?nt=20table=20(ported=20from=20b.m.o)=20-=20Patch=20by=20Fr?=
 =?UTF-8?q?=EF=BF=BD=EF=BF=BDd=EF=BF=BD=EF=BF=BDric=20Buclin=20<LpSolit@gm?=
 =?UTF-8?q?ail.com>=20(UI=20proposed=20by=20Mike=20Connor=20<mconnor@mozil?=
 =?UTF-8?q?la.com>)=20r=3Dbkor=20a=3DLpSolit?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 webtools/bugzilla/skins/standard/global.css   |  29 +++++
 .../en/default/attachment/list.html.tmpl      | 100 +++++++++---------
 2 files changed, 77 insertions(+), 52 deletions(-)

diff --git a/webtools/bugzilla/skins/standard/global.css b/webtools/bugzilla/skins/standard/global.css
index 7504cb0fbe4..bd558d5c517 100644
--- a/webtools/bugzilla/skins/standard/global.css
+++ b/webtools/bugzilla/skins/standard/global.css
@@ -281,6 +281,35 @@ dl dl > dt {
     border-top: none;
 }
 
+/* Style of the attachment table */
+#attachment_table {
+    border-collapse: collapse;
+    width: 40em;
+    border: 1px solid #333333;
+}
+
+#attachment_table th, .bz_attach_footer {
+    background-color: #E0E0E0;
+    color: black;
+}
+
+#attachment_table td {
+    border: 1px solid #333333;
+}
+
+.bz_attach_extra_info {
+    font-size: smaller;
+}
+
+.bz_attach_flags, .bz_attach_footer {
+    white-space: nowrap;
+}
+
+.bz_attach_view_hide {
+    float: right;
+    padding-left: 1em;
+}
+
 .box {
     border: 1px solid black;
     color: black;
diff --git a/webtools/bugzilla/template/en/default/attachment/list.html.tmpl b/webtools/bugzilla/template/en/default/attachment/list.html.tmpl
index 154cb98b183..275e92df27e 100644
--- a/webtools/bugzilla/template/en/default/attachment/list.html.tmpl
+++ b/webtools/bugzilla/template/en/default/attachment/list.html.tmpl
@@ -51,18 +51,13 @@
 </script>
 
 <br>
-<table id="attachment_table" cellspacing="0" cellpadding="4" border="1">
+<table id="attachment_table" cellspacing="0" cellpadding="4">
   <tr>
-    <th bgcolor="#cccccc" align="left"><a name="a0" id="a0">Attachment</a></th>
-    <th bgcolor="#cccccc" align="left">Type</th>
-    <th bgcolor="#cccccc" align="left">Creator</th>
-    <th bgcolor="#cccccc" align="left">Created</th>
-    <th bgcolor="#cccccc" align="left">Size</th>
-    [% IF show_attachment_flags %]
-      <th bgcolor="#cccccc" align="left">Flags</th>
-    [% END %]
-    <th bgcolor="#cccccc" align="left">Actions</th>
+    <th colspan="[% show_attachment_flags ? 3 : 2 %]" align="left">
+      <a name="a0" id="a0">Attachments</a>
+    </th>
   </tr>
+
   [% count = 0 %]
   [% has_obsolete_attachments = 0 %]
 
@@ -72,40 +67,44 @@
       [% SET has_obsolete_attachments = 1 IF attachment.isobsolete %]
       <tr class="[% "bz_private" IF attachment.isprivate %][%-%]
                  [%+ "bz_tr_obsolete" IF attachment.isobsolete %]">
-        <td valign="top">
-          <a name="a[% count %]" href="attachment.cgi?id=[% attachment.id %]">[% attachment.description FILTER html FILTER obsolete(attachment.isobsolete) %]</a>
-        </td>
-
-        <td valign="top">
-          [% IF attachment.ispatch %]
-            <i>patch</i>
-          [% ELSIF attachment.isurl %]
-            <i>url</i>
-          [% ELSE %]
-            [% attachment.contenttype FILTER html %]
-          [% END %]
-        </td>
-
-        <td valign="top">
-          <a href="mailto:[% attachment.attacher.email FILTER html %]">
-          [% attachment.attacher.name || attachment.attacher.login FILTER html %]
-          </a>
-        </td>
-        <td valign="top">
-          <a href="#attach_[% attachment.id %]">[% attachment.attached FILTER time %]</a>
-        </td>
         <td valign="top">
           [% IF attachment.datasize %]
-            [% attachment.datasize FILTER unitconvert %]
-          [% ELSE %]
-            <em>deleted</em>
+            <a name="a[% count %]" href="attachment.cgi?id=[% attachment.id %]"
+               title="View the content of the attachment">
           [% END %]
+          [% attachment.description FILTER html FILTER obsolete(attachment.isobsolete) %]
+          [% "</a>" IF attachment.datasize %]
+
+          <span class="bz_attach_extra_info">
+            [% IF attachment.datasize %]
+              ([% attachment.datasize FILTER unitconvert %],
+              [% IF attachment.ispatch %]
+                <i>patch</i>)
+              [% ELSIF attachment.isurl %]
+                <i>url</i>)
+              [% ELSE %]
+                [%+ attachment.contenttype FILTER html %])
+              [% END %]
+            [% ELSE %]
+              (<em>deleted</em>)
+            [% END %]
+
+            <br>
+            <a href="#attach_[% attachment.id %]"
+               title="Go to the comment associated with the attachment">
+              [%- attachment.attached FILTER time %]</a>,
+
+            <a href="mailto:[% attachment.attacher.email FILTER html %]"
+               title="Write an email to the creator of the attachment">
+              [% attachment.attacher.name || attachment.attacher.login FILTER html %]
+            </a>
+          </span>
         </td>
 
         [% IF show_attachment_flags %]
-          <td valign="top">
+          <td class="bz_attach_flags" valign="top">
             [% IF attachment.flags.size == 0 %]
-              <i>none</i>
+              <i>no flag</i>
             [% ELSE %]
               [% FOREACH flag = attachment.flags %]
                 [% flag.setter.nick FILTER html %]:
@@ -129,22 +128,19 @@
     [% END %]
   [% END %]
 
-  <tr>
-    <td colspan="[% show_attachment_flags ? 6 : 5 %]">
-      <a href="attachment.cgi?bugid=[% bugid %]&amp;action=enter">Create a New Attachment</a> (proposed patch, testcase, etc.)
+  <tr class="bz_attach_footer">
+    <td colspan="[% show_attachment_flags ? 3 : 2 %]">
+      [% IF attachments.size %]
+        <span class="bz_attach_view_hide">
+          [% IF has_obsolete_attachments %]
+            <a href="#a0" onClick="return toggle_display(this);">Hide Obsolete</a> |
+          [% END %]
+          <a href="attachment.cgi?bugid=[% bugid %]&amp;action=viewall">View All</a>
+        </span>
+      [% END %]
+      <a href="attachment.cgi?bugid=[% bugid %]&amp;action=enter">Add an attachment</a>
+      (proposed patch, testcase, etc.)
     </td>
-    [% IF attachments.size %]
-      <td>
-        <a href="attachment.cgi?bugid=[% bugid %]&amp;action=viewall">View All</a>
-        [% IF has_obsolete_attachments %]
-          <br><a href="#a0" onClick="return toggle_display(this);">Hide Obsolete</a>
-        [% END %]
-      </td>
-    [% ELSE %]
-      <td class="bz_disabled">
-        View All
-      </td>
-    [% END %]
   </tr>
 </table>
 <br>