From: Tom Clegg <tom@curoverse.com>
Date: Sun, 16 Feb 2014 10:58:28 +0000 (-0800)
Subject: Lighten and add tooltips to "delete tag" buttons.
X-Git-Tag: 1.1.0~2722^2~4
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/78a1683680177da480dc3ffaea2d4f2b79ab793d

Lighten and add tooltips to "delete tag" buttons.
---

diff --git a/apps/workbench/app/assets/javascripts/application.js b/apps/workbench/app/assets/javascripts/application.js
index cb20b32896..dc7177060e 100644
--- a/apps/workbench/app/assets/javascripts/application.js
+++ b/apps/workbench/app/assets/javascripts/application.js
@@ -78,7 +78,7 @@ jQuery(function($){
                 attr('data-tag-link-uuid', new_tag_uuid).
                 text(new_tag).
                 css('opacity', '0.2').
-                append('&nbsp;<a><i class="glyphicon glyphicon-trash"></i></a>&nbsp;');
+                append('&nbsp;<a title="Delete tag"><i class="glyphicon glyphicon-trash"></i></a>&nbsp;');
             $(this).
                 parent().
                 find('>span').
diff --git a/apps/workbench/app/assets/stylesheets/application.css.scss b/apps/workbench/app/assets/stylesheets/application.css.scss
index 5ee950803c..7d241addf8 100644
--- a/apps/workbench/app/assets/stylesheets/application.css.scss
+++ b/apps/workbench/app/assets/stylesheets/application.css.scss
@@ -140,6 +140,7 @@ td.add-tag-button .add-tag-button:hover {
     margin-top: .2em;
 }
 .label.removable-tag a {
+    color: #fff;
     cursor: pointer;
 }
 
diff --git a/apps/workbench/app/views/collections/_index_tbody.html.erb b/apps/workbench/app/views/collections/_index_tbody.html.erb
index 19a13ed9a2..258b00b01c 100644
--- a/apps/workbench/app/views/collections/_index_tbody.html.erb
+++ b/apps/workbench/app/views/collections/_index_tbody.html.erb
@@ -36,7 +36,7 @@
       <% @collection_info[c.uuid][:tag_links].each do |tag_link| %>
         <span class="label label-info removable-tag" data-tag-link-uuid="<%= tag_link.uuid %>"><%= tag_link.name %>
           <% if tag_link.owner_uuid == current_user.uuid %>
-          &nbsp;<a><i class="glyphicon glyphicon-trash"></i></a>
+          &nbsp;<a title="Delete tag"><i class="glyphicon glyphicon-trash"></i></a>
           <% end %>
         </span>
       <% end %>