ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

add screenreader text to account setting report description links 

test plan:
* using a screen-reader, confirm that the question mark buttons
on the Account settings/Reports tab have descriptive text

closes #CNVS-13913

Change-Id: I8c453cb60e27156d8c6b2da9f98d2940c203320d
Reviewed-on: https://gerrit.instructure.com/37977
Tested-by: Jenkins <jenkins@instructure.com>
Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
QA-Review: Clare Strong <clare@instructure.com>
Product-Review: James Williams  <jamesw@instructure.com>
This commit is contained in:
James Williams 2014-07-22 09:44:15 -06:00
parent 8a131e8cbf
commit 2e0d9e635e
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
app/views/accounts/settings.html.erb
View File

@ -863,8 +863,11 @@ acknowledge that you have read and agreed to the
<td class="title reports">
<span class="title"><%= title %></span>
<% if description_partial %>
<a href="#" class="open_report_description_link"><i class="icon-question standalone-icon"></i></a>
<div style="display: none" class="report_description">
<a href="#" class="open_report_description_link" role="button" aria-haspopup="true" aria-owns="<%= "#{report}_report_description" %>">
<i class="icon-question standalone-icon"></i>
<span class="screenreader-only"><%= t(:open_report_description, "Open Report Description") %></span>
</a>
<div id="<%= "#{report}_report_description" %>" style="display: none" class="report_description">
<%= render(:partial => description_partial) %>
</div>
<% end %>