Browse Source

Merge pull request #1008 from Automattic/fix/issue-1003

Sophisticated Business: Make sure the first block has margin-top: 0
Jeffrey Pearce 6 years ago
parent
commit
cf665ae9c6

+ 7 - 0
sophisticated-business/sass/blocks/_blocks.scss

@@ -86,6 +86,13 @@
 	}
 }
 
+/*
+ * Make sure the first block has margin-top: 0
+ */
+.entry:not(.has-post-thumbnail) .entry-content > :first-child {
+	margin-top: 0;
+}
+
 /*
  * Unset nested content selector styles
  * - Prevents layout styles from cascading too deeply

+ 3 - 2
sophisticated-business/sass/site/primary/_posts-and-pages.scss

@@ -31,18 +31,19 @@
 
 	.entry-header {
 
-		margin: calc(3 * #{ $size__spacing-unit}) $size__spacing-unit $size__spacing-unit;
+		margin: calc(3 * #{ $size__spacing-unit}) $size__spacing-unit $size__vertical-spacing-unit;
 		position: relative;
 		text-align: center;
 
 		@include media(tablet) {
-			margin: calc(3 * #{ $size__spacing-unit}) $size__site-margins $size__spacing-unit;
+			margin: calc(3 * #{ $size__spacing-unit}) $size__site-margins $size__vertical-spacing-unit;
 		}
 	}
 
 	.entry-title {
 
 		margin: 0;
+		margin-bottom: $size__vertical-spacing-unit;
 
 		a {
 			color: inherit;

+ 10 - 2
sophisticated-business/style-rtl.css

@@ -2287,19 +2287,20 @@ body.page .main-navigation {
 }
 
 .entry .entry-header {
-  margin: calc(3 * 1rem) 1rem 1rem;
+  margin: calc(3 * 1rem) 1rem 32px;
   position: relative;
   text-align: center;
 }
 
 @media only screen and (min-width: 768px) {
   .entry .entry-header {
-    margin: calc(3 * 1rem) calc(10% + 60px) 1rem;
+    margin: calc(3 * 1rem) calc(10% + 60px) 32px;
   }
 }
 
 .entry .entry-title {
   margin: 0;
+  margin-bottom: 32px;
 }
 
 .entry .entry-title a {
@@ -3388,6 +3389,13 @@ body.page .main-navigation {
   }
 }
 
+/*
+ * Make sure the first block has margin-top: 0
+ */
+.entry:not(.has-post-thumbnail) .entry-content > :first-child {
+  margin-top: 0;
+}
+
 /*
  * Unset nested content selector styles
  * - Prevents layout styles from cascading too deeply

+ 10 - 2
sophisticated-business/style.css

@@ -2293,19 +2293,20 @@ body.page .main-navigation {
 }
 
 .entry .entry-header {
-  margin: calc(3 * 1rem) 1rem 1rem;
+  margin: calc(3 * 1rem) 1rem 32px;
   position: relative;
   text-align: center;
 }
 
 @media only screen and (min-width: 768px) {
   .entry .entry-header {
-    margin: calc(3 * 1rem) calc(10% + 60px) 1rem;
+    margin: calc(3 * 1rem) calc(10% + 60px) 32px;
   }
 }
 
 .entry .entry-title {
   margin: 0;
+  margin-bottom: 32px;
 }
 
 .entry .entry-title a {
@@ -3400,6 +3401,13 @@ body.page .main-navigation {
   }
 }
 
+/*
+ * Make sure the first block has margin-top: 0
+ */
+.entry:not(.has-post-thumbnail) .entry-content > :first-child {
+  margin-top: 0;
+}
+
 /*
  * Unset nested content selector styles
  * - Prevents layout styles from cascading too deeply