Also close settings menu after other items

See #15313
WordPress · May 10, 2019 · 957e0c8 · 957e0c8
1 parent 7d4b257
commit 957e0c8
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/packages/block-editor/src/components/block-settings-menu/index.js b/packages/block-editor/src/components/block-settings-menu/index.js
@@ -70,7 +70,10 @@ export function BlockSettingsMenu( { clientIds, onSelect } ) {
 							{ ! isLocked && canDuplicate && (
 								<MenuItem
 									className="editor-block-settings-menu__control block-editor-block-settings-menu__control"
-									onClick={ onDuplicate }
+									onClick={ () => {
+										onClose();
+										onDuplicate();
+									} }
 									icon="admin-page"
 									shortcut={ shortcuts.duplicate.display }
 								>
@@ -81,15 +84,21 @@ export function BlockSettingsMenu( { clientIds, onSelect } ) {
 								<>
 									<MenuItem
 										className="editor-block-settings-menu__control block-editor-block-settings-menu__control"
-										onClick={ onInsertBefore }
+										onClick={ () => {
+											onClose();
+											onInsertBefore();
+										} }
 										icon="insert-before"
 										shortcut={ shortcuts.insertBefore.display }
 									>
 										{ __( 'Insert Before' ) }
 									</MenuItem>
 									<MenuItem
 										className="editor-block-settings-menu__control block-editor-block-settings-menu__control"
-										onClick={ onInsertAfter }
+										onClick={ () => {
+											onClose();
+											onInsertAfter();
+										} }
 										icon="insert-after"
 										shortcut={ shortcuts.insertAfter.display }
 									>