From 4e15227fd07bb1ea49ded11722ee2572e6b4aff6 Mon Sep 17 00:00:00 2001
From: Neil Lalonde <neillalonde@gmail.com>
Date: Tue, 26 Feb 2013 17:05:17 -0500
Subject: [PATCH] Admin dashboard: use ember's class name binding

---
 .../admin/templates/dashboard.js.handlebars   |  4 ++--
 .../admin/views/admin_dashboard_view.js       | 20 +------------------
 2 files changed, 3 insertions(+), 21 deletions(-)

diff --git a/app/assets/javascripts/admin/templates/dashboard.js.handlebars b/app/assets/javascripts/admin/templates/dashboard.js.handlebars
index 316621c7d97..fce9e63a3f4 100644
--- a/app/assets/javascripts/admin/templates/dashboard.js.handlebars
+++ b/app/assets/javascripts/admin/templates/dashboard.js.handlebars
@@ -1,7 +1,7 @@
 <h3>{{i18n admin.dashboard.welcome}}</h3>
 
 {{#if Discourse.SiteSettings.version_checks}}
-  <p {{bindAttr class="view.priorityClass"}}>
+  <p {{bindAttr class=":version-check versionCheck.critical_updates:critical:normal"}}>
     {{#if loading }}
       {{i18n loading}}
     {{else}}
@@ -15,7 +15,7 @@
         {{#if versionCheck.upToDate }}
           <i class='icon icon-ok update-to-date'></i> {{i18n admin.dashboard.up_to_date}}
         {{else}}
-          <i {{bindAttr class="view.updateIconClasses"}}></i>
+          <i {{bindAttr class=":icon :icon-warning-sign versionCheck.critical_updates:critical-updates-available:updates-available"}}></i>
           <span class="critical-note">{{i18n admin.dashboard.critical_available}}</span>
           <span class="normal-note">{{i18n admin.dashboard.updates_available}}</span>
           {{i18n admin.dashboard.please_upgrade}}
diff --git a/app/assets/javascripts/admin/views/admin_dashboard_view.js b/app/assets/javascripts/admin/views/admin_dashboard_view.js
index 0d0f6faf5c2..6e2581d3d50 100644
--- a/app/assets/javascripts/admin/views/admin_dashboard_view.js
+++ b/app/assets/javascripts/admin/views/admin_dashboard_view.js
@@ -7,25 +7,7 @@
   @module Discourse
 **/
 Discourse.AdminDashboardView = Discourse.View.extend({
-  templateName: 'admin/templates/dashboard',
-
-  updateIconClasses: function() {
-    var classes;
-    classes = "icon icon-warning-sign ";
-    if (this.get('controller.versionCheck.critical_updates')) {
-      classes += "critical-updates-available";
-    } else {
-      classes += "updates-available";
-    }
-    return classes;
-  }.property('controller.versionCheck.critical_updates'),
-
-  priorityClass: function() {
-    if (this.get('controller.versionCheck.critical_updates')) {
-      return 'version-check critical';
-    }
-    return 'version-check normal';
-  }.property('controller.versionCheck.critical_updates')
+  templateName: 'admin/templates/dashboard'
 });