fix highlighter: add checkmark to active color

This commit is contained in:
Felix Niklas 2015-06-24 17:53:01 +02:00
parent ebedae537c
commit a4c6299179
2 changed files with 4 additions and 5 deletions

View file

@ -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()

View file

@ -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 %>