From 7af6409dc54d0511997f2b542e061c997487c3dc Mon Sep 17 00:00:00 2001
From: Giulio Gratta <giulio@giuliogratta.com>
Date: Mon, 10 Jun 2013 11:43:18 -0700
Subject: [PATCH 1/7] use jquery to focus on alert messages

---
 lms/templates/login.html    | 4 ++--
 lms/templates/register.html | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/lms/templates/login.html b/lms/templates/login.html
index fbd2f6d07c..fd51e429e9 100644
--- a/lms/templates/login.html
+++ b/lms/templates/login.html
@@ -53,7 +53,7 @@
             location.href="${reverse('dashboard')}";
           }
         } else {
-          $('.message.submission-error').addClass('is-shown');
+          $('.message.submission-error').addClass('is-shown').focus();
           $('.message.submission-error .message-copy').html(json.value);
         }
       });
@@ -94,7 +94,7 @@
         <p class="message-copy">Lorem ipsum dolor sit amet, consectetuer adipiscing elit. Lorem ipsum dolor sit amet, consectetuer adipiscing elit.</p>
       </div>
 
-      <div role="alert" class="status message submission-error">
+      <div role="alert" class="status message submission-error" tabindex="-1">
         <h3 class="message-title">The following errors occured while logging you in: </h3>
         <ul class="message-copy">
           <li>Your email or password is incorrect</li>
diff --git a/lms/templates/register.html b/lms/templates/register.html
index 1a42d402e5..1c2490c0a0 100644
--- a/lms/templates/register.html
+++ b/lms/templates/register.html
@@ -54,7 +54,7 @@
           $('.message.submission-error').removeClass('is-shown');
           location.href="${reverse('dashboard')}";
        } else {
-         $('.status.message.submission-error').addClass('is-shown');
+         $('.status.message.submission-error').addClass('is-shown').focus();
          $('.status.message.submission-error .message-copy').html(json.value).stop().css("display", "block");
          $(".field-error").removeClass('field-error');
          $("[data-field='"+json.field+"']").addClass('field-error')
@@ -97,7 +97,7 @@
         <p class="message-copy">Lorem ipsum dolor sit amet, consectetuer adipiscing elit. Lorem ipsum dolor sit amet, consectetuer adipiscing elit.</p>
       </div>
 
-      <div role="alert" class="status message submission-error">
+      <div role="alert" class="status message submission-error" tabindex="-1">
         <h3 class="message-title">The following errors occured while processing your registration: </h3>
         <ul class="message-copy"> </ul>
       </div>

From 7452381f1cde08e69a69bd9ce25d0601905733c4 Mon Sep 17 00:00:00 2001
From: Giulio Gratta <giulio@giuliogratta.com>
Date: Mon, 10 Jun 2013 15:58:08 -0700
Subject: [PATCH 2/7] making sequence titles available to non-sighted users

---
 common/lib/xmodule/xmodule/css/sequence/display.scss | 1 -
 lms/templates/seq_module.html                        | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/common/lib/xmodule/xmodule/css/sequence/display.scss b/common/lib/xmodule/xmodule/css/sequence/display.scss
index ed09d5cf0f..5fa7e0196b 100644
--- a/common/lib/xmodule/xmodule/css/sequence/display.scss
+++ b/common/lib/xmodule/xmodule/css/sequence/display.scss
@@ -166,7 +166,6 @@ nav.sequence-nav {
         p {
           background: #333;
           color: #fff;
-          display: none;
           font-family: $sans-serif;
           line-height: lh();
           left: 0px;
diff --git a/lms/templates/seq_module.html b/lms/templates/seq_module.html
index 8b94ff5658..f372d495ca 100644
--- a/lms/templates/seq_module.html
+++ b/lms/templates/seq_module.html
@@ -16,7 +16,7 @@
            data-id="${item['id']}"
            data-element="${idx+1}"
            href="javascript:void(0);">
-            <p>${item['title']}</p>
+            <p class="sr">${item['title']}, ${item['type']}</p>
           </a>
         </li>
         % endfor

From 93da389671dc8ffb4dd3f10616cb5305924e31e6 Mon Sep 17 00:00:00 2001
From: Giulio Gratta <giulio@giuliogratta.com>
Date: Mon, 24 Jun 2013 13:23:20 -0700
Subject: [PATCH 3/7] Removing selection styling from sass files

- removed ::selection css from reset.scss
- removed variables from variables.scss
- removed variable calls from course/base/base.scss and base/base.scss
---
 lms/static/sass/base/_reset.scss       | 3 ---
 lms/static/sass/base/_variables.scss   | 2 --
 lms/static/sass/course/base/_base.scss | 6 ------
 3 files changed, 11 deletions(-)

diff --git a/lms/static/sass/base/_reset.scss b/lms/static/sass/base/_reset.scss
index aa7fc975c8..e774e7a59b 100644
--- a/lms/static/sass/base/_reset.scss
+++ b/lms/static/sass/base/_reset.scss
@@ -9,9 +9,6 @@ html { font-size: 100%; -webkit-text-size-adjust: 100%; -ms-text-size-adjust: 10
 html, button, input, select, textarea { font-family: sans-serif; color: #222; }
 body { margin: 0; font-size: 1em; line-height: 1.4; }
 
-::-moz-selection { background: #fe57a1; color: #fff; text-shadow: none; }
-::selection { background: #fe57a1; color: #fff; text-shadow: none; }
-
 a { color: #00e; }
 a:visited { color: #551a8b; }
 a:hover { color: #06e; }
diff --git a/lms/static/sass/base/_variables.scss b/lms/static/sass/base/_variables.scss
index 30d49f5e0e..67f1215ea2 100644
--- a/lms/static/sass/base/_variables.scss
+++ b/lms/static/sass/base/_variables.scss
@@ -104,8 +104,6 @@ $border-color-4: rgb(252,252,252);
 $link-color: $blue;
 $link-color-d1: $m-blue;
 $link-hover: $pink;
-$selection-color-1: $pink;
-$selection-color-2: #444;
 $site-status-color: $pink;
 
 $button-color: $blue;
diff --git a/lms/static/sass/course/base/_base.scss b/lms/static/sass/course/base/_base.scss
index a1c948d4f5..a750029001 100644
--- a/lms/static/sass/course/base/_base.scss
+++ b/lms/static/sass/course/base/_base.scss
@@ -101,12 +101,6 @@ img {
   max-width: 100%;
 }
 
-::selection, ::-moz-selection, ::-webkit-selection {
-  background: $selection-color-2;
-  color: #fff;
-}
-
-
 .tooltip {
   position: absolute;
   top: 0;

From 86a1e4263dedaee4b53bbdd39a750fecd5bed190 Mon Sep 17 00:00:00 2001
From: Giulio Gratta <giulio@giuliogratta.com>
Date: Mon, 24 Jun 2013 15:00:27 -0700
Subject: [PATCH 4/7] added alt tags to course cover images and find course
 page logo

---
 lms/templates/course.html             | 2 +-
 lms/templates/courseware/courses.html | 4 ++--
 lms/templates/dashboard.html          | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/lms/templates/course.html b/lms/templates/course.html
index e8c7cd5875..e3dd9baf43 100644
--- a/lms/templates/course.html
+++ b/lms/templates/course.html
@@ -19,7 +19,7 @@
       </header>
       <section class="info">
         <div class="cover-image">
-          <img src="${course_image_url(course)}">
+          <img src="${course_image_url(course)}" alt="${course.number} ${get_course_about_section(course, 'title')} Cover Image" />
         </div>
         <div class="desc">
           <p>${get_course_about_section(course, 'short_description')}</p>
diff --git a/lms/templates/courseware/courses.html b/lms/templates/courseware/courses.html
index 7f0d596f4b..f731f0d989 100644
--- a/lms/templates/courseware/courses.html
+++ b/lms/templates/courseware/courses.html
@@ -10,9 +10,9 @@
       <hgroup>
         <div class="logo">
           % if self.stanford_theme_enabled():
-            <img src="${static.url('themes/stanford/images/seal.png')}" />
+            <img src="${static.url('themes/stanford/images/seal.png')}" alt="Stanford Seal Logo" />
           % else:
-            <img src="${static.url('images/edx_bw.png')}" />
+            <img src="${static.url('images/edx_bw.png')}" alt="Black and White edX Logo" />
           % endif
         </div>
         % if self.stanford_theme_enabled():
diff --git a/lms/templates/dashboard.html b/lms/templates/dashboard.html
index c41d753444..835e466ab2 100644
--- a/lms/templates/dashboard.html
+++ b/lms/templates/dashboard.html
@@ -211,11 +211,11 @@
 
           % if course.id in show_courseware_links_for:
             <a href="${course_target}" class="cover">
-              <img src="${course_image_url(course)}" />
+              <img src="${course_image_url(course)}" alt="${course.number} ${course.display_name_with_default} Cover Image" />
             </a>
           % else:
             <div class="cover">
-              <img src="${course_image_url(course)}" />
+              <img src="${course_image_url(course)}" alt="${course.number} ${course.display_name_with_default} Cover Image" />
             </div>
           % endif
 

From a5c9db08899d571340c9eb44a37292c0358a4769 Mon Sep 17 00:00:00 2001
From: Giulio Gratta <giulio@giuliogratta.com>
Date: Mon, 24 Jun 2013 15:11:09 -0700
Subject: [PATCH 5/7] increased text contrast to make text more legible

---
 lms/static/sass/base/_variables.scss                  | 2 ++
 lms/static/sass/course/layout/_courseware_header.scss | 2 +-
 lms/static/sass/multicourse/_account.scss             | 2 +-
 lms/static/sass/multicourse/_dashboard.scss           | 2 +-
 lms/static/sass/shared/_course_object.scss            | 2 +-
 5 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/lms/static/sass/base/_variables.scss b/lms/static/sass/base/_variables.scss
index 67f1215ea2..c0bbcfb9ee 100644
--- a/lms/static/sass/base/_variables.scss
+++ b/lms/static/sass/base/_variables.scss
@@ -61,6 +61,8 @@ $baseFontColor: rgb(60,60,60);
 $lighter-base-font-color: rgb(100,100,100);
 $text-color: $dark-gray;
 
+$dark-trans-bg: rgba(0, 0, 0, .75);
+
 $body-bg: rgb(250,250,250);
 $container-bg: $white;
 $header-image: linear-gradient(-90deg, rgba(255,255,255, 1), rgba(230,230,230, 0.9));
diff --git a/lms/static/sass/course/layout/_courseware_header.scss b/lms/static/sass/course/layout/_courseware_header.scss
index b1c14f930a..2670e55ba4 100644
--- a/lms/static/sass/course/layout/_courseware_header.scss
+++ b/lms/static/sass/course/layout/_courseware_header.scss
@@ -148,7 +148,7 @@ header.global.slim {
     float: left;
     font-size: 0.9em;
     font-weight: 600;
-    color: #777;
+    color: $lighter-base-font-color;
     letter-spacing: 0;
     margin-top: 9px;
     margin-bottom: 0;
diff --git a/lms/static/sass/multicourse/_account.scss b/lms/static/sass/multicourse/_account.scss
index 5fcff57074..ab625a71ee 100644
--- a/lms/static/sass/multicourse/_account.scss
+++ b/lms/static/sass/multicourse/_account.scss
@@ -268,7 +268,7 @@
         @include transition(color 0.15s ease-in-out);
         display: block;
         margin-top: ($baseline/4);
-        color: tint($outer-border-color, 50%);
+        color: $lighter-base-font-color;
         font-size: em(13);
       }
 
diff --git a/lms/static/sass/multicourse/_dashboard.scss b/lms/static/sass/multicourse/_dashboard.scss
index c0dac89199..62bf0d583a 100644
--- a/lms/static/sass/multicourse/_dashboard.scss
+++ b/lms/static/sass/multicourse/_dashboard.scss
@@ -561,7 +561,7 @@
       float: right;
       display: block;
       font-style: italic;
-      color: #a0a0a0;
+      color: $lighter-base-font-color;
       text-decoration: underline;
       font-size: .8em;
       margin-top: 32px;
diff --git a/lms/static/sass/shared/_course_object.scss b/lms/static/sass/shared/_course_object.scss
index bd4a8dc049..8064b0708c 100644
--- a/lms/static/sass/shared/_course_object.scss
+++ b/lms/static/sass/shared/_course_object.scss
@@ -71,7 +71,7 @@
       }
 
       .meta-info {
-        background: rgba(0,0,0, 0.6);
+        background: $dark-trans-bg;
         bottom: 6px;
         border: 1px solid rgba(0,0,0, 0.5);
         @include border-right-radius(2px);

From 1bb71a862139f5b6e136561438392ba98fb19a7d Mon Sep 17 00:00:00 2001
From: Giulio Gratta <giulio@giuliogratta.com>
Date: Tue, 25 Jun 2013 16:10:55 -0700
Subject: [PATCH 6/7] increasing visibility of tab navigation on site by making
 focus state very obvious

---
 lms/static/sass/_discussion.scss                | 4 ----
 lms/static/sass/base/_base.scss                 | 4 ++++
 lms/static/sass/course/courseware/_sidebar.scss | 4 ++--
 lms/static/sass/shared/_course_object.scss      | 5 +++++
 4 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/lms/static/sass/_discussion.scss b/lms/static/sass/_discussion.scss
index ddfe4a88f1..068af0e24a 100644
--- a/lms/static/sass/_discussion.scss
+++ b/lms/static/sass/_discussion.scss
@@ -257,7 +257,6 @@ body.discussion {
         font-size: 11px;
         line-height: 16px;
         color: #333;
-        outline: 0;
       }
     }
 
@@ -932,7 +931,6 @@ body.discussion {
       font-size: 11px;
       line-height: 16px;
       color: #333;
-      outline: 0;
     }
 
     .post-search {
@@ -959,7 +957,6 @@ body.discussion {
       font-size: 13px;
       line-height: 20px;
       color: #333;
-      outline: 0;
       cursor: pointer;
       pointer-events: none;
       @include transition(all .2s ease-out);
@@ -1642,7 +1639,6 @@ body.discussion {
     border-radius: 3px;
     box-shadow: 0 1px 3px rgba(0, 0, 0, .1) inset;
     @include transition(border-color .1s);
-    outline: 0;
 
     &:focus {
       border-color: #4697c1;
diff --git a/lms/static/sass/base/_base.scss b/lms/static/sass/base/_base.scss
index 9314877249..6e6272c010 100644
--- a/lms/static/sass/base/_base.scss
+++ b/lms/static/sass/base/_base.scss
@@ -84,6 +84,10 @@ a:link, a:visited {
   }
 }
 
+a:focus {
+  outline: 3px groove $black;
+}
+
 .content-wrapper {
   width: flex-grid(12);
   margin: 0 auto;
diff --git a/lms/static/sass/course/courseware/_sidebar.scss b/lms/static/sass/course/courseware/_sidebar.scss
index 24bda451a7..206fa42b37 100644
--- a/lms/static/sass/course/courseware/_sidebar.scss
+++ b/lms/static/sass/course/courseware/_sidebar.scss
@@ -5,7 +5,7 @@ section.course-index {
   border-right: 1px solid $border-color-2;
 
   #open_close_accordion {
-    display: none;
+    //display: none;
   }
 
   header {
@@ -23,7 +23,7 @@ section.course-index {
     h3 {
       @include border-radius(0);
       margin: 0;
-      overflow: hidden;
+      overflow: visible;
 
       &:first-child {
         border: none;
diff --git a/lms/static/sass/shared/_course_object.scss b/lms/static/sass/shared/_course_object.scss
index 8064b0708c..9ca2e60511 100644
--- a/lms/static/sass/shared/_course_object.scss
+++ b/lms/static/sass/shared/_course_object.scss
@@ -66,6 +66,11 @@
         width: 0;
       }
 
+      a {
+        position: relative;
+        display: block;
+      }
+
       a:hover {
         text-decoration: none;
       }

From 3c461f7a9429e1b095c0283083156e1c73587239 Mon Sep 17 00:00:00 2001
From: Giulio Gratta <giulio@giuliogratta.com>
Date: Mon, 1 Jul 2013 14:32:06 -0700
Subject: [PATCH 7/7] reverts to browser default focus state and reenables the
 open_close_accordion feature

---
 lms/static/sass/base/_base.scss                 | 6 +++++-
 lms/static/sass/course/courseware/_sidebar.scss | 2 +-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/lms/static/sass/base/_base.scss b/lms/static/sass/base/_base.scss
index 6e6272c010..2ac5b0a1da 100644
--- a/lms/static/sass/base/_base.scss
+++ b/lms/static/sass/base/_base.scss
@@ -85,7 +85,11 @@ a:link, a:visited {
 }
 
 a:focus {
-  outline: 3px groove $black;
+  /**
+   *  Add general focus styling here
+   *  for example:
+   *      outline: 3px groove $black;
+   **/
 }
 
 .content-wrapper {
diff --git a/lms/static/sass/course/courseware/_sidebar.scss b/lms/static/sass/course/courseware/_sidebar.scss
index 206fa42b37..09a89e4d95 100644
--- a/lms/static/sass/course/courseware/_sidebar.scss
+++ b/lms/static/sass/course/courseware/_sidebar.scss
@@ -5,7 +5,7 @@ section.course-index {
   border-right: 1px solid $border-color-2;
 
   #open_close_accordion {
-    //display: none;
+    display: none;
   }
 
   header {