RNMobile: Ensure playsInline attribute conforms when falling back t…

…o core Video block (#35981) * fix: Rename playsInline to conform to Video block Co-Authored-By: Carlos Garcia <[email protected]> * docs: Add CHANGELOG entry --------- Co-authored-by: Carlos Garcia <[email protected]>
Automattic · Feb 28, 2024 · 69c0650 · 69c0650
1 parent 2415cf1
commit 69c0650
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/...ects/plugins/jetpack/changelog/rnmobile-ensure-playsinline-setting-confirms-to-core-video b/...ects/plugins/jetpack/changelog/rnmobile-ensure-playsinline-setting-confirms-to-core-video
@@ -0,0 +1,4 @@
+Significance: patch
+Type: other
+
+RNMobile: Ensure playsInline setting always conforms to Video block schema.
diff --git a/projects/plugins/jetpack/extensions/blocks/videopress/save.js b/projects/plugins/jetpack/extensions/blocks/videopress/save.js
@@ -42,7 +42,14 @@ const VideoPressSave = CoreVideoSave => props => {
 		 * @see https://github.com/WordPress/gutenberg/blob/c5f9bd88125282a0c35f887cc8d835f065893112/packages/editor/src/hooks/generated-class-name.js#L42
 		 * @see https://github.com/Automattic/wp-calypso/pull/30546#issuecomment-463637946
 		 */
-		return CoreVideoSave( props );
+		// Rename `playsinline` to `playsInline` to conform the block schema of core Video block.
+		const { playsinline: videoPressPlayinline, ...restAttributes } = props.attributes;
+		const coreVideoAttributes = { ...restAttributes, playsInline: playsinline };
+
+		return CoreVideoSave( {
+			...props,
+			attributes: coreVideoAttributes,
+		} );
 	}
 
 	const url = getVideoPressUrl( guid, {