Remove unnecessary wrapper functions, unused imports, clean up code, …

…onClick => onSelect
WordPress · Jun 13, 2023 · 3701a7b · 3701a7b
1 parent e01b967
commit 3701a7b
Show file tree

Hide file tree

Showing 7 changed files with 98 additions and 108 deletions.
diff --git a/packages/block-editor/src/components/block-settings-menu-controls/README.md b/packages/block-editor/src/components/block-settings-menu-controls/README.md
@@ -13,7 +13,7 @@ function ReusableBlocksMenuItems() {
 		<BlockSettingsMenuControls>
 			{/* Is this a breaking change? */}
 			{/* Should this use a menu item example? */}
-			{ () => <MyToggleButton onToggle={ /* ... */ } /> }
+			<MyToggleButton onToggle={ /* ... */ } />
 		</BlockSettingsMenuControls>
 	);
 }

diff --git a/packages/block-editor/src/components/block-settings-menu-controls/index.js b/packages/block-editor/src/components/block-settings-menu-controls/index.js
@@ -7,7 +7,6 @@ import {
 	privateApis as componentsPrivateApis,
 } from '@wordpress/components';
 import { useSelect } from '@wordpress/data';
-import { pipe } from '@wordpress/compose';
 import { __ } from '@wordpress/i18n';
 
 /**

diff --git a/packages/block-editor/src/components/block-settings-menu/block-settings-dropdown.js b/packages/block-editor/src/components/block-settings-menu/block-settings-dropdown.js
@@ -426,7 +426,7 @@ export function BlockSettingsDropdown( {
 							<DropdownMenuSeparatorV2 />
 							<DropdownMenuGroupV2>
 								<DropdownMenuItemV2
-									onClick={ pipe(
+									onSelect={ pipe(
 										onRemove,
 										updateSelectionAfterRemove
 									) }

diff --git a/packages/block-editor/src/hooks/content-lock-ui.js b/packages/block-editor/src/hooks/content-lock-ui.js
@@ -130,31 +130,27 @@ export const withBlockControls = createHigherOrderComponent(
 				) }
 				{ showStartEditingAsBlocks && (
 					<BlockSettingsMenuControls>
-						{ () => (
-							/* TODO: check if this used in other legacy dropdown menus */
-							<DropdownMenuItemV2
-								onSelect={ () => {
-									__unstableMarkNextChangeAsNotPersistent();
-									updateBlockAttributes( props.clientId, {
-										templateLock: undefined,
-									} );
-									updateBlockListSettings( props.clientId, {
-										...getBlockListSettings(
-											props.clientId
-										),
-										templateLock: false,
-									} );
-									focusModeToRevert.current =
-										getSettings().focusMode;
-									updateSettings( { focusMode: true } );
-									__unstableSetTemporarilyEditingAsBlocks(
-										props.clientId
-									);
-								} }
-							>
-								{ __( 'Modify' ) }
-							</DropdownMenuItemV2>
-						) }
+						{ /* TODO: check if this used in other legacy dropdown menus */ }
+						<DropdownMenuItemV2
+							onSelect={ () => {
+								__unstableMarkNextChangeAsNotPersistent();
+								updateBlockAttributes( props.clientId, {
+									templateLock: undefined,
+								} );
+								updateBlockListSettings( props.clientId, {
+									...getBlockListSettings( props.clientId ),
+									templateLock: false,
+								} );
+								focusModeToRevert.current =
+									getSettings().focusMode;
+								updateSettings( { focusMode: true } );
+								__unstableSetTemporarilyEditingAsBlocks(
+									props.clientId
+								);
+							} }
+						>
+							{ __( 'Modify' ) }
+						</DropdownMenuItemV2>
 					</BlockSettingsMenuControls>
 				) }
 				<BlockEdit key="edit" { ...props } />

diff --git a/packages/block-library/src/template-part/edit/index.js b/packages/block-library/src/template-part/edit/index.js
@@ -163,28 +163,26 @@ export default function TemplatePartEdit( {
 				) }
 				{ canReplace && (
 					<BlockSettingsMenuControls>
-						{ () => (
-							/* TODO: check if this used in other legacy dropdown menus */
-							<DropdownMenuItemV2
-								onSelect={ ( event ) => {
-									setIsTemplatePartSelectionOpen( true );
-									// Keep the dropdown menu open
-									event.preventDefault();
-								} }
-							>
-								{ createInterpolateElement(
-									__( 'Replace <BlockTitle />' ),
-									{
-										BlockTitle: (
-											<BlockTitle
-												clientId={ clientId }
-												maximumLength={ 25 }
-											/>
-										),
-									}
-								) }
-							</DropdownMenuItemV2>
-						) }
+						{ /* TODO: check if this used in other legacy dropdown menus */ }
+						<DropdownMenuItemV2
+							onSelect={ ( event ) => {
+								setIsTemplatePartSelectionOpen( true );
+								// Keep the dropdown menu open
+								event.preventDefault();
+							} }
+						>
+							{ createInterpolateElement(
+								__( 'Replace <BlockTitle />' ),
+								{
+									BlockTitle: (
+										<BlockTitle
+											clientId={ clientId }
+											maximumLength={ 25 }
+										/>
+									),
+								}
+							) }
+						</DropdownMenuItemV2>
 					</BlockSettingsMenuControls>
 				) }
 				{ isEntityAvailable && (

diff --git a/packages/edit-post/src/components/block-settings-menu/plugin-block-settings-menu-item.js b/packages/edit-post/src/components/block-settings-menu/plugin-block-settings-menu-item.js
@@ -6,7 +6,6 @@ import {
 	Icon,
 	privateApis as componentsPrivateApis,
 } from '@wordpress/components';
-import { compose } from '@wordpress/compose';
 
 /**
  * Internal dependencies
@@ -97,7 +96,7 @@ const PluginBlockSettingsMenuItem = ( {
 	role,
 } ) => (
 	<BlockSettingsMenuControls>
-		{ ( { selectedBlocks /*onClose*/ } ) => {
+		{ ( { selectedBlocks } ) => {
 			if ( ! shouldRenderItem( selectedBlocks, allowedBlocks ) ) {
 				return null;
 			}

diff --git a/...eusable-blocks/src/components/reusable-blocks-menu-items/reusable-block-convert-button.js b/...eusable-blocks/src/components/reusable-blocks-menu-items/reusable-block-convert-button.js
@@ -112,69 +112,67 @@ export default function ReusableBlockConvertButton( {
 	/* TODO: check if this used in other legacy dropdown menus */
 	return (
 		<BlockSettingsMenuControls>
-			{ () => (
-				<>
-					<DropdownMenuItemV2
-						prefix={ <Icon icon={ symbol } size={ 24 } /> }
-						onSelect={ ( event ) => {
-							setIsModalOpen( true );
-							// Keep the dropdown menu open
-							event.preventDefault();
+			<>
+				<DropdownMenuItemV2
+					prefix={ <Icon icon={ symbol } size={ 24 } /> }
+					onSelect={ ( event ) => {
+						setIsModalOpen( true );
+						// Keep the dropdown menu open
+						event.preventDefault();
+					} }
+				>
+					{ __( 'Create Reusable block' ) }
+				</DropdownMenuItemV2>
+				{ isModalOpen && (
+					<Modal
+						title={ __( 'Create Reusable block' ) }
+						onRequestClose={ () => {
+							setIsModalOpen( false );
+							setTitle( '' );
 						} }
+						overlayClassName="reusable-blocks-menu-items__convert-modal"
 					>
-						{ __( 'Create Reusable block' ) }
-					</DropdownMenuItemV2>
-					{ isModalOpen && (
-						<Modal
-							title={ __( 'Create Reusable block' ) }
-							onRequestClose={ () => {
+						<form
+							onSubmit={ ( event ) => {
+								event.preventDefault();
+								onConvert( title );
 								setIsModalOpen( false );
 								setTitle( '' );
+								// TODO: the modal is expected to close the dropdown!
+								// - can we keep it open?
+								// - can we close the dropdown when the modal is opened instead?
+								// - otherwise, how do we go about it? We'd need to switch to
+								//   controlled mode at the root, and pass `onClose` around
+								// onClose();
 							} }
-							overlayClassName="reusable-blocks-menu-items__convert-modal"
 						>
-							<form
-								onSubmit={ ( event ) => {
-									event.preventDefault();
-									onConvert( title );
-									setIsModalOpen( false );
-									setTitle( '' );
-									// TODO: the modal is expected to close the dropdown!
-									// - can we keep it open?
-									// - can we close the dropdown when the modal is opened instead?
-									// - otherwise, how do we go about it? We'd need to switch to
-									//   controlled mode at the root, and pass `onClose` around
-									// onClose();
-								} }
-							>
-								<VStack spacing="5">
-									<TextControl
-										__nextHasNoMarginBottom
-										label={ __( 'Name' ) }
-										value={ title }
-										onChange={ setTitle }
-									/>
-									<HStack justify="right">
-										<Button
-											variant="tertiary"
-											onClick={ () => {
-												setIsModalOpen( false );
-												setTitle( '' );
-											} }
-										>
-											{ __( 'Cancel' ) }
-										</Button>
+							<VStack spacing="5">
+								<TextControl
+									__nextHasNoMarginBottom
+									label={ __( 'Name' ) }
+									value={ title }
+									onChange={ setTitle }
+								/>
+								<HStack justify="right">
+									<Button
+										variant="tertiary"
+										onClick={ () => {
+											setIsModalOpen( false );
+											setTitle( '' );
+										} }
+									>
+										{ __( 'Cancel' ) }
+									</Button>
 
-										<Button variant="primary" type="submit">
-											{ __( 'Save' ) }
-										</Button>
-									</HStack>
-								</VStack>
-							</form>
-						</Modal>
-					) }
-				</>
-			) }
+									<Button variant="primary" type="submit">
+										{ __( 'Save' ) }
+									</Button>
+								</HStack>
+							</VStack>
+						</form>
+					</Modal>
+				) }
+			</>
 		</BlockSettingsMenuControls>
 	);
 }