From b6e4bf274ec5e6efdf2b63b656665281253dafa0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alba=20Rinc=C3=B3n?= <alba.rincon@automattic.com>
Date: Tue, 17 May 2022 14:34:26 +0200
Subject: [PATCH] Don't allow alt if the image is a bg not an img element

---
 assets/js/blocks/featured-product/block.js | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/assets/js/blocks/featured-product/block.js b/assets/js/blocks/featured-product/block.js
index db747fa433e..263b309343d 100644
--- a/assets/js/blocks/featured-product/block.js
+++ b/assets/js/blocks/featured-product/block.js
@@ -278,6 +278,8 @@ const FeaturedProduct = ( {
 		// so we need to check if it exists before using it.
 		const focalPointPickerExists = typeof FocalPointPicker === 'function';
 
+		const isImgElement = ! isRepeated && ! hasParallax;
+
 		return (
 			<>
 				<InspectorControls key="inspector">
@@ -404,7 +406,7 @@ const FeaturedProduct = ( {
 											} )
 										}
 									/>
-									{ ! isRepeated && (
+									{ isImgElement && (
 										<TextareaControl
 											label={ __(
 												'Alt text (alternative text)',
@@ -807,6 +809,7 @@ export default compose( [
 			state = {
 				doUrlUpdate: false,
 			};
+
 			componentDidUpdate() {
 				const {
 					attributes,
@@ -828,9 +831,11 @@ export default compose( [
 					this.setState( { doUrlUpdate: false } );
 				}
 			}
+
 			triggerUrlUpdate = () => {
 				this.setState( { doUrlUpdate: true } );
 			};
+
 			render() {
 				return (
 					<ProductComponent
@@ -840,6 +845,7 @@ export default compose( [
 				);
 			}
 		}
+
 		return WrappedComponent;
 	}, 'withUpdateButtonAttributes' ),
 ] )( FeaturedProduct );