From 16471b19bccacba50b2332d73c54fe5d86def847 Mon Sep 17 00:00:00 2001 From: JohnAlbin Date: Sun, 7 May 2017 20:27:50 +0800 Subject: [PATCH] Remove opinionated outline rule on "a" element. --- fork-versions/default/_normalize.scss | 10 ---------- fork-versions/typey-chroma-kss/base/links/_links.scss | 7 ------- fork-versions/typey/_normalize.scss | 10 ---------- sass/normalize/_normalize-mixin.scss | 10 ---------- test/fixtures/fork-versions/default/output.css | 8 -------- .../fixtures/fork-versions/typey-chroma-kss/output.css | 4 ---- test/fixtures/fork-versions/typey/output.css | 8 -------- test/fixtures/import-now/output.css | 8 -------- test/fixtures/normalize/exclude-multiple/output.css | 8 -------- test/fixtures/normalize/exclude-single/output.css | 8 -------- test/fixtures/normalize/exclude-string/output.css | 8 -------- test/fixtures/normalize/include-single/output.css | 8 -------- test/fixtures/variables/default/output.css | 8 -------- test/fixtures/variables/font/output.css | 8 -------- .../indent-amount-and-vertical-rhythm/output.css | 8 -------- test/fixtures/variables/indent-amount/output.css | 8 -------- 16 files changed, 129 deletions(-) diff --git a/fork-versions/default/_normalize.scss b/fork-versions/default/_normalize.scss index 98896d6fb..201d6e7ce 100644 --- a/fork-versions/default/_normalize.scss +++ b/fork-versions/default/_normalize.scss @@ -241,16 +241,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ - -a:active, -a:hover { - outline-width: 0; -} - /* Text-level semantics ========================================================================== */ diff --git a/fork-versions/typey-chroma-kss/base/links/_links.scss b/fork-versions/typey-chroma-kss/base/links/_links.scss index b77730f9c..074060b7c 100644 --- a/fork-versions/typey-chroma-kss/base/links/_links.scss +++ b/fork-versions/typey-chroma-kss/base/links/_links.scss @@ -44,13 +44,6 @@ a:active { } } -// Remove the outline on focused links when they are also active or hovered -// in all browsers (opinionated). -a:active, -a:hover { - outline-width: 0; -} - @media print { // Underline all links. :link, diff --git a/fork-versions/typey/_normalize.scss b/fork-versions/typey/_normalize.scss index 7d3f02cac..2a5983ea6 100644 --- a/fork-versions/typey/_normalize.scss +++ b/fork-versions/typey/_normalize.scss @@ -194,16 +194,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ - -a:active, -a:hover { - outline-width: 0; -} - /* Text-level semantics ========================================================================== */ diff --git a/sass/normalize/_normalize-mixin.scss b/sass/normalize/_normalize-mixin.scss index e8f4e9b13..ca640a55b 100644 --- a/sass/normalize/_normalize-mixin.scss +++ b/sass/normalize/_normalize-mixin.scss @@ -269,16 +269,6 @@ background-color: transparent; /* 1 */ -webkit-text-decoration-skip: objects; /* 2 */ } - - /** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ - - a:active, - a:hover { - outline-width: 0; - } } @if _normalize-include(text) { diff --git a/test/fixtures/fork-versions/default/output.css b/test/fixtures/fork-versions/default/output.css index e2ca5e726..b8c1e1f3c 100644 --- a/test/fixtures/fork-versions/default/output.css +++ b/test/fixtures/fork-versions/default/output.css @@ -98,14 +98,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } - /* Text-level semantics ========================================================================== */ /** diff --git a/test/fixtures/fork-versions/typey-chroma-kss/output.css b/test/fixtures/fork-versions/typey-chroma-kss/output.css index 0d23aabb8..62ac08f0c 100644 --- a/test/fixtures/fork-versions/typey-chroma-kss/output.css +++ b/test/fixtures/fork-versions/typey-chroma-kss/output.css @@ -127,10 +127,6 @@ a:active { a:active:not(.button) { background-color: transparent; } -a:active, -a:hover { - outline-width: 0; } - @media print { :link, :visited { diff --git a/test/fixtures/fork-versions/typey/output.css b/test/fixtures/fork-versions/typey/output.css index 8cc6acf47..52109a129 100644 --- a/test/fixtures/fork-versions/typey/output.css +++ b/test/fixtures/fork-versions/typey/output.css @@ -167,14 +167,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } - /* Text-level semantics ========================================================================== */ /** diff --git a/test/fixtures/import-now/output.css b/test/fixtures/import-now/output.css index ba81625d5..5ead397d1 100644 --- a/test/fixtures/import-now/output.css +++ b/test/fixtures/import-now/output.css @@ -96,14 +96,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } - /* Text-level semantics ========================================================================== */ /** diff --git a/test/fixtures/normalize/exclude-multiple/output.css b/test/fixtures/normalize/exclude-multiple/output.css index 9ac09ce3f..d7e06c64d 100644 --- a/test/fixtures/normalize/exclude-multiple/output.css +++ b/test/fixtures/normalize/exclude-multiple/output.css @@ -53,14 +53,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } - /* Text-level semantics ========================================================================== */ /** diff --git a/test/fixtures/normalize/exclude-single/output.css b/test/fixtures/normalize/exclude-single/output.css index febdf6931..a7f0a47da 100644 --- a/test/fixtures/normalize/exclude-single/output.css +++ b/test/fixtures/normalize/exclude-single/output.css @@ -96,14 +96,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } - /* Text-level semantics ========================================================================== */ /** diff --git a/test/fixtures/normalize/exclude-string/output.css b/test/fixtures/normalize/exclude-string/output.css index db6ca3ff0..4f1162162 100644 --- a/test/fixtures/normalize/exclude-string/output.css +++ b/test/fixtures/normalize/exclude-string/output.css @@ -96,14 +96,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } - /* Text-level semantics ========================================================================== */ /** diff --git a/test/fixtures/normalize/include-single/output.css b/test/fixtures/normalize/include-single/output.css index 71978f330..84021998a 100644 --- a/test/fixtures/normalize/include-single/output.css +++ b/test/fixtures/normalize/include-single/output.css @@ -10,11 +10,3 @@ a { /* 1 */ -webkit-text-decoration-skip: objects; /* 2 */ } - -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } diff --git a/test/fixtures/variables/default/output.css b/test/fixtures/variables/default/output.css index ba81625d5..5ead397d1 100644 --- a/test/fixtures/variables/default/output.css +++ b/test/fixtures/variables/default/output.css @@ -96,14 +96,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } - /* Text-level semantics ========================================================================== */ /** diff --git a/test/fixtures/variables/font/output.css b/test/fixtures/variables/font/output.css index 2171a34be..4c98eb3cf 100644 --- a/test/fixtures/variables/font/output.css +++ b/test/fixtures/variables/font/output.css @@ -160,14 +160,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } - /* Text-level semantics ========================================================================== */ /** diff --git a/test/fixtures/variables/indent-amount-and-vertical-rhythm/output.css b/test/fixtures/variables/indent-amount-and-vertical-rhythm/output.css index 937c9d766..13e6992c4 100644 --- a/test/fixtures/variables/indent-amount-and-vertical-rhythm/output.css +++ b/test/fixtures/variables/indent-amount-and-vertical-rhythm/output.css @@ -158,14 +158,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } - /* Text-level semantics ========================================================================== */ /** diff --git a/test/fixtures/variables/indent-amount/output.css b/test/fixtures/variables/indent-amount/output.css index d9a95c1d1..9e961f744 100644 --- a/test/fixtures/variables/indent-amount/output.css +++ b/test/fixtures/variables/indent-amount/output.css @@ -96,14 +96,6 @@ a { -webkit-text-decoration-skip: objects; /* 2 */ } -/** - * Remove the outline on focused links when they are also active or hovered - * in all browsers (opinionated). - */ -a:active, -a:hover { - outline-width: 0; } - /* Text-level semantics ========================================================================== */ /**