fix (image alt): only override the image alt if it's empty and don't …

…remove it when clearing the image. fixes gambitph#2524

src/block-components/image/edit.js

@@ -95,7 +95,6 @@ const Controls = props => {
 						imageUrl: '',
 						imageWidthAttribute: '',
 						imageHeightAttribute: '',
-						imageAlt: '',
 					} ) }
 					onChange={ image => {
 						// Get the URL of the currently selected image size.
@@ -115,7 +114,7 @@ const Controls = props => {
 							imageUrl: url,
 							imageWidthAttribute: width,
 							imageHeightAttribute: height,
-							imageAlt: image.alt || '',
+							...( attributes.imageAlt ? {} : { imageAlt: image.alt || '' } ), // Only set the alt if it's empty.
 						} )
 					} }
 				/>

src/block-components/image/use-image.js

@@ -1,18 +1,21 @@
 /**
  * WordPress Dependencies
  */
-import { useBlockSetAttributesContext } from '~stackable/hooks'
+import { useBlockAttributesContext, useBlockSetAttributesContext } from '~stackable/hooks'
 
 export const useImage = () => {
 	const setAttributes = useBlockSetAttributesContext()
+	const attributes = useBlockAttributesContext( attributes => ( {
+		imageAlt: attributes.imageAlt,
+	} ) )
 
 	const onChange = image => {
 		setAttributes( {
 			imageUrl: image.url,
 			imageId: image.id,
 			imageHeightAttribute: image.height,
 			imageWidthAttribute: image.width,
-			imageAlt: image.alt,
+			...( attributes.imageAlt ? {} : { imageAlt: image.alt || '' } ), // Only add the image alt if it's empty.
 		} )
 	}
 
@@ -79,7 +82,6 @@ export const useImage = () => {
 			imageId: '',
 			imageHeightAttribute: '',
 			imageWidthAttribute: '',
-			imageAlt: '',
 		} )
 	}