- Patch #639460 by Jeff Burnz, effulgentsia, Jacine, mgifford, james.elliott: evaluate CSS of #skip-link, .element-focusable, and upcoming 'disable overlay' links for their impact on contrib/custom themes.

3 files changed, 33 insertions, 40 deletions

diff --git a/themes/bartik/css/style.css b/themes/bartik/css/style.css
index a1b556afc..bc6b63243 100644
--- a/themes/bartik/css/style.css
+++ b/themes/bartik/css/style.css
@@ -220,7 +220,7 @@ ul.tips {
   margin-left: -5.25em;
   margin-top: 0;
   position: absolute;
-  width: auto !important;
+  width: auto;
   z-index: 50;
 }
 #skip-link a,
@@ -231,12 +231,9 @@ ul.tips {
   color: #fff;
   display: block;
   font-size: 0.94em;
-  height: 1px;
   line-height: 1.7;
-  position: absolute;
+  padding: 1px 10px 2px 10px;
   text-decoration: none;
-  top: -10000px;
-  width: 1px;
   -khtml-border-radius: 0 0 10px 10px;
   -moz-border-radius: 0 0 10px 10px;
   -webkit-border-top-left-radius: 0;
@@ -248,12 +245,7 @@ ul.tips {
 #skip-link a:hover,
 #skip-link a:active,
 #skip-link a:focus {
-  height: auto;
   outline: 0;
-  overflow: visible;
-  padding: 1px 10px 2px 10px;
-  position: static;
-  width: auto;
 }
 #logo,
 #preview #preview-header #preview-logo {
diff --git a/themes/garland/style.css b/themes/garland/style.css
index 4434a2d2b..82de2381e 100644
--- a/themes/garland/style.css
+++ b/themes/garland/style.css
@@ -312,29 +312,38 @@ span.form-required {
 /**
  * Skip link
  */
+#skip-link {
+  left: 50%;
+  margin-left: -5.25em;
+  margin-top: 0;
+  position: absolute;
+  width: auto;
+  z-index: 1000;
+}
+#skip-link a,
 #skip-link a:link,
 #skip-link a:visited {
-  font-weight: bold;
-  background: #fff;
-  padding: 0px 5px;
+  background: #444;
+  background: rgba(0, 0, 0, 0.6);
+  color: #fff;
+  display: block;
+  font-size: 0.94em;
+  line-height: 1.7;
+  margin-top: 1px;
+  padding: 2px 10px;
   text-decoration: none;
-  font-size: 80%;
-  text-align: right;
-  top: -99em;
-  left: auto;
-  position: absolute;
+  -khtml-border-radius: 0 0 2px 2px;
+  -moz-border-radius: 0 0 2px 2px;
+  -webkit-border-top-left-radius: 0;
+  -webkit-border-top-right-radius: 0;
+  -webkit-border-bottom-left-radius: 2px;
+  -webkit-border-bottom-right-radius: 2px;
+  border-radius: 0 0 2px 2px;
 }
-
 #skip-link a:hover,
-#skip-link a:focus,
-#skip-link a:active  {
-  height: auto;
-  width: auto;
-  overflow: visible;
-  top:0;
-  left:0;
-  position: fixed;
-  z-index: 1000;
+#skip-link a:active,
+#skip-link a:focus {
+  outline: 0;
 }
 
 /**
diff --git a/themes/seven/style.css b/themes/seven/style.css
index 3bf32e818..e64f8723f 100644
--- a/themes/seven/style.css
+++ b/themes/seven/style.css
@@ -158,15 +158,11 @@ pre {
 #skip-link a,
 #skip-link a:link,
 #skip-link a:visited {
-  position: absolute;
   display: block;
-  top: auto;
-  left: -10000px;
-  width: 1px;
-  height: 1px;
   background: #444;
   color: #fff;
   font-size: 0.94em;
+  padding: 1px 10px 2px 10px;
   text-decoration: none;
   border-radius:0 0 10px 10px;
   -moz-border-radius: 0 0 10px 10px;
@@ -176,13 +172,9 @@ pre {
   -webkit-border-bottom-right-radius: 10px;
 }
 #skip-link a:hover,
-#skip-link a:active,
-#skip-link a:focus {
-  position: static;
-  width: auto;
-  height: auto;
-  overflow: visible;
-  padding: 1px 10px 2px 10px;
+#skip-link a:focus,
+#skip-link a:active {
+  outline: 0;
 }
 
 /**