From 869bd23937448fdcbae37c1062d373ef3ae7722a Mon Sep 17 00:00:00 2001 From: Mark Otto Date: Sun, 27 Nov 2016 17:05:29 -0800 Subject: [PATCH] Update clearfix mixin (#21224) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Update clearfix to use block instead of table display (also reorder properties for linting) * update docs snippet for clearfix mixin—was apparently still using Less syntax and had old clearfix hack (even before the block change in this PR) --- docs/utilities/clearfix.md | 11 ++++------- scss/mixins/_clearfix.scss | 2 +- 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/docs/utilities/clearfix.md b/docs/utilities/clearfix.md index a023a79aaf..49410df098 100644 --- a/docs/utilities/clearfix.md +++ b/docs/utilities/clearfix.md @@ -12,13 +12,10 @@ Easily clear `float`s by adding `.clearfix` **to the parent element**. Utilizes {% highlight scss %} // Mixin itself -.clearfix() { - &:before, - &:after { - content: " "; - display: table; - } - &:after { +@mixin clearfix() { + &::after { + display: block; + content: ""; clear: both; } } diff --git a/scss/mixins/_clearfix.scss b/scss/mixins/_clearfix.scss index d0ae125f3d..b72cf27128 100644 --- a/scss/mixins/_clearfix.scss +++ b/scss/mixins/_clearfix.scss @@ -1,7 +1,7 @@ @mixin clearfix() { &::after { + display: block; content: ""; - display: table; clear: both; } }