From 22fcac2ceac9edf1f7fdffd27215e17833b6d68b Mon Sep 17 00:00:00 2001
From: Mark Otto <markdotto@gmail.com>
Date: Sat, 31 Dec 2016 12:26:11 -0800
Subject: [PATCH] Rip out the tab-focus mixin, remove application to anchors in
reboot
---
scss/_mixins.scss | 1 -
scss/_reboot.scss | 6 +-----
scss/mixins/_tab-focus.scss | 9 ---------
3 files changed, 1 insertion(+), 15 deletions(-)
delete mode 100644 scss/mixins/_tab-focus.scss
diff --git a/scss/_mixins.scss b/scss/_mixins.scss
index d207a335b2..da4738297c 100644
--- a/scss/_mixins.scss
+++ b/scss/_mixins.scss
@@ -26,7 +26,6 @@
@import "mixins/resize";
@import "mixins/screen-reader";
@import "mixins/size";
-@import "mixins/tab-focus";
@import "mixins/reset-text";
@import "mixins/text-emphasis";
@import "mixins/text-hide";
diff --git a/scss/_reboot.scss b/scss/_reboot.scss
index e90d7cd3b3..3c0ffe13e9 100644
--- a/scss/_reboot.scss
+++ b/scss/_reboot.scss
@@ -166,10 +166,6 @@ a {
color: $link-hover-color;
text-decoration: $link-hover-decoration;
}
-
- &:focus {
- @include tab-focus();
- }
}
// And undo these styles for placeholder links/named anchors (without href)
@@ -188,7 +184,7 @@ a:not([href]):not([tabindex]) {
}
&:focus {
- outline: none;
+ outline: 0;
}
}
diff --git a/scss/mixins/_tab-focus.scss b/scss/mixins/_tab-focus.scss
deleted file mode 100644
index f16ed6428a..0000000000
--- a/scss/mixins/_tab-focus.scss
+++ /dev/null
@@ -1,9 +0,0 @@
-// WebKit-style focus
-
-@mixin tab-focus() {
- // WebKit-specific. Other browsers will keep their default outline style.
- // (Initially tried to also force default via `outline: initial`,
- // but that seems to erroneously remove the outline in Firefox altogether.)
- outline: 5px auto -webkit-focus-ring-color;
- outline-offset: -2px;
-}
--
GitLab