diff options
author | JohnAlbin <virtually.johnalbin@gmail.com> | 2017-05-07 16:07:14 +0300 |
---|---|---|
committer | JohnAlbin <virtually.johnalbin@gmail.com> | 2017-05-10 16:44:45 +0300 |
commit | 9e5bee421e47747ebaf568d7a41560afba1961d2 (patch) | |
tree | 6cb124a822838a351482b9cd8c8e58cc1a15fd87 | |
parent | 63909b01ffa5eedcf9b94d3cc93162a595dcf744 (diff) |
Remove opinionated properties on fieldset element.
-rw-r--r-- | fork-versions/default/_normalize.scss | 6 | ||||
-rw-r--r-- | fork-versions/typey-chroma-kss/base/forms/_forms.scss | 5 | ||||
-rw-r--r-- | fork-versions/typey/_normalize.scss | 5 | ||||
-rw-r--r-- | sass/normalize/_normalize-mixin.scss | 6 | ||||
-rw-r--r-- | test/fixtures/fork-versions/default/output.css | 6 | ||||
-rw-r--r-- | test/fixtures/fork-versions/typey-chroma-kss/output.css | 5 | ||||
-rw-r--r-- | test/fixtures/fork-versions/typey/output.css | 7 | ||||
-rw-r--r-- | test/fixtures/import-now/output.css | 6 | ||||
-rw-r--r-- | test/fixtures/normalize/exclude-multiple/output.css | 6 | ||||
-rw-r--r-- | test/fixtures/normalize/exclude-string/output.css | 6 | ||||
-rw-r--r-- | test/fixtures/variables/default/output.css | 6 | ||||
-rw-r--r-- | test/fixtures/variables/font/output.css | 6 | ||||
-rw-r--r-- | test/fixtures/variables/indent-amount-and-vertical-rhythm/output.css | 6 | ||||
-rw-r--r-- | test/fixtures/variables/indent-amount/output.css | 6 |
14 files changed, 29 insertions, 53 deletions
diff --git a/fork-versions/default/_normalize.scss b/fork-versions/default/_normalize.scss index 561a304..30c5969 100644 --- a/fork-versions/default/_normalize.scss +++ b/fork-versions/default/_normalize.scss @@ -527,13 +527,11 @@ input { } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - border: 1px solid #c0c0c0; - margin: 0 2px; - padding: 0.35em 0.625em 0.75em; + padding: 0.35em 0.75em 0.625em; } /** diff --git a/fork-versions/typey-chroma-kss/base/forms/_forms.scss b/fork-versions/typey-chroma-kss/base/forms/_forms.scss index 3dc5641..8a6fa29 100644 --- a/fork-versions/typey-chroma-kss/base/forms/_forms.scss +++ b/fork-versions/typey-chroma-kss/base/forms/_forms.scss @@ -188,10 +188,9 @@ input { // Style guide: forms.base.fieldset fieldset { - // Change the border, margin, and padding in all browsers (opinionated). - @include padding(.35 .625 .65); + // Correct the padding in Firefox. + @include padding(.35 .5 .65); border: 1px solid color(border); - margin: 0 2px; } legend { diff --git a/fork-versions/typey/_normalize.scss b/fork-versions/typey/_normalize.scss index 1513dd3..ede3847 100644 --- a/fork-versions/typey/_normalize.scss +++ b/fork-versions/typey/_normalize.scss @@ -475,14 +475,13 @@ input { } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { // Apply borders and padding that keep the vertical rhythm. - @include padding(.35 .625 .65); + @include padding(.35 .5 .65); border: 1px solid #c0c0c0; - margin: 0 2px; } /** diff --git a/sass/normalize/_normalize-mixin.scss b/sass/normalize/_normalize-mixin.scss index 653bf76..a366f7b 100644 --- a/sass/normalize/_normalize-mixin.scss +++ b/sass/normalize/_normalize-mixin.scss @@ -539,13 +539,11 @@ } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - border: 1px solid #c0c0c0; - margin: 0 2px; - padding: 0.35em 0.625em 0.75em; + padding: 0.35em 0.75em 0.625em; } /** diff --git a/test/fixtures/fork-versions/default/output.css b/test/fixtures/fork-versions/default/output.css index cc71ebe..90f518f 100644 --- a/test/fixtures/fork-versions/default/output.css +++ b/test/fixtures/fork-versions/default/output.css @@ -337,12 +337,10 @@ input { /* 2 */ } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - border: 1px solid #c0c0c0; - margin: 0 2px; - padding: 0.35em 0.625em 0.75em; } + padding: 0.35em 0.75em 0.625em; } /** * 1. Correct the text wrapping in Edge and IE. diff --git a/test/fixtures/fork-versions/typey-chroma-kss/output.css b/test/fixtures/fork-versions/typey-chroma-kss/output.css index 62ac08f..091d0d9 100644 --- a/test/fixtures/fork-versions/typey-chroma-kss/output.css +++ b/test/fixtures/fork-versions/typey-chroma-kss/output.css @@ -312,9 +312,8 @@ button[disabled], padding: 0; } fieldset { - padding: 0.525rem 0.9375rem 0.975rem; - border: 1px solid #cccccc; - margin: 0 2px; } + padding: 0.525rem 0.75rem 0.975rem; + border: 1px solid #cccccc; } legend { box-sizing: border-box; diff --git a/test/fixtures/fork-versions/typey/output.css b/test/fixtures/fork-versions/typey/output.css index fcfc303..5515493 100644 --- a/test/fixtures/fork-versions/typey/output.css +++ b/test/fixtures/fork-versions/typey/output.css @@ -406,12 +406,11 @@ input { /* 2 */ } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - padding: 0.525rem 0.9375rem 0.975rem; - border: 1px solid #c0c0c0; - margin: 0 2px; } + padding: 0.525rem 0.75rem 0.975rem; + border: 1px solid #c0c0c0; } /** * 1. Correct the text wrapping in Edge and IE. diff --git a/test/fixtures/import-now/output.css b/test/fixtures/import-now/output.css index 966836f..d725150 100644 --- a/test/fixtures/import-now/output.css +++ b/test/fixtures/import-now/output.css @@ -319,12 +319,10 @@ input { /* 2 */ } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - border: 1px solid #c0c0c0; - margin: 0 2px; - padding: 0.35em 0.625em 0.75em; } + padding: 0.35em 0.75em 0.625em; } /** * 1. Correct the text wrapping in Edge and IE. diff --git a/test/fixtures/normalize/exclude-multiple/output.css b/test/fixtures/normalize/exclude-multiple/output.css index 062f8ca..ad16984 100644 --- a/test/fixtures/normalize/exclude-multiple/output.css +++ b/test/fixtures/normalize/exclude-multiple/output.css @@ -248,12 +248,10 @@ input { /* 2 */ } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - border: 1px solid #c0c0c0; - margin: 0 2px; - padding: 0.35em 0.625em 0.75em; } + padding: 0.35em 0.75em 0.625em; } /** * 1. Correct the text wrapping in Edge and IE. diff --git a/test/fixtures/normalize/exclude-string/output.css b/test/fixtures/normalize/exclude-string/output.css index 6d0554a..c9f897a 100644 --- a/test/fixtures/normalize/exclude-string/output.css +++ b/test/fixtures/normalize/exclude-string/output.css @@ -319,12 +319,10 @@ input { /* 2 */ } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - border: 1px solid #c0c0c0; - margin: 0 2px; - padding: 0.35em 0.625em 0.75em; } + padding: 0.35em 0.75em 0.625em; } /** * 1. Correct the text wrapping in Edge and IE. diff --git a/test/fixtures/variables/default/output.css b/test/fixtures/variables/default/output.css index 966836f..d725150 100644 --- a/test/fixtures/variables/default/output.css +++ b/test/fixtures/variables/default/output.css @@ -319,12 +319,10 @@ input { /* 2 */ } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - border: 1px solid #c0c0c0; - margin: 0 2px; - padding: 0.35em 0.625em 0.75em; } + padding: 0.35em 0.75em 0.625em; } /** * 1. Correct the text wrapping in Edge and IE. diff --git a/test/fixtures/variables/font/output.css b/test/fixtures/variables/font/output.css index d80c89c..1e95491 100644 --- a/test/fixtures/variables/font/output.css +++ b/test/fixtures/variables/font/output.css @@ -383,12 +383,10 @@ input { /* 2 */ } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - border: 1px solid #c0c0c0; - margin: 0 2px; - padding: 0.35em 0.625em 0.75em; } + padding: 0.35em 0.75em 0.625em; } /** * 1. Correct the text wrapping in Edge and IE. 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 0cd4ea1..60a333e 100644 --- a/test/fixtures/variables/indent-amount-and-vertical-rhythm/output.css +++ b/test/fixtures/variables/indent-amount-and-vertical-rhythm/output.css @@ -381,12 +381,10 @@ input { /* 2 */ } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - border: 1px solid #c0c0c0; - margin: 0 2px; - padding: 0.35em 0.625em 0.75em; } + padding: 0.35em 0.75em 0.625em; } /** * 1. Correct the text wrapping in Edge and IE. diff --git a/test/fixtures/variables/indent-amount/output.css b/test/fixtures/variables/indent-amount/output.css index 2b4c008..4cffa78 100644 --- a/test/fixtures/variables/indent-amount/output.css +++ b/test/fixtures/variables/indent-amount/output.css @@ -319,12 +319,10 @@ input { /* 2 */ } /** - * Change the border, margin, and padding in all browsers (opinionated). + * Correct the padding in Firefox. */ fieldset { - border: 1px solid #c0c0c0; - margin: 0 2px; - padding: 0.35em 0.625em 0.75em; } + padding: 0.35em 0.75em 0.625em; } /** * 1. Correct the text wrapping in Edge and IE. |