From 70332f897b7850d722392d7c3845602eb5bb075b Mon Sep 17 00:00:00 2001
From: Giulio Gratta <giulio@giuliogratta.com>
Date: Tue, 4 Jun 2013 17:51:46 -0700
Subject: [PATCH] courseware active location, suppress browser form validation,
 and progress graph accessibilit changes

---
 lms/templates/courseware/course_navigation.html | 3 +++
 lms/templates/courseware/progress.html          | 9 +++++++--
 lms/templates/login.html                        | 2 +-
 lms/templates/register.html                     | 2 +-
 4 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/lms/templates/courseware/course_navigation.html b/lms/templates/courseware/course_navigation.html
index ea367873e2..76868ebb99 100644
--- a/lms/templates/courseware/course_navigation.html
+++ b/lms/templates/courseware/course_navigation.html
@@ -20,6 +20,9 @@ def url_class(is_active):
           <li>
              <a href="${tab.link | h}" class="${url_class(tab.is_active)}">
                  ${tab.name | h}
+                 % if tab.is_active == True:
+                  <span class="visuallyhidden">, current location</span>
+                 %endif
                  % if tab.has_img == True:
                     <img src="${tab.img}"/>
                  %endif
diff --git a/lms/templates/courseware/progress.html b/lms/templates/courseware/progress.html
index fcd4348f96..f7021c9fd6 100644
--- a/lms/templates/courseware/progress.html
+++ b/lms/templates/courseware/progress.html
@@ -35,7 +35,7 @@ ${progress_graph.body(grade_summary, course.grade_cutoffs, "grade-detail-graph",
       </header>
 
       %if not course.disable_progress_graph:
-        <div id="grade-detail-graph"></div>
+        <div id="grade-detail-graph" aria-hidden="true"></div>
       %endif
 
       <ol class="chapters">
@@ -54,7 +54,12 @@ ${progress_graph.body(grade_summary, course.grade_cutoffs, "grade-detail-graph",
               %>
 
               <h3><a href="${reverse('courseware_section', kwargs=dict(course_id=course.id, chapter=chapter['url_name'], section=section['url_name']))}">
-                ${ section['display_name'] }</a>
+                ${ section['display_name'] }
+                %if total > 0 or earned > 0:
+                  <span class="visuallyhidden">
+                    ${"{0:.3n} of {1:.3n} possible points".format( float(earned), float(total) )}
+                  </span></a>
+                %endif
                 %if total > 0 or earned > 0:
                   <span> ${"({0:.3n}/{1:.3n}) {2}".format( float(earned), float(total), percentageString )}</span>
                 %endif
diff --git a/lms/templates/login.html b/lms/templates/login.html
index d3c3384847..4107a463e1 100644
--- a/lms/templates/login.html
+++ b/lms/templates/login.html
@@ -86,7 +86,7 @@
 
 <section class="login container">
   <section role="main" class="content">
-    <form role="form" id="login-form" method="post" data-remote="true" action="/login_ajax">
+    <form role="form" id="login-form" method="post" data-remote="true" action="/login_ajax" novalidate>
 
       <!-- status messages -->
       <div role="alert" class="status message">
diff --git a/lms/templates/register.html b/lms/templates/register.html
index 5339a55cfc..694f94558d 100644
--- a/lms/templates/register.html
+++ b/lms/templates/register.html
@@ -89,7 +89,7 @@
 
 <section class="register container">
   <section role="main" class="content">
-    <form role="form" id="register-form" method="post" data-remote="true" action="/create_account">
+    <form role="form" id="register-form" method="post" data-remote="true" action="/create_account" novalidate>
 
       <!-- status messages -->
       <div role="alert" class="status message">