From a99d25b174e07bf58ae8d0c5055291065038f81a Mon Sep 17 00:00:00 2001
From: Phil Hughes <me@iamphill.com>
Date: Fri, 10 Jun 2016 08:42:03 +0100
Subject: Checks against date parsing instead of string Removes template for
 html string that isn't needed

---
 app/assets/javascripts/due_date_select.js.coffee | 2 +-
 app/assets/javascripts/labels_select.js.coffee   | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/assets/javascripts/due_date_select.js.coffee b/app/assets/javascripts/due_date_select.js.coffee
index d5cb3f620b1..32c143cae16 100644
--- a/app/assets/javascripts/due_date_select.js.coffee
+++ b/app/assets/javascripts/due_date_select.js.coffee
@@ -50,7 +50,7 @@ class @DueDateSelect
               $selectbox.hide()
             $value.css('display', '')
 
-            cssClass = if mediumDate is "No due date" then 'no-value' else 'bold'
+            cssClass = if Date.parse(mediumDate) then 'bold' else 'no-value'
             $valueContent.html("<span class='#{cssClass}'>#{mediumDate}</span>")
             $sidebarValue.html(mediumDate)
 
diff --git a/app/assets/javascripts/labels_select.js.coffee b/app/assets/javascripts/labels_select.js.coffee
index 6dff9fc4fd0..5df3af6091a 100644
--- a/app/assets/javascripts/labels_select.js.coffee
+++ b/app/assets/javascripts/labels_select.js.coffee
@@ -39,7 +39,7 @@ class @LabelsSelect
             </a>
             <% }); %>'
         )
-        labelNoneHTMLTemplate = _.template('<span class="no-value">None</span>')
+        labelNoneHTMLTemplate = '<span class="no-value">None</span>'
 
       if newLabelField.length
 
@@ -142,7 +142,7 @@ class @LabelsSelect
             template = labelHTMLTemplate(data)
             labelCount = data.labels.length
           else
-            template = labelNoneHTMLTemplate()
+            template = labelNoneHTMLTemplate
           $value
             .removeAttr('style')
             .html(template)
-- 
cgit v1.2.1