From 31df4716c847b114a71bbd82e3b216e3989d3e96 Mon Sep 17 00:00:00 2001
From: Morten Rand-Hendriksen <morten@pinkandyellow.com>
Date: Tue, 2 Jan 2018 14:57:02 -0800
Subject: [PATCH] Refactor the css layout to use grid and flex

Remove #primary

Remove #primary, take 2

Change primary layout module to CSS grid.

Remove ::before / ::after based clearfixes. When using flex and grid, pseudo-elements are considered flex/grid elements and cause unexpected behavior. Clearfix is a tool to resolve clearing when using float-based layouts. Modern flex/grid-based layouts do not need them.

Main menu: Use flex as layout module.

Main menu cleanup.

Comments, Posts, and Post Navigation: Use flex for layout. Automatic RTL.

Remove errant test data.

- Fixed skip to content link to point to main content (#primary).
- Changed ID for <main> to #primary to preserve original separation between main and sidebar (#secondary).
- Removed superfluous ID reference to #secondary in CSS.
- Cleaned up whitespace triggering Travis errors.

Fix wpcs issues

Cleaning the rebase

Compile CSS

Remove primary div from WooCommerce too

grid-template-columns property doesn't support negative values
---
 404.php                           | 22 ++++++-------
 README.md                         |  2 +-
 archive.php                       |  6 ++--
 footer.php                        |  2 --
 header.php                        |  4 +--
 inc/woocommerce.php               |  6 ++--
 index.php                         |  6 ++--
 layouts/content-sidebar.css       | 26 ++++++++-------
 layouts/sidebar-content.css       | 26 ++++++++-------
 page.php                          |  6 ++--
 sass/layout/_content-sidebar.scss | 27 ++++++++-------
 sass/layout/_sidebar-content.scss | 27 ++++++++-------
 sass/mixins/_mixins-master.scss   | 12 -------
 sass/modules/_accessibility.scss  |  2 +-
 sass/modules/_clearings.scss      | 18 ----------
 sass/navigation/_menus.scss       | 21 ++++++------
 sass/shop/_components.scss        |  4 ---
 sass/shop/_products.scss          |  2 --
 sass/shop/_single-product.scss    |  4 ---
 sass/style.scss                   |  6 ----
 search.php                        |  6 ++--
 single.php                        |  6 ++--
 style-rtl.css                     | 55 ++++++++-----------------------
 style.css                         | 55 ++++++++-----------------------
 woocommerce.css                   | 15 ---------
 25 files changed, 123 insertions(+), 243 deletions(-)
 delete mode 100644 sass/modules/_clearings.scss

diff --git a/404.php b/404.php
index ba5d537a..70f199df 100644
--- a/404.php
+++ b/404.php
@@ -10,16 +10,15 @@
 get_header();
 ?>
 
-	<div id="primary" class="content-area">
-		<main id="main" class="site-main">
+	<main id="primary" class="site-main">
 
-			<section class="error-404 not-found">
-				<header class="page-header">
-					<h1 class="page-title"><?php esc_html_e( 'Oops! That page can&rsquo;t be found.', '_s' ); ?></h1>
-				</header><!-- .page-header -->
+		<section class="error-404 not-found">
+			<header class="page-header">
+				<h1 class="page-title"><?php esc_html_e( 'Oops! That page can&rsquo;t be found.', '_s' ); ?></h1>
+			</header><!-- .page-header -->
 
-				<div class="page-content">
-					<p><?php esc_html_e( 'It looks like nothing was found at this location. Maybe try one of the links below or a search?', '_s' ); ?></p>
+			<div class="page-content">
+				<p><?php esc_html_e( 'It looks like nothing was found at this location. Maybe try one of the links below or a search?', '_s' ); ?></p>
 
 					<?php
 					get_search_form();
@@ -52,11 +51,10 @@ get_header();
 					the_widget( 'WP_Widget_Tag_Cloud' );
 					?>
 
-				</div><!-- .page-content -->
-			</section><!-- .error-404 -->
+			</div><!-- .page-content -->
+		</section><!-- .error-404 -->
 
-		</main><!-- #main -->
-	</div><!-- #primary -->
+	</main><!-- #main -->
 
 <?php
 get_footer();
diff --git a/README.md b/README.md
index 9239747f..a88285a8 100644
--- a/README.md
+++ b/README.md
@@ -13,7 +13,7 @@ My ultra-minimal CSS might make me look like theme tartare but that means less s
 * Custom template tags in `inc/template-tags.php` that keep your templates clean and neat and prevent code duplication.
 * Some small tweaks in `inc/template-functions.php` that can improve your theming experience.
 * A script at `js/navigation.js` that makes your menu a toggled dropdown on small screens (like your phone), ready for CSS artistry. It's enqueued in `functions.php`.
-* 2 sample CSS layouts in `layouts/` for a sidebar on either side of your content.
+* 2 sample CSS layouts in `layouts/` made using CSS Grid Layout Module for a sidebar on either side of your content.
 Note: `.no-sidebar` styles are not automatically loaded.
 * Smartly organized starter CSS in `style.css` that will help you to quickly get your design off the ground.
 * Full support for `WooCommerce plugin` integration with hooks in `inc/woocommerce.php`, styling override woocommerce.css with product gallery features (zoom, swipe, lightbox) enabled.
diff --git a/archive.php b/archive.php
index db1313f3..fae599bd 100644
--- a/archive.php
+++ b/archive.php
@@ -10,8 +10,7 @@
 get_header();
 ?>
 
-	<div id="primary" class="content-area">
-		<main id="main" class="site-main">
+	<main id="primary" class="site-main">
 
 		<?php if ( have_posts() ) : ?>
 
@@ -45,8 +44,7 @@ get_header();
 		endif;
 		?>
 
-		</main><!-- #main -->
-	</div><!-- #primary -->
+	</main><!-- #main -->
 
 <?php
 get_sidebar();
diff --git a/footer.php b/footer.php
index 9c0a099e..37fcb0f4 100644
--- a/footer.php
+++ b/footer.php
@@ -11,8 +11,6 @@
 
 ?>
 
-	</div><!-- #content -->
-
 	<footer id="colophon" class="site-footer">
 		<div class="site-info">
 			<a href="<?php echo esc_url( __( 'https://wordpress.org/', '_s' ) ); ?>">
diff --git a/header.php b/header.php
index 527ab38b..7bcd7cf0 100644
--- a/header.php
+++ b/header.php
@@ -23,7 +23,7 @@
 <body <?php body_class(); ?>>
 <?php wp_body_open(); ?>
 <div id="page" class="site">
-	<a class="skip-link screen-reader-text" href="#content"><?php esc_html_e( 'Skip to content', '_s' ); ?></a>
+	<a class="skip-link screen-reader-text" href="#primary"><?php esc_html_e( 'Skip to content', '_s' ); ?></a>
 
 	<header id="masthead" class="site-header">
 		<div class="site-branding">
@@ -57,5 +57,3 @@
 			?>
 		</nav><!-- #site-navigation -->
 	</header><!-- #masthead -->
-
-	<div id="content" class="site-content">
diff --git a/inc/woocommerce.php b/inc/woocommerce.php
index 8a3a8b9d..e900ab6c 100644
--- a/inc/woocommerce.php
+++ b/inc/woocommerce.php
@@ -118,9 +118,8 @@ if ( ! function_exists( '_s_woocommerce_wrapper_before' ) ) {
 	 */
 	function _s_woocommerce_wrapper_before() {
 		?>
-		<div id="primary" class="content-area">
-			<main id="main" class="site-main">
-			<?php
+			<main id="primary" class="site-main">
+		<?php
 	}
 }
 add_action( 'woocommerce_before_main_content', '_s_woocommerce_wrapper_before' );
@@ -136,7 +135,6 @@ if ( ! function_exists( '_s_woocommerce_wrapper_after' ) ) {
 	function _s_woocommerce_wrapper_after() {
 		?>
 			</main><!-- #main -->
-		</div><!-- #primary -->
 		<?php
 	}
 }
diff --git a/index.php b/index.php
index cf3c487a..1826605a 100644
--- a/index.php
+++ b/index.php
@@ -15,8 +15,7 @@
 get_header();
 ?>
 
-	<div id="primary" class="content-area">
-		<main id="main" class="site-main">
+	<main id="primary" class="site-main">
 
 		<?php
 		if ( have_posts() ) :
@@ -51,8 +50,7 @@ get_header();
 		endif;
 		?>
 
-		</main><!-- #main -->
-	</div><!-- #primary -->
+	</main><!-- #main -->
 
 <?php
 get_sidebar();
diff --git a/layouts/content-sidebar.css b/layouts/content-sidebar.css
index 60b85d0b..e1202a55 100644
--- a/layouts/content-sidebar.css
+++ b/layouts/content-sidebar.css
@@ -6,22 +6,26 @@
  * Learn more: https://developer.wordpress.org/themes/basics/template-files/
 */
 
-.content-area {
-	float: left;
-	margin: 0 -25% 0 0;
-	width: 100%;
+.site {
+	display: grid;
+	grid-template-columns: auto 25%;
+	grid-template-areas:
+		"header header"
+		"main sidebar"
+		"footer footer";
+}
+.site-header {
+	grid-area: header;
 }
 .site-main {
-	margin: 0 25% 0 0;
+	grid-area: main;
+	overflow: hidden; /* Resolves issue with <pre> elements forcing full width. */
 }
-.site-content .widget-area {
-	float: right;
-	overflow: hidden;
-	width: 25%;
+.widget-area {
+	grid-area: sidebar;
 }
 .site-footer {
-	clear: both;
-	width: 100%;
+	grid-area: footer;
 }
 
 .no-sidebar .content-area {
diff --git a/layouts/sidebar-content.css b/layouts/sidebar-content.css
index 0dda00ee..0d1f8eed 100644
--- a/layouts/sidebar-content.css
+++ b/layouts/sidebar-content.css
@@ -6,22 +6,26 @@
  * Learn more: https://developer.wordpress.org/themes/basics/template-files/
 */
 
-.content-area {
-	float: right;
-	margin: 0 0 0 -25%;
-	width: 100%;
+.site {
+	display: grid;
+	grid-template-columns: 25% auto;
+	grid-template-areas:
+		"header header"
+		"sidebar main"
+		"footer footer";
+}
+.site-header {
+	grid-area: header;
 }
 .site-main {
-	margin: 0 0 0 25%;
+	grid-area: main;
+	overflow: hidden; /* Resolves issue with <pre> elements forcing full width. */
 }
-.site-content .widget-area {
-	float: left;
-	overflow: hidden;
-	width: 25%;
+.widget-area {
+	grid-area: sidebar;
 }
 .site-footer {
-	clear: both;
-	width: 100%;
+	grid-area: footer;
 }
 
 .no-sidebar .content-area {
diff --git a/page.php b/page.php
index 9c840673..25fc0d13 100644
--- a/page.php
+++ b/page.php
@@ -15,8 +15,7 @@
 get_header();
 ?>
 
-	<div id="primary" class="content-area">
-		<main id="main" class="site-main">
+	<main id="primary" class="site-main">
 
 		<?php
 		while ( have_posts() ) :
@@ -32,8 +31,7 @@ get_header();
 		endwhile; // End of the loop.
 		?>
 
-		</main><!-- #main -->
-	</div><!-- #primary -->
+	</main><!-- #main -->
 
 <?php
 get_sidebar();
diff --git a/sass/layout/_content-sidebar.scss b/sass/layout/_content-sidebar.scss
index 507cc595..760d241a 100644
--- a/sass/layout/_content-sidebar.scss
+++ b/sass/layout/_content-sidebar.scss
@@ -1,22 +1,27 @@
-.content-area {
-	float: left;
-	margin: 0 (-$size__site-sidebar) 0 0;
-	width: $size__site-main;
+.site {
+	display: grid;
+	grid-template-columns: auto ($size__site-sidebar);
+	grid-template-areas:
+		"header header"
+		"main sidebar"
+		"footer footer";
+}
+
+.site-header {
+	grid-area: header;
 }
 
 .site-main {
-	margin: 0 $size__site-sidebar 0 0;
+	grid-area: main;
+	overflow: hidden; /* Resolves issue with <pre> elements forcing full width. */
 }
 
-.site-content .widget-area {
-	float: right;
-	overflow: hidden;
-	width: $size__site-sidebar;
+.widget-area {
+	grid-area: sidebar;
 }
 
 .site-footer {
-	clear: both;
-	width: $size__site-main;
+	grid-area: footer;
 }
 
 @import "no-sidebar";
diff --git a/sass/layout/_sidebar-content.scss b/sass/layout/_sidebar-content.scss
index e765590e..ec312c10 100644
--- a/sass/layout/_sidebar-content.scss
+++ b/sass/layout/_sidebar-content.scss
@@ -1,22 +1,27 @@
-.content-area {
-	float: right;
-	margin: 0 0 0 (-$size__site-sidebar);
-	width: $size__site-main;
+.site {
+	display: grid;
+	grid-template-columns: ($size__site-sidebar) auto;
+	grid-template-areas:
+		"header header"
+		"sidebar main"
+		"footer footer";
+}
+
+.site-header {
+	grid-area: header;
 }
 
 .site-main {
-	margin: 0 0 0 $size__site-sidebar;
+	grid-area: main;
+	overflow: hidden; /* Resolves issue with <pre> elements forcing full width. */
 }
 
-.site-content .widget-area {
-	float: left;
-	overflow: hidden;
-	width: $size__site-sidebar;
+.widget-area {
+	grid-area: sidebar;
 }
 
 .site-footer {
-	clear: both;
-	width: $size__site-main;
+	grid-area: footer;
 }
 
 @import "no-sidebar";
diff --git a/sass/mixins/_mixins-master.scss b/sass/mixins/_mixins-master.scss
index 54ff5336..41155be0 100644
--- a/sass/mixins/_mixins-master.scss
+++ b/sass/mixins/_mixins-master.scss
@@ -11,18 +11,6 @@
 	margin-right: auto;
 }
 
-// Clearfix
-@mixin clearfix() {
-	content: "";
-	display: table;
-	table-layout: fixed;
-}
-
-// Clear after (not all clearfix need this also)
-@mixin clearfix-after() {
-	clear: both;
-}
-
 // Column width with margin
 @mixin column-width($numberColumns: 3) {
 	width: map-get($columns, $numberColumns) - ( ( $columns__margin * ( $numberColumns - 1 ) ) / $numberColumns );
diff --git a/sass/modules/_accessibility.scss b/sass/modules/_accessibility.scss
index b935555b..b93d15bd 100644
--- a/sass/modules/_accessibility.scss
+++ b/sass/modules/_accessibility.scss
@@ -35,6 +35,6 @@
 }
 
 /* Do not show the outline on the skip link target. */
-#content[tabindex="-1"]:focus {
+#primary[tabindex="-1"]:focus {
 	outline: 0;
 }
diff --git a/sass/modules/_clearings.scss b/sass/modules/_clearings.scss
deleted file mode 100644
index 7654d769..00000000
--- a/sass/modules/_clearings.scss
+++ /dev/null
@@ -1,18 +0,0 @@
-.clear,
-.entry-content,
-.comment-content,
-.site-header,
-.site-content,
-.site-footer {
-
-	&::before,
-	&::after {
-
-		@include clearfix;
-	}
-
-	&::after {
-
-		@include clearfix-after;
-	}
-}
diff --git a/sass/navigation/_menus.scss b/sass/navigation/_menus.scss
index d88b0a68..33a526da 100644
--- a/sass/navigation/_menus.scss
+++ b/sass/navigation/_menus.scss
@@ -1,7 +1,5 @@
 .main-navigation {
-	clear: both;
 	display: block;
-	float: left;
 	width: 100%;
 
 	ul {
@@ -27,7 +25,8 @@
 
 				&:hover > ul,
 				&.focus > ul {
-					left: 100%;
+					display: block;
+					left: auto;
 				}
 			}
 
@@ -51,7 +50,6 @@
 	}
 
 	li {
-		float: left;
 		position: relative;
 
 		&:hover > a,
@@ -84,7 +82,7 @@
 	}
 
 	.main-navigation ul {
-		display: block;
+		display: flex;
 	}
 }
 
@@ -94,17 +92,18 @@
 
 	.site-main & {
 		margin: 0 0 1.5em;
-		overflow: hidden;
+	}
+
+	.nav-links {
+		display: flex;
 	}
 
 	.nav-previous {
-		float: left;
-		width: 50%;
+		flex: 1 0 50%;
 	}
 
 	.nav-next {
-		float: right;
-		text-align: right;
-		width: 50%;
+		text-align: end;
+		flex: 1 0 50%;
 	}
 }
diff --git a/sass/shop/_components.scss b/sass/shop/_components.scss
index 9a4633d7..a995d048 100644
--- a/sass/shop/_components.scss
+++ b/sass/shop/_components.scss
@@ -6,8 +6,6 @@
 	margin: 0;
 	padding: 0;
 
-	@include clearfix;
-
 	.cart-contents {
 		text-decoration: none;
 	}
@@ -226,8 +224,6 @@ p.stars {
 .woocommerce-error,
 .woocommerce-noreviews,
 p.no-comments {
-
-	@include clearfix;
 	background-color: $woocommerce__color-success;
 	clear: both;
 }
diff --git a/sass/shop/_products.scss b/sass/shop/_products.scss
index 5d39372e..7ff3bfe4 100644
--- a/sass/shop/_products.scss
+++ b/sass/shop/_products.scss
@@ -1,6 +1,4 @@
 ul.products {
-
-	@include clearfix;
 	margin: 0;
 	padding: 0;
 
diff --git a/sass/shop/_single-product.scss b/sass/shop/_single-product.scss
index e8bb8676..54513403 100644
--- a/sass/shop/_single-product.scss
+++ b/sass/shop/_single-product.scss
@@ -1,8 +1,6 @@
 .single-product {
 
 	div.product {
-
-		@include clearfix;
 		position: relative;
 
 		.woocommerce-product-gallery {
@@ -22,8 +20,6 @@
 			}
 
 			.flex-control-thumbs {
-
-				@include clearfix;
 				margin: 0;
 				padding: 0;
 
diff --git a/sass/style.scss b/sass/style.scss
index 445764d2..0344f4ce 100644
--- a/sass/style.scss
+++ b/sass/style.scss
@@ -32,7 +32,6 @@ Nicolas Gallagher and Jonathan Neal https://necolas.github.io/normalize.css/
 	## Menus
 # Accessibility
 # Alignments
-# Clearings
 # Widgets
 # Content
 	## Posts and pages
@@ -80,11 +79,6 @@ Nicolas Gallagher and Jonathan Neal https://necolas.github.io/normalize.css/
 --------------------------------------------------------------*/
 @import "modules/alignments";
 
-/*--------------------------------------------------------------
-# Clearings
---------------------------------------------------------------*/
-@import "modules/clearings";
-
 /*--------------------------------------------------------------
 # Widgets
 --------------------------------------------------------------*/
diff --git a/search.php b/search.php
index d0914266..eafd3737 100644
--- a/search.php
+++ b/search.php
@@ -10,8 +10,7 @@
 get_header();
 ?>
 
-	<section id="primary" class="content-area">
-		<main id="main" class="site-main">
+	<main id="primary" class="site-main">
 
 		<?php if ( have_posts() ) : ?>
 
@@ -47,8 +46,7 @@ get_header();
 		endif;
 		?>
 
-		</main><!-- #main -->
-	</section><!-- #primary -->
+	</main><!-- #main -->
 
 <?php
 get_sidebar();
diff --git a/single.php b/single.php
index 86319dc2..bd982141 100644
--- a/single.php
+++ b/single.php
@@ -10,8 +10,7 @@
 get_header();
 ?>
 
-	<div id="primary" class="content-area">
-		<main id="main" class="site-main">
+	<main id="primary" class="site-main">
 
 		<?php
 		while ( have_posts() ) :
@@ -34,8 +33,7 @@ get_header();
 		endwhile; // End of the loop.
 		?>
 
-		</main><!-- #main -->
-	</div><!-- #primary -->
+	</main><!-- #main -->
 
 <?php
 get_sidebar();
diff --git a/style-rtl.css b/style-rtl.css
index 7aae515e..a1484cb9 100644
--- a/style-rtl.css
+++ b/style-rtl.css
@@ -32,7 +32,6 @@ Nicolas Gallagher and Jonathan Neal https://necolas.github.io/normalize.css/
 	## Menus
 # Accessibility
 # Alignments
-# Clearings
 # Widgets
 # Content
 	## Posts and pages
@@ -626,9 +625,7 @@ a:active {
 ## Menus
 --------------------------------------------------------------*/
 .main-navigation {
-	clear: both;
 	display: block;
-	float: right;
 	width: 100%;
 }
 
@@ -655,7 +652,8 @@ a:active {
 
 .main-navigation ul ul li:hover > ul,
 .main-navigation ul ul li.focus > ul {
-	right: 100%;
+	display: block;
+	right: auto;
 }
 
 .main-navigation ul ul a {
@@ -668,7 +666,6 @@ a:active {
 }
 
 .main-navigation li {
-	float: right;
 	position: relative;
 }
 
@@ -690,7 +687,7 @@ a:active {
 	}
 
 	.main-navigation ul {
-		display: block;
+		display: flex;
 	}
 }
 
@@ -700,22 +697,25 @@ a:active {
 .site-main
 .post-navigation {
 	margin: 0 0 1.5em;
-	overflow: hidden;
+}
+
+.comment-navigation .nav-links,
+.posts-navigation .nav-links,
+.post-navigation .nav-links {
+	display: flex;
 }
 
 .comment-navigation .nav-previous,
 .posts-navigation .nav-previous,
 .post-navigation .nav-previous {
-	float: right;
-	width: 50%;
+	flex: 1 0 50%;
 }
 
 .comment-navigation .nav-next,
 .posts-navigation .nav-next,
 .post-navigation .nav-next {
-	float: left;
-	text-align: left;
-	width: 50%;
+	text-align: end;
+	flex: 1 0 50%;
 }
 
 /*--------------------------------------------------------------
@@ -758,7 +758,7 @@ a:active {
 }
 
 /* Do not show the outline on the skip link target. */
-#content[tabindex="-1"]:focus {
+#primary[tabindex="-1"]:focus {
 	outline: 0;
 }
 
@@ -787,35 +787,6 @@ a:active {
 	margin-bottom: 1.5em;
 }
 
-/*--------------------------------------------------------------
-# Clearings
---------------------------------------------------------------*/
-.clear::before,
-.clear::after,
-.entry-content::before,
-.entry-content::after,
-.comment-content::before,
-.comment-content::after,
-.site-header::before,
-.site-header::after,
-.site-content::before,
-.site-content::after,
-.site-footer::before,
-.site-footer::after {
-	content: "";
-	display: table;
-	table-layout: fixed;
-}
-
-.clear::after,
-.entry-content::after,
-.comment-content::after,
-.site-header::after,
-.site-content::after,
-.site-footer::after {
-	clear: both;
-}
-
 /*--------------------------------------------------------------
 # Widgets
 --------------------------------------------------------------*/
diff --git a/style.css b/style.css
index ca1e0eaf..22c5bb0e 100644
--- a/style.css
+++ b/style.css
@@ -32,7 +32,6 @@ Nicolas Gallagher and Jonathan Neal https://necolas.github.io/normalize.css/
 	## Menus
 # Accessibility
 # Alignments
-# Clearings
 # Widgets
 # Content
 	## Posts and pages
@@ -626,9 +625,7 @@ a:active {
 ## Menus
 --------------------------------------------------------------*/
 .main-navigation {
-	clear: both;
 	display: block;
-	float: left;
 	width: 100%;
 }
 
@@ -655,7 +652,8 @@ a:active {
 
 .main-navigation ul ul li:hover > ul,
 .main-navigation ul ul li.focus > ul {
-	left: 100%;
+	display: block;
+	left: auto;
 }
 
 .main-navigation ul ul a {
@@ -668,7 +666,6 @@ a:active {
 }
 
 .main-navigation li {
-	float: left;
 	position: relative;
 }
 
@@ -690,7 +687,7 @@ a:active {
 	}
 
 	.main-navigation ul {
-		display: block;
+		display: flex;
 	}
 }
 
@@ -700,22 +697,25 @@ a:active {
 .site-main
 .post-navigation {
 	margin: 0 0 1.5em;
-	overflow: hidden;
+}
+
+.comment-navigation .nav-links,
+.posts-navigation .nav-links,
+.post-navigation .nav-links {
+	display: flex;
 }
 
 .comment-navigation .nav-previous,
 .posts-navigation .nav-previous,
 .post-navigation .nav-previous {
-	float: left;
-	width: 50%;
+	flex: 1 0 50%;
 }
 
 .comment-navigation .nav-next,
 .posts-navigation .nav-next,
 .post-navigation .nav-next {
-	float: right;
-	text-align: right;
-	width: 50%;
+	text-align: end;
+	flex: 1 0 50%;
 }
 
 /*--------------------------------------------------------------
@@ -758,7 +758,7 @@ a:active {
 }
 
 /* Do not show the outline on the skip link target. */
-#content[tabindex="-1"]:focus {
+#primary[tabindex="-1"]:focus {
 	outline: 0;
 }
 
@@ -787,35 +787,6 @@ a:active {
 	margin-bottom: 1.5em;
 }
 
-/*--------------------------------------------------------------
-# Clearings
---------------------------------------------------------------*/
-.clear::before,
-.clear::after,
-.entry-content::before,
-.entry-content::after,
-.comment-content::before,
-.comment-content::after,
-.site-header::before,
-.site-header::after,
-.site-content::before,
-.site-content::after,
-.site-footer::before,
-.site-footer::after {
-	content: "";
-	display: table;
-	table-layout: fixed;
-}
-
-.clear::after,
-.entry-content::after,
-.comment-content::after,
-.site-header::after,
-.site-content::after,
-.site-footer::after {
-	clear: both;
-}
-
 /*--------------------------------------------------------------
 # Widgets
 --------------------------------------------------------------*/
diff --git a/woocommerce.css b/woocommerce.css
index 0d5c41b6..7153d559 100644
--- a/woocommerce.css
+++ b/woocommerce.css
@@ -72,9 +72,6 @@ table.shop_table_responsive tr td.download-actions .button {
  * Products
  */
 ul.products {
-	content: "";
-	display: table;
-	table-layout: fixed;
 	margin: 0;
 	padding: 0;
 }
@@ -143,9 +140,6 @@ ul.products li.product .button {
  * Single product
  */
 .single-product div.product {
-	content: "";
-	display: table;
-	table-layout: fixed;
 	position: relative;
 }
 
@@ -167,9 +161,6 @@ ul.products li.product .button {
 }
 
 .single-product div.product .woocommerce-product-gallery .flex-control-thumbs {
-	content: "";
-	display: table;
-	table-layout: fixed;
 	margin: 0;
 	padding: 0;
 }
@@ -284,9 +275,6 @@ ul.products li.product .button {
 	position: relative;
 	margin: 0;
 	padding: 0;
-	content: "";
-	display: table;
-	table-layout: fixed;
 }
 
 .site-header-cart .cart-contents {
@@ -469,9 +457,6 @@ p.stars.selected a:not(.active)::before {
 .woocommerce-error,
 .woocommerce-noreviews,
 p.no-comments {
-	content: "";
-	display: table;
-	table-layout: fixed;
 	background-color: #0f834d;
 	clear: both;
 }