From 488409010680a7dab280914b16e43e623e01ee1d Mon Sep 17 00:00:00 2001
From: Romano Licker <r.licker@finn.de>
Date: Fri, 16 Dec 2011 14:16:01 +0100
Subject: [PATCH] [accessibility] more labels and alt tags

---
 app/views/costlog/edit.rhtml         |  6 +++---
 app/views/groups/_users_unused.rhtml | 10 +++++-----
 config/locales/de.yml                |  1 +
 config/locales/en.yml                |  1 +
 4 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/app/views/costlog/edit.rhtml b/app/views/costlog/edit.rhtml
index 0f256d24ef..b847ced186 100644
--- a/app/views/costlog/edit.rhtml
+++ b/app/views/costlog/edit.rhtml
@@ -34,7 +34,7 @@
         </span>
         <br /><em><%= l(:help_override_rate) %></em>
       <% end %>
-      
+
     </p>
     <p><%= f.text_field :comments, :size => 100 %></p>
     </div>
@@ -44,7 +44,7 @@
 
 <% content_for :header_tags do %>
   <%= javascript_include_tag 'editinplace', :plugin => 'redmine_costs' %>
-  <%= javascript_tag "initialize_editinplace('src=\"#{image_path('cancel.png')}\" value=\"#{l(:label_cancel)}\"' )" %>
-  
+  <%= javascript_tag "initialize_editinplace('src=\"#{image_path('cancel.png')}\" value=\"#{l(:label_cancel)}\" alt=\"#{l(:button_cancel_edit_costs)}\" title=\"#{l(:button_cancel_edit_costs)}\"' )" %>
+
   <%= stylesheet_link_tag 'costs', :plugin => 'redmine_costs' %>
 <% end %>
diff --git a/app/views/groups/_users_unused.rhtml b/app/views/groups/_users_unused.rhtml
index 09db6ef5e8..0f402252f4 100644
--- a/app/views/groups/_users_unused.rhtml
+++ b/app/views/groups/_users_unused.rhtml
@@ -14,7 +14,7 @@
         <td align="center" style="white-space: nowrap;">
           <% remote_form_for :group_user, group_user, :url => { :controller => 'groups', :action => 'set_membership_type', :id => @group, :user_id => user}, :method => :post  do |f| %>
             <%= f.select :membership_type, GroupUser::MEMBERSHIP_TYPES.collect{|t| [l(t), t]} %>
-            <%= image_submit_tag "save.png" %>
+            <%= image_submit_tag "save.png", :alt => l(:button_save) %>
           <% end %>
         </td>
         <td class="buttons">
@@ -36,24 +36,24 @@
 <% if users.any? %>
   <% remote_form_for(:group, @group, :url => {:controller => 'groups', :action => 'add_users', :id => @group}, :method => :post) do |f| %>
     <fieldset><legend><%=l(:label_user_new)%></legend>
-    
+
     <p><%= text_field_tag 'user_search', nil, :size => "40" %></p>
     <p>
       <%= label_tag :membership_type, l(:label_membership_type) %>
       <%= select_tag :membership_type, options_for_select(GroupUser::MEMBERSHIP_TYPES.collect{|t| [l(t), t]}, GroupUser::DEFAULT_MEMBERSHIP_TYPE) %>
     </p>
-    
+
     <%= observe_field(:user_search,
                  :frequency => 0.5,
                  :update => :users,
                  :url => { :controller => 'groups', :action => 'autocomplete_for_user', :id => @group },
                  :with => 'q')
                   %>
-    
+
     <div id="users">
       <%= principals_check_box_tags 'user_ids[]', users %>
     </div>
-    
+
     <p><%= submit_tag l(:button_add) %></p>
     </fieldset>
   <% end %>
diff --git a/config/locales/de.yml b/config/locales/de.yml
index 4c4de354a7..ace75de440 100644
--- a/config/locales/de.yml
+++ b/config/locales/de.yml
@@ -165,4 +165,5 @@ de:
   caption_locked_on: "Gesperrt am"
   caption_show_locked: "Gesperrte Typen anzeigen"
   button_cancel_edit_budget: "Budget bearbeiten abbrechen"
+  button_cancel_edit_costs: "Kosten bearbeiten abbrechen"
   description_date_for_new_rate: "Datum für neuen Satz"
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 9246fbd3c8..db235032f5 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -163,4 +163,5 @@ en:
   caption_locked_on: "Locked on"
   caption_show_locked: "Show locked types"
   button_cancel_edit_budget: "Cancel editing budget"
+  button_cancel_edit_costs: "Cancel editing costs"
   description_date_for_new_rate: "Date for new rate"