Changeset 58198

Timestamp:

05/26/2024 08:30:51 AM (6 months ago)

Author:

karmatosed

Message:

Twenty Nineteen: Make h1 consistent for entry title.

The h1 for entry-title was not the same between the editor and front end. This fixes that in the by setting wp-block-post-title font sizes together with h2 and combines post-section-dash mixin for both h1 and h2.

Props sabernhardt, nkeller15, shailu25.
Fixes #58440.

Location:

trunk/src/wp-content/themes/twentynineteen

Files:

• style-editor.css (modified) (4 diffs)
• style-editor.scss (modified) (2 diffs)

trunk/src/wp-content/themes/twentynineteen/style-editor.css

@@ -686 +686 @@
 }
 
-h1:before {
-  background: #767676;
-  content: "\020";
-  display: block;
-  height: 2px;
-  margin: 1rem 0;
-  width: 1em;
-}
-
-h1.has-text-align-center:before {
-  margin-left: auto;
-  margin-right: auto;
-}
-
-h1.has-text-align-right:before {
-  margin-left: auto;
-}
-
 @media only screen and (min-width: 768px) {
   h1 {
@@ -710 +692 @@
 }
 
+.wp-block-post-title,
 h2 {
   font-size: 1.6875em;
 }
 
+@media only screen and (min-width: 768px) {
+  .wp-block-post-title,
+  h2 {
+    font-size: 2.25em;
+  }
+}
+
+h1:before,
 h2:before {
   background: #767676;
@@ -723 +714 @@
 }
 
+h1.has-text-align-center:before,
 h2.has-text-align-center:before {
   margin-left: auto;
@@ -728 +720 @@
 }
 
+h1.has-text-align-right:before,
 h2.has-text-align-right:before {
   margin-left: auto;
-}
-
-@media only screen and (min-width: 768px) {
-  h2 {
-    font-size: 2.25em;
-  }
 }
 

trunk/src/wp-content/themes/twentynineteen/style-editor.scss

@@ -120 +120 @@
 h1 {
     font-size: $font__size-xl;
-    @include post-section-dash;
 
     @include media(tablet) {
@@ -127 +126 @@
 }
 
+.wp-block-post-title,
 h2 {
     font-size: $font__size-lg;
-    @include post-section-dash;
 
     @include media(tablet) {
         font-size: $font__size-xl;
     }
+}
+
+h1,
+h2 {
+    @include post-section-dash;
 }