From a4c6299179d15364e9597efdacae443af0335aa0 Mon Sep 17 00:00:00 2001
From: Felix Niklas <mrflix@gmail.com>
Date: Wed, 24 Jun 2015 17:53:01 +0200
Subject: [PATCH] fix highlighter: add checkmark to active color

---
 .../app/controllers/ticket_zoom/higlighter.js.coffee         | 5 ++---
 .../javascripts/app/views/ticket_zoom/highlighter.jst.eco    | 4 ++--
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/app/assets/javascripts/app/controllers/ticket_zoom/higlighter.js.coffee b/app/assets/javascripts/app/controllers/ticket_zoom/higlighter.js.coffee
index 91ca915e3..9f71af42f 100644
--- a/app/assets/javascripts/app/controllers/ticket_zoom/higlighter.js.coffee
+++ b/app/assets/javascripts/app/controllers/ticket_zoom/higlighter.js.coffee
@@ -162,9 +162,8 @@ class App.TicketZoomHighlighter extends App.Controller
         #@articles.attr('data-highlightcolor', @colors[@activeColorIndex].name)
 
   pickColor: (e) =>
-    # TODO: @mrflix - still needed?
-    #@$('.js-highlightColor .visibility-change.active').removeClass('active')
-    #$(e.currentTarget).find('.visibility-change').addClass('active')
+    @$('.js-highlightColor .visibility-change.is-active').removeClass('is-active')
+    $(e.currentTarget).find('.visibility-change').addClass('is-active')
 
     @activeColorIndex = $(e.currentTarget).attr('data-key')
     @setColor()
diff --git a/app/assets/javascripts/app/views/ticket_zoom/highlighter.jst.eco b/app/assets/javascripts/app/views/ticket_zoom/highlighter.jst.eco
index d59889b9e..6c0634d0f 100644
--- a/app/assets/javascripts/app/views/ticket_zoom/highlighter.jst.eco
+++ b/app/assets/javascripts/app/views/ticket_zoom/highlighter.jst.eco
@@ -13,8 +13,8 @@
           <span class="color-swatch icon" style="background: <%= entry.color %>"></span>
         </span>
         <%- @Ti(entry.name) %>
-        <span class="dropdown-activeSpacer visibility-change<%= ' active' if i is @activeColorIndex %>">
-          <span class="white checkmark icon" data-visible="active"></span>
+        <span class="dropdown-activeSpacer visibility-change<%= ' is-active' if i is @activeColorIndex %>">
+          <svg class="icon-checkmark" data-visible="active"><use xlink:href="#icon-checkmark" fill="white" /></svg>
         </span>
       </a>
     <% end %>