feat(mjml): handle social icons block aligment options (#257)

Automattic · Jul 7, 2020 · ce10875 · ce10875
1 parent 99f2f1f
commit ce10875
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 8 deletions.
diff --git a/includes/class-newspack-newsletters-renderer.php b/includes/class-newspack-newsletters-renderer.php
@@ -403,14 +403,18 @@ private static function render_mjml_component( $block, $is_in_column = false, $i
 				);
 
 				$social_wrapper_attrs = array(
-					'align'         => isset( $attrs['align'] ) && 'center' == $attrs['align'] ? 'center' : 'left',
 					'icon-size'     => '22px',
 					'mode'          => 'horizontal',
 					'padding'       => '0',
 					'border-radius' => '999px',
 					'icon-padding'  => '8px',
 				);
-				$markup               = '<mj-social ' . self::array_to_attributes( $social_wrapper_attrs ) . '>';
+				if ( isset( $attrs['align'] ) ) {
+					$social_wrapper_attrs['align'] = $attrs['align'];
+				} else {
+					$social_wrapper_attrs['align'] = 'left';
+				}
+				$markup = '<mj-social ' . self::array_to_attributes( $social_wrapper_attrs ) . '>';
 				foreach ( $inner_blocks as $link_block ) {
 					if ( isset( $link_block['attrs']['url'] ) ) {
 						$url = $link_block['attrs']['url'];

diff --git a/src/editor/index.js b/src/editor/index.js
@@ -27,12 +27,7 @@ domReady( () => {
 
 addFilter( 'blocks.registerBlockType', 'newspack-newsletters/core-blocks', ( settings, name ) => {
 	/* Remove left/right alignment options wherever possible */
-	if (
-		'core/paragraph' === name ||
-		'core/social-links' === name ||
-		'core/buttons' === name ||
-		'core/columns' === name
-	) {
+	if ( 'core/paragraph' === name || 'core/buttons' === name || 'core/columns' === name ) {
 		settings.supports = { ...settings.supports, align: [] };
 	}
 	if ( 'core/group' === name ) {