From 21a5de76e496fda28dca59af4746396a8011607c Mon Sep 17 00:00:00 2001
From: Coleman Watts <coleman@civicrm.org>
Date: Sun, 25 Oct 2015 10:36:47 -0400
Subject: [PATCH] CRM-13823 - Fix invalid markup and add animation

---
 ang/crmStatusPage/StatusPage.html       | 4 ++--
 ang/crmStatusPage/StatusPageServices.js | 4 ++++
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/ang/crmStatusPage/StatusPage.html b/ang/crmStatusPage/StatusPage.html
index 9ffa5d9134..9e5de8d4da 100644
--- a/ang/crmStatusPage/StatusPage.html
+++ b/ang/crmStatusPage/StatusPage.html
@@ -7,7 +7,7 @@
       <h1>{{ts('Status Messages')}}</h1>
     </div>
 
-    <div crm-ui-tab-set>
+    <div id="crm-status-list" crm-ui-tab-set>
       <div
         ng-repeat="tab in [{is_visible: 1, icon: 'fa-bell'}, {is_visible: 0, icon: 'fa-bell-slash-o'}]"
         crm-ui-tab
@@ -16,7 +16,7 @@
         crm-title="tab.is_visible ? ts('Active') : ts('Hidden')"
         crm-icon="{{tab.icon}}"
         >
-        <div id="crm-status-list" ng-repeat="status in statuses | filter:{is_visible: tab.is_visible}" >
+        <div class="crm-status-item" ng-repeat="status in statuses | filter:{is_visible: tab.is_visible}" >
           <h3 class="crm-severity-{{status.severity}}">
             <i ng-if="status.icon" class="crm-i {{status.icon}}"></i>
             {{status.title}}
diff --git a/ang/crmStatusPage/StatusPageServices.js b/ang/crmStatusPage/StatusPageServices.js
index c4e97fc516..0173484b5c 100644
--- a/ang/crmStatusPage/StatusPageServices.js
+++ b/ang/crmStatusPage/StatusPageServices.js
@@ -24,6 +24,10 @@
               });
             });
           });
+          // TODO: Is there a more "Angular" way to do this animation?
+          element.on('click', 'button:not(.hush-menu-button), li', function() {
+            $(this).closest('div.crm-status-item').slideUp();
+          })
         }
       };
     });
-- 
2.25.1