after review

elementor · Dec 25, 2024 · 618c076 · 618c076
1 parent c635cc7
commit 618c076
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 22 deletions.
diff --git a/modules/content/assets/scss/hello-plus-zigzag.scss b/modules/content/assets/scss/hello-plus-zigzag.scss
@@ -135,29 +135,32 @@
 		}
 	}
 
-	&.has-direction-start {
-		.ehp-zigzag__item-wrapper:nth-child(odd) {
-			.ehp-zigzag__item-container{
-				flex-direction: row;
+	&.has-direction {
+		&-start {
+			.ehp-zigzag__item-wrapper:nth-child(odd) {
+				.ehp-zigzag__item-container {
+					flex-direction: row;
+				}
 			}
-		}
 
-		.ehp-zigzag__item-wrapper:nth-child(even) {
-			.ehp-zigzag__item-container {
-				flex-direction: row-reverse;
+			.ehp-zigzag__item-wrapper:nth-child(even) {
+				.ehp-zigzag__item-container {
+					flex-direction: row-reverse;
+				}
 			}
 		}
-	}
 
-	&.has-direction-end {
-		.ehp-zigzag__item-wrapper:nth-child(odd) {
-			.ehp-zigzag__item-container {
-				flex-direction: row-reverse;
+		&-end {
+			.ehp-zigzag__item-wrapper:nth-child(odd) {
+				.ehp-zigzag__item-container {
+					flex-direction: row-reverse;
+				}
 			}
-		}
-		.ehp-zigzag__item-wrapper:nth-child(even) {
-			.ehp-zigzag__item-container{
-				flex-direction: row;
+
+			.ehp-zigzag__item-wrapper:nth-child(even) {
+				.ehp-zigzag__item-container {
+					flex-direction: row;
+				}
 			}
 		}
 	}

diff --git a/modules/content/classes/render/widget-zig-zag-render.php b/modules/content/classes/render/widget-zig-zag-render.php
@@ -27,13 +27,16 @@ public function __construct( Zig_Zag $widget ) {
 	}
 
 	public function render(): void {
-		$layout_classnames = self::LAYOUT_CLASSNAME;
 		$first_zigzag_direction = $this->settings['first_zigzag_direction'];
 		$has_alternate_icon_color = $this->settings['has_alternate_icon_color'];
 
-		$layout_classnames .= ' has-direction-' . $first_zigzag_direction;
+		$layout_classnames = [
+			self::LAYOUT_CLASSNAME,
+			'has-direction-' . $first_zigzag_direction,
+		];
+
 		if ( 'yes' === $has_alternate_icon_color ) {
-			$layout_classnames .= ' has-alternate-icon-color';
+			$layout_classnames[] = 'has-alternate-icon-color';
 		}
 
 		$this->widget->add_render_attribute( 'layout', [
@@ -47,9 +50,8 @@ public function render(): void {
 			$repeater = 'image' === $graphic_element ? $this->settings['image_zigzag_items'] : $this->settings['icon_zigzag_items'];
 
 			foreach ( $repeater as $key => $item ) {
-				$item_class = self::ITEM_CLASSNAME;
 				$this->widget->add_render_attribute( 'zigzag-item-' . $key, [
-					'class' => $item_class,
+					'class' => self::ITEM_CLASSNAME,
 				] );
 				?>
 				<div class="ehp-zigzag__item-wrapper">