From d383174aee2e56fb13812cdc18ac6f66c46055aa Mon Sep 17 00:00:00 2001
From: Coleman Watts <coleman@civicrm.org>
Date: Wed, 3 Aug 2016 15:07:01 -0400
Subject: [PATCH] Blog dashlet improvements

---
 CRM/Dashlet/Page/Blog.php           |  7 ++++++-
 css/dashboard.css                   |  2 +-
 templates/CRM/Dashlet/Page/Blog.tpl | 31 ++++++++++++++++++++++++-----
 3 files changed, 33 insertions(+), 7 deletions(-)

diff --git a/CRM/Dashlet/Page/Blog.php b/CRM/Dashlet/Page/Blog.php
index 31462f81f1..acc144ba9e 100644
--- a/CRM/Dashlet/Page/Blog.php
+++ b/CRM/Dashlet/Page/Blog.php
@@ -117,7 +117,12 @@ class CRM_Dashlet_Page_Blog extends CRM_Core_Page {
         $item = (array) $item;
         // Clean up description - remove tags that would break dashboard layout
         $description = preg_replace('#<h[1-3][^>]*>(.+?)</h[1-3][^>]*>#s', '<h4>$1</h4>', $item['description']);
-        $item['description'] = strip_tags($description, "<a><p><h4><h5><h6><b><i><em><strong><ol><ul><li><dd><dt><code><pre><br/>");
+        $description = strip_tags($description, "<a><p><h4><h5><h6><b><i><em><strong><ol><ul><li><dd><dt><code><pre><br/>");
+        // Add paragraph markup if it's missing.
+        if (strpos($description, '<p') === FALSE) {
+          $description = '<p>' . $description . '</p>';
+        }
+        $item['description'] = $description;
         $blog[] = $item;
       }
       if ($blog) {
diff --git a/css/dashboard.css b/css/dashboard.css
index 60017e5e60..c2f63fb653 100644
--- a/css/dashboard.css
+++ b/css/dashboard.css
@@ -75,7 +75,7 @@
 #crm-container .widget-content .crm-accordion-header {
   background-color: #EFEFE5;
   background-image: url("../i/TreeMinus.gif");
-  color: #3E3E3E;
+  color: #080808;
 }
 
 #crm-container .widget-content .crm-accordion-wrapper.collapsed .crm-accordion-header {
diff --git a/templates/CRM/Dashlet/Page/Blog.tpl b/templates/CRM/Dashlet/Page/Blog.tpl
index 4f130dbe15..a59bbabad6 100644
--- a/templates/CRM/Dashlet/Page/Blog.tpl
+++ b/templates/CRM/Dashlet/Page/Blog.tpl
@@ -23,14 +23,35 @@
  | see the CiviCRM license FAQ at http://civicrm.org/licensing        |
  +--------------------------------------------------------------------+
 *}
-{foreach from=$blog item=article}
+{strip}{literal}
+<style type="text/css">
+  #civicrm-blog-feed .collapsed .crm-accordion-header {
+    text-overflow: ellipsis;
+    text-wrap: none;
+    white-space: nowrap;
+    overflow: hidden;
+  }
+  #civicrm-blog-feed .crm-blog-preview {
+    color: #8d8d8d;
+    display: none;
+  }
+  #civicrm-blog-feed .collapsed .crm-blog-preview {
+    display: inline;
+  }
+</style>
+{/literal}
 <div id="civicrm-blog-feed">
+{foreach from=$blog item=article}
   <div class="crm-accordion-wrapper collapsed">
-    <div class="crm-accordion-header">{$article.title}</div>
-    <div class="crm-accordion-body help">
+    <div class="crm-accordion-header">
+      <span class="crm-blog-title">{$article.title}</span>
+      <span class="crm-blog-preview"> - {$article.description|strip_tags|substr:0:100}…</span>
+    </div>
+    <div class="crm-accordion-body">
       <div>{$article.description}</div>
-      <div><a href="{$article.link}" title="{$article.title}">{ts}read more{/ts}.</a></div>
+      <div><a target="_blank" href="{$article.link}" title="{$article.title}"><i class="crm-i fa-external-link"></i> {ts}read more{/ts}…</a></div>
     </div>
   </div>
-</div>
 {/foreach}
+</div>
+{/strip}
-- 
2.25.1