Blocks: Disable Convert to Reusable for nested blocks

WordPress · Feb 14, 2018 · 8872058 · 8872058
1 parent ad1fd15
commit 8872058
Showing 1 changed file with 15 additions and 2 deletions.
diff --git a/editor/components/block-settings-menu/reusable-block-settings.js b/editor/components/block-settings-menu/reusable-block-settings.js
@@ -15,17 +15,28 @@ import { isReusableBlock } from '@wordpress/blocks';
 /**
  * Internal dependencies
  */
-import { getBlock, getReusableBlock } from '../../store/selectors';
+import {
+	getBlock,
+	getBlockOrder,
+	getReusableBlock,
+} from '../../store/selectors';
 import { convertBlockToStatic, convertBlockToReusable, deleteReusableBlock } from '../../store/actions';
 
-export function ReusableBlockSettings( { reusableBlock, onConvertToStatic, onConvertToReusable, onDelete } ) {
+export function ReusableBlockSettings( {
+	reusableBlock,
+	isValidForConvert,
+	onConvertToStatic,
+	onConvertToReusable,
+	onDelete,
+} ) {
 	return (
 		<Fragment>
 			{ ! reusableBlock && (
 				<IconButton
 					className="editor-block-settings-menu__control"
 					icon="controls-repeat"
 					onClick={ onConvertToReusable }
+					disabled={ ! isValidForConvert }
 				>
 					{ __( 'Convert to Reusable Block' ) }
 				</IconButton>
@@ -56,7 +67,9 @@ export function ReusableBlockSettings( { reusableBlock, onConvertToStatic, onCon
 export default connect(
 	( state, { uid } ) => {
 		const block = getBlock( state, uid );
+
 		return {
+			isValidForConvert: ! getBlockOrder( state, block.uid ).length,
 			reusableBlock: isReusableBlock( block ) ? getReusableBlock( state, block.attributes.ref ) : null,
 		};
 	},