Updates to speakers block 2

Signed-off-by: James Hunt <10615884+thetwopct@users.noreply.github.com>
linuxfoundation · Sep 10, 2024 · 897c49b · 897c49b
1 parent 47ad963
commit 897c49b
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 14 deletions.
diff --git a/web/wp-content/plugins/speakers-block-2/block-speakers-block.php b/web/wp-content/plugins/speakers-block-2/block-speakers-block.php
@@ -9,7 +9,7 @@
  * Author:            cjyabraham, <a href="https://www.thetwopercent.co.uk">James Hunt</a>
  * License:           GPL-2.0-or-later
  * License URI:       https://www.gnu.org/licenses/gpl-2.0.html
- * Text Domain:       block-speakers-block-2
+ * Text Domain:       speakers-block-2
  *
  * @package           WordPress
  */

diff --git a/web/wp-content/plugins/speakers-block-2/block.json b/web/wp-content/plugins/speakers-block-2/block.json
@@ -30,7 +30,7 @@
 			"type": "string"
 		}
 	},
-	"textdomain": "block-speakers-block-2",
+	"textdomain": "speakers-block-2",
 	"editorScript": "file:./build/index.js",
 	"editorStyle": "file:./build/index.css"
 }
diff --git a/web/wp-content/plugins/speakers-block-2/includes/speaker-block.php b/web/wp-content/plugins/speakers-block-2/includes/speaker-block.php
@@ -47,7 +47,7 @@
 			$speaker_alt = 'Generic Speaker Image';
 			if ( has_post_thumbnail() ) {
 				$speaker_image_id = get_post_thumbnail_id();
-				$speaker_alt = 'Picture of ' . the_title_attribute( 'echo=0' );
+				$speaker_alt = the_title_attribute( 'echo=0' ) . ' headshot';
 			}
 			echo wp_get_attachment_image(
 				$speaker_image_id,
@@ -83,7 +83,7 @@
 			?>
 			<?php
 			if ( $company_logo ) {
-				echo wp_get_attachment_image( $company_logo, 'full', '', array( 'class' => 'sb2-speaker__company-logo', 'alt' => 'Logo of ' . $company ) );
+				echo wp_get_attachment_image( $company_logo, 'full', '', array( 'class' => 'sb2-speaker__company-logo', 'alt' => $company . ' logo' ) );
 			} else {
 				?>
 			<div class="sb2-speaker__company-container">
@@ -115,7 +115,7 @@
 			$speaker_alt = 'Generic Speaker Image';
 			if ( has_post_thumbnail() ) {
 				$speaker_image_id = get_post_thumbnail_id();
-				$speaker_alt = 'Picture of ' . the_title_attribute( 'echo=0' );
+				$speaker_alt = the_title_attribute( 'echo=0' ) . ' headshot';
 			}
 			echo wp_get_attachment_image(
 				$speaker_image_id,
@@ -155,7 +155,7 @@
 							false,
 							array(
 								'class' => 'sb2-speaker__company-logo',
-								'alt' => 'Logo of ' . $company ) );
+								'alt' => $company . ' logo' ) );
 						} else {
 							?>
 							<div class="sb2-speaker__company-container">

diff --git a/web/wp-content/plugins/speakers-block-2/package-lock.json b/web/wp-content/plugins/speakers-block-2/package-lock.json
diff --git a/web/wp-content/plugins/speakers-block-2/package.json b/web/wp-content/plugins/speakers-block-2/package.json
@@ -1,5 +1,5 @@
 {
-	"name": "block-speakers-block-2",
+	"name": "speakers-block-2",
 	"version": "0.3.0",
 	"description": "Gutenberg block which allows for insertion of a Speakers showcase in a page/post. It requires an existing Speakers CPT already setup <a href=\"https://github.com/linuxfoundation/lfevents/blob/main/web/wp-content/mu-plugins/custom/lfevents/admin/class-lfevents-admin.php#L164\">here</a>.",
 	"author": "cjyabraham, James Hunt",

diff --git a/web/wp-content/themes/lfevents/src/scss/blocks/_speakers-block-2.scss b/web/wp-content/themes/lfevents/src/scss/blocks/_speakers-block-2.scss
@@ -73,7 +73,7 @@
   }
   &__title {
     margin-top: 5px;
-    margin-bottom: 10px;
+    margin-bottom: 20px;
     font-weight: 500;
     font-size: 16px;
     line-height: 140%;
@@ -95,11 +95,10 @@
     cursor: pointer;
   }
   &__company-logo {
-    height: 60px;
-    width: auto;
-    max-width: 180px;
+    height: 50px;
     object-fit: contain;
-    object-position: 0%;
+    margin-bottom: 10px;
+    width: min(100%, 92%, 280px);
   }
   &__company-text {
     font-size: 20px;
@@ -288,7 +287,7 @@
   &-sched {
     line-height: 150%;
     letter-spacing: 0.02em;
-    background-color: #EDEDED;
+    background-color: #ededed;
     padding-top: 16px;
     padding-bottom: 16px;
     @media (min-width: 340px) and (max-width: 767px) {