Make the attachment icon clickable to expand the attachment list

diff --git a/imp/templates/chunks/message.php b/imp/templates/chunks/message.php
index 6cf9a95..9c8e5a6 100644 (file)
--- a/imp/templates/chunks/message.php
+++ b/imp/templates/chunks/message.php
@@ -66,7 +66,13 @@
 <?php endforeach; ?>
 <?php if (isset($show_msg_result['atc_label'])): ?>
         <tr id="msgAtc">
-         <td class="label"><?php if ($show_msg_result['atc_list']): ?><span class="iconImg attachmentImg attachmentImage"></span><a id="partlist_toggle"><span id="partlist_col" class="iconImg"></span><span id="partlist_exp" class="iconImg" style="display:none"></span></a><?php else: ?><span class="iconImg attachmentImg attachmentImage"></span><?php endif; ?></td>
+         <td class="label" id="partlist_toggle">
+          <span class="iconImg attachmentImg attachmentImage"></span>
+<?php if ($show_msg_result['atc_list']): ?>
+          <span id="partlist_col" class="iconImg"></span>
+          <span id="partlist_exp" class="iconImg" style="display:none"></span>
+<?php endif; ?>
+         </td>
          <td>
           <span class="atcLabel"><?php echo $show_msg_result['atc_label'] ?></span><?php echo isset($show_msg_result['atc_download']) ? $show_msg_result['atc_download'] : '' ?>
 <?php if (isset($show_msg_result['atc_list'])): ?>
@@ -80,9 +86,14 @@
         </tr>
 <?php endif; ?>
         <tr id="msgLogInfo" style="display:none">
-         <td class="label"><a id="msgloglist_toggle"><span class="iconImg" id="msgloglist_col"></span><span class="iconImg" id="msgloglist_exp" style="display:none"></span></a></td>
+         <td class="label" id="msgloglist_toggle">
+          <span class="iconImg" id="msgloglist_col"></span>
+          <span class="iconImg" id="msgloglist_exp" style="display:none"></span>
+         </td>
          <td>
-          <div><span class="msgLogLabel"><?php echo _("Message Log") ?></span></div>
+          <div>
+           <span class="msgLogLabel"><?php echo _("Message Log") ?></span>
+          </div>
           <div id="msgloglist" style="display:none">
            <ul></ul>
           </div>

diff --git a/imp/templates/index/index-dimp.inc b/imp/templates/index/index-dimp.inc
index dfc163d..54960bd 100644 (file)
--- a/imp/templates/index/index-dimp.inc
+++ b/imp/templates/index/index-dimp.inc
@@ -276,7 +276,11 @@ function _simpleButton($id, $text, $image)
              <td class="cc"></td>
             </tr>
             <tr id="msgAtc" style="display:none">
-             <td class="label"><span class="iconImg attachmentImg attachmentImage"></span><a id="partlist_toggle"><span class="iconImg" id="partlist_col"></span><span class="iconImg" id="partlist_exp" style="display:none"></span></a></td>
+             <td class="label" id="partlist_toggle">
+              <span class="iconImg attachmentImg attachmentImage"></span>
+              <span class="iconImg" id="partlist_col"></span>
+              <span class="iconImg" id="partlist_exp" style="display:none"></span>
+              </td>
              <td>
               <div></div>
               <div id="partlist" style="display:none">
@@ -285,8 +289,9 @@ function _simpleButton($id, $text, $image)
              </td>
             </tr>
             <tr id="msgLogInfo" style="display:none">
-             <td class="label">
-              <a id="msgloglist_toggle"><span class="iconImg" id="msgloglist_col"></span><span class="iconImg" id="msgloglist_exp" style="display:none"></span></a>
+             <td class="label" id="msgloglist_toggle">
+              <span class="iconImg" id="msgloglist_col"></span>
+              <span class="iconImg" id="msgloglist_exp" style="display:none"></span>
              </td>
              <td>
               <div><span class="msgLogLabel"><?php echo _("Message Log") ?></span></div>

diff --git a/imp/themes/screen-dimp.css b/imp/themes/screen-dimp.css
index 9f43f31..2194187 100644 (file)
--- a/imp/themes/screen-dimp.css
+++ b/imp/themes/screen-dimp.css
@@ -484,6 +484,10 @@ div.dimpActionsCompose, div.dimpActionsMsg {
     padding-right: 3px;
 }
 
+#partlist_toggle span.attachmentImg {
+    margin-right: 0;
+}
+
 #msgloglist {
     padding-top: 4px;
 }
@@ -491,6 +495,10 @@ div.dimpActionsCompose, div.dimpActionsMsg {
     list-style-type: none;
 }
 
+#msgloglist_toggle, #partlist_toggle {
+    cursor: pointer;
+}
+
 #msgHeadersContent {
     overflow: hidden;
 }