Focus styles

openedx · Feb 12, 2016 · 3655775 · 3655775
1 parent 1394219
commit 3655775
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 9 deletions.
diff --git a/cms/static/sass/_base.scss b/cms/static/sass/_base.scss
@@ -657,9 +657,13 @@ hr.divider {
   border-bottom: 1px solid $gray-l4;
   padding: ($baseline*0.75) ($baseline/2);
 
-  &:focus, &:active {
+  &:focus,
+  &:active {
     position: relative;
     top: auto;
+    width: auto;
+    height: auto;
+    margin: 0;
   }
 }
 
@@ -722,4 +726,3 @@ hr.divider {
     color: $gray-l1;
   }
 }
-
diff --git a/cms/static/sass/elements/_navigation.scss b/cms/static/sass/elements/_navigation.scss
@@ -34,9 +34,13 @@ nav {
   border-bottom: 1px solid $gray-l4;
   padding: ($baseline*0.75) ($baseline/2);
 
-  &:focus, &:active {
+  &:focus,
+  &:active {
     position: relative;
     top: auto;
+    width: auto;
+    height: auto;
+    margin: 0;
   }
 }
 

diff --git a/lms/static/sass/base/_base.scss b/lms/static/sass/base/_base.scss
@@ -114,10 +114,6 @@ a, a:visited {
   margin: 0 auto;
   background: $content-wrapper-bg;
 
-  &:focus {
-    outline: none;
-  }
-
   @media print {
     padding-bottom: 0;
   }
@@ -349,8 +345,12 @@ mark {
   border-bottom: 1px solid $border-color-4;
   padding: ($baseline*0.75) ($baseline/2);
 
-  &:focus, &:active {
+  &:focus,
+  &:active {
     position: relative;
     top: auto;
+    width: auto;
+    height: auto;
+    margin: 0;
   }
 }
diff --git a/lms/templates/wiki/article.html b/lms/templates/wiki/article.html
@@ -13,7 +13,7 @@
 
   <div class="article-wrapper">
 
-    <article class="main-article" id="main-article">
+    <article class="main-article" id="main-article" tabindex="-1">
         {% if selected_tab != "edit" %}
           <h1>{{ article.current_revision.title }}</h1>