Merge pull request #104 from zoombody/accessibility

wrap the todo dropdown arrow in an "a" tag to make it tab-accessible
This commit is contained in:
Matt Rogers 2012-09-26 21:00:50 -07:00
commit d03230008a
3 changed files with 8 additions and 3 deletions

View file

@ -9,6 +9,6 @@
* compiled file, but it's generally better to create a new file per style scope.
*
*= require_self
*= require tracks
*= require_tree ../../../vendor/assets/stylesheets
*= require tracks
*/

View file

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

View file

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