From 44beee128c9da06834aef796f03282bca362bcc6 Mon Sep 17 00:00:00 2001
From: Michael M Slusarz <slusarz@curecanti.org>
Date: Wed, 4 Nov 2009 16:46:34 -0700
Subject: [PATCH] Try to distinguish summary between inline part and attachment

---
 imp/lib/UI/Message.php        | 5 +++--
 imp/lib/Views/ShowMessage.php | 2 +-
 imp/message.php               | 2 +-
 imp/themes/screen.css         | 7 +++++++
 4 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/imp/lib/UI/Message.php b/imp/lib/UI/Message.php
index 3d1f15d07..718800100 100644
--- a/imp/lib/UI/Message.php
+++ b/imp/lib/UI/Message.php
@@ -413,16 +413,17 @@ class IMP_UI_Message
      * @param array $summary  Summary information from
      *                        IMP_Summary::getSummary().
      * @param array $display  The fields to display (in this order).
+     * @param boolean $atc    Is this an attachment?
      *
      * @return string  The formatted summary string.
      */
-    public function formatSummary($summary, $display)
+    public function formatSummary($summary, $display, $atc = false)
     {
         $tmp_summary = array();
         foreach ($display as $val) {
             $tmp_summary[] = $summary[$val];
         }
-        return '<div class="mimePartInfo"><div>' . implode(' ', $tmp_summary) . '</div></div>';
+        return '<div class="mimePartInfo' . ($atc ? ' mimePartInfoAtc' : '') . '"><div>' . implode(' ', $tmp_summary) . '</div></div>';
     }
 
     /**
diff --git a/imp/lib/Views/ShowMessage.php b/imp/lib/Views/ShowMessage.php
index 9f6fbb9ad..705b2f11b 100644
--- a/imp/lib/Views/ShowMessage.php
+++ b/imp/lib/Views/ShowMessage.php
@@ -275,7 +275,7 @@ class IMP_Views_ShowMessage
                     }
 
                     if ($GLOBALS['prefs']->getValue('atc_display')) {
-                        $result['msgtext'] .= $imp_ui->formatSummary($imp_contents->getSummary($mime_id, $contents_mask), $part_info_display);
+                        $result['msgtext'] .= $imp_ui->formatSummary($imp_contents->getSummary($mime_id, $contents_mask), $part_info_display, true);
                     }
                 }
                 continue;
diff --git a/imp/message.php b/imp/message.php
index 2f8763c64..0ee3c5944 100644
--- a/imp/message.php
+++ b/imp/message.php
@@ -609,7 +609,7 @@ foreach ($parts_list as $mime_id => $mime_type) {
             }
 
             if ($prefs->getValue('atc_display')) {
-                $msgtext .= $imp_ui->formatSummary($imp_contents->getSummary($mime_id, $contents_mask), $part_info_display);
+                $msgtext .= $imp_ui->formatSummary($imp_contents->getSummary($mime_id, $contents_mask), $part_info_display, true);
             }
         }
         continue;
diff --git a/imp/themes/screen.css b/imp/themes/screen.css
index a6129a2c7..24a2c260b 100644
--- a/imp/themes/screen.css
+++ b/imp/themes/screen.css
@@ -556,6 +556,13 @@ div.mimeStatusMessage, div.mimePartInfo {
 .mimePartInfo div img {
     vertical-align: bottom;
 }
+.mimePartInfoAtc div {
+    border: 1px dashed;
+    background-image: url("graphics/attachment.png");
+    background-position: 2px 50%;
+    background-repeat: no-repeat;
+    padding-left: 25px;
+}
 
 .mimeStatusMessage table td {
     padding: 0;
-- 
2.11.0