feat: hide navigation if only one post presented

Automattic · Apr 6, 2021 · c439d60 · c439d60
1 parent 9660922
commit c439d60
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/blocks/carousel/edit.js b/src/blocks/carousel/edit.js
@@ -137,6 +137,7 @@ class Edit extends Component {
 		);
 		const dateFormat = __experimentalGetSettings().formats.date;
 		const hasNoPosts = latestPosts && ! latestPosts.length;
+		const hasOnePost = latestPosts && latestPosts.length === 1;
 		return (
 			<Fragment>
 				<div className={ classes } ref={ this.carouselRef }>
@@ -212,7 +213,7 @@ class Edit extends Component {
 									</article>
 								) ) }
 							</div>
-							{ ! hasNoPosts && (
+							{ ! hasNoPosts && ! hasOnePost && (
 								<>
 									<button
 										className="amp-carousel-button amp-carousel-button-prev swiper-button-prev"

diff --git a/src/blocks/carousel/view.php b/src/blocks/carousel/view.php
@@ -236,7 +236,7 @@ function newspack_blocks_render_block_carousel( $attributes ) {
 			'<div class="swiper-pagination-bullets amp-pagination">%s</div>',
 			implode( '', $buttons )
 		);
-		$navigation  = sprintf(
+		$navigation  = 1 === $counter ? '' : sprintf(
 			'<button class="swiper-button swiper-button-prev" aria-label="%s"></button><button class="swiper-button swiper-button-next" aria-label="%s"></button>',
 			esc_attr__( 'Previous Slide', 'newspack-blocks' ),
 			esc_attr__( 'Next Slide', 'newspack-blocks' )
@@ -257,6 +257,9 @@ function newspack_blocks_render_block_carousel( $attributes ) {
 		$data_attributes[] = sprintf( 'data-autoplay_delay=%s', esc_attr( $delay ) );
 	}
 	Newspack_Blocks::enqueue_view_assets( 'carousel' );
+	if ( 1 === $counter ) {
+		$selector = '';
+	}
 	return sprintf(
 		'<div class="%1$s" id="wp-block-newspack-carousel__%2$d" %3$s>%4$s%5$s%6$s</div>',
 		esc_attr( $classes ),