Changeset 8253 for sites/trunk/wordcamp.org/public_html/wp-content/mu-plugins/blocks/assets/src/speakers/block-content.scss

Timestamp:

02/15/2019 01:12:00 AM (2 years ago)

Author:

coreymckrill

Message:

WordCamp Blocks: Code adjustments to the Speakers block

This includes a wide range of changes and enhancements based on
code review and feedback. See the following for more info:

• ​https://github.com/WordPress/wordcamp.org/pull/13
• ​https://github.com/WordPress/wordcamp.org/pull/12

File:

• sites/trunk/wordcamp.org/public_html/wp-content/mu-plugins/blocks/assets/src/speakers/block-content.scss (modified) (3 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/mu-plugins/blocks/assets/src/speakers/block-content.scss

@@ -6 +6 @@
 
 .wordcamp-speakers-block {
-    &,
-    .editor-styles-wrapper & {
+    /* The double-ampersand defeats an overzealous ul rule in Core's editor-blocks.css. */
+    /* https://css-tricks.com/the-sass-ampersand/#article-header-id-11 */
+    &#{&},
+    .block-editor &#{&} {
         list-style: none;
+        margin: 0 0 1em 0;
+    }
+
+    &.layout-list > li {
+        margin-bottom: 1em;
+        &:last-child {
+            margin-bottom: 0;
+        }
     }
 
@@ -43 +53 @@
 
 .wordcamp-speaker-session-list {
-    &,
-    .editor-styles-wrapper & {
+    /* See note above about the double-ampersand. */
+    ul#{&},
+    .block-editor ul#{&} {
         list-style: none;
+        margin: 0
     }
 }
 
-.wordcamp-speaker-avatar {
-    &.align-left,
-    &.align-center,
-    &.align-right {
-        display: block;
-    }
-
+.wordcamp-speaker-avatar-container {
     &.align-left {
         float: left;
@@ -64 +70 @@
         margin-left: auto;
         margin-right: auto;
+        text-align: center;
     }