From 6bf21caf22b91942a229c1e06f3db7f8aab35ffb Mon Sep 17 00:00:00 2001
From: Dan Rice <drice@diego>
Date: Wed, 26 Sep 2012 16:17:07 -0400
Subject: [PATCH] wrap the todo dropdown arrow in an "a" tag to make it
 tab-accessible

---
 app/assets/stylesheets/tracks.css.scss | 5 +++++
 app/views/todos/_todo.html.erb         | 4 ++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/app/assets/stylesheets/tracks.css.scss b/app/assets/stylesheets/tracks.css.scss
index eb8fbd8c..bb801c83 100644
--- a/app/assets/stylesheets/tracks.css.scss
+++ b/app/assets/stylesheets/tracks.css.scss
@@ -1431,6 +1431,11 @@ div.auto_complete ul strong.highlight {
   background: white image-url('ui-anim_basic_16x16.gif') right center no-repeat;
 }
 
+a.item-downarrow {
+  margin-left: 0;
+  margin-right: 0;
+}
+
 ul.todo-submenu > li > a {
   position: relative;
   padding-left: 24px;
diff --git a/app/views/todos/_todo.html.erb b/app/views/todos/_todo.html.erb
index 7a9ee02a..9600ad87 100644
--- a/app/views/todos/_todo.html.erb
+++ b/app/views/todos/_todo.html.erb
@@ -17,7 +17,7 @@ cache [todo, current_user.date.strftime("%Y%m%d")] do
       <%= remote_toggle_checkbox(todo) %>
       <%= remote_edit_button(todo) unless suppress_edit_button %>
       <ul class="sf-menu sf-item-menu">
-        <li style="z-index:<%=@z_index_counter%>"><%= image_tag "downarrow.png", :class => "todo-submenu", :alt=> "" %>
+        <li style="z-index:<%=@z_index_counter%>"><a href="#" class="item-downarrow"><%= image_tag "downarrow.png", :class => "todo-submenu", :alt=> "" %></a>
           <ul id="ul<%= dom_id(todo) %>" class="todo-submenu">
             <li><%= remote_delete_menu_item(todo) %></li>
             <% unless todo.completed? || todo.deferred? -%>
@@ -47,4 +47,4 @@ cache [todo, current_user.date.strftime("%Y%m%d")] do
       <div class="placeholder"> </div>
     </div>
   </div>
-<% end %>
\ No newline at end of file
+<% end %>