Revert perf optimization

docs/reference-guides/data/data-core-block-editor.md

@@ -1191,10 +1191,6 @@ _Parameters_
 
 Action that hides the insertion point.
 
-_Returns_
-
--   `Object`: Action object.
-
 ### insertAfterBlock
 
 Action that inserts an empty block after a given block.
@@ -1516,6 +1512,10 @@ _Parameters_
 -   _index_ `?number`: Index at which block should be inserted.
 -   _\_\_unstableOptions_ `Object`: Whether or not to show an inserter button.
 
+_Returns_
+
+-   `Object`: Action object.
+
 ### startDraggingBlocks
 
 Returns an action object used in signalling that the user has begun to drag blocks.

packages/block-editor/src/components/block-list/use-in-between-inserter.js

@@ -146,7 +146,6 @@ export function useInBetweenInserter() {
 
 				showInsertionPoint( rootClientId, index, {
 					__unstableWithInserter: true,
-					delay: 500,
 				} );
 			}
 

packages/block-editor/src/components/block-tools/insertion-point.js

@@ -149,13 +149,13 @@ function InsertionPointPopover( {
 			...( ! isVertical ? horizontalLine.rest : verticalLine.rest ),
 			opacity: 1,
 			borderRadius: '2px',
-			transition: { delay: isInserterShown ? 0.1 : 0, type: 'tween' },
+			transition: { delay: isInserterShown ? 0.5 : 0, type: 'tween' },
 		},
 		hover: {
 			...( ! isVertical ? horizontalLine.hover : verticalLine.hover ),
 			opacity: 1,
 			borderRadius: '2px',
-			transition: { delay: 0.1, type: 'tween' },
+			transition: { delay: 0.5, type: 'tween' },
 		},
 	};
 
@@ -165,7 +165,7 @@ function InsertionPointPopover( {
 		},
 		rest: {
 			scale: 1,
-			transition: { type: 'tween' },
+			transition: { delay: 0.4, type: 'tween' },
 		},
 	};
 

packages/block-editor/src/store/actions.js

@@ -621,49 +621,35 @@ export const insertBlocks =
 		}
 	};
 
-const delayedInsertionPointCalls = new WeakMap();
-
 /**
  * Action that shows the insertion point.
  *
  * @param {?string} rootClientId      Optional root client ID of block list on
  *                                    which to insert.
  * @param {?number} index             Index at which block should be inserted.
  * @param {Object}  __unstableOptions Whether or not to show an inserter button.
+ *
+ * @return {Object} Action object.
  */
-export const showInsertionPoint =
-	( rootClientId, index, __unstableOptions = {} ) =>
-	( { dispatch, registry } ) => {
-		const previousCall = delayedInsertionPointCalls.get( registry );
-		if ( previousCall ) {
-			clearTimeout( previousCall );
-		}
-		const { __unstableWithInserter, delay } = __unstableOptions;
-		delayedInsertionPointCalls.set(
-			registry,
-			setTimeout( () => {
-				dispatch( {
-					type: 'SHOW_INSERTION_POINT',
-					rootClientId,
-					index,
-					__unstableWithInserter,
-				} );
-			}, delay )
-		);
+export function showInsertionPoint(
+	rootClientId,
+	index,
+	__unstableOptions = {}
+) {
+	const { __unstableWithInserter } = __unstableOptions;
+	return {
+		type: 'SHOW_INSERTION_POINT',
+		rootClientId,
+		index,
+		__unstableWithInserter,
 	};
-
+}
 /**
  * Action that hides the insertion point.
- *
- * @return {Object} Action object.
  */
 export const hideInsertionPoint =
 	() =>
-	( { select, dispatch, registry } ) => {
-		const previousCall = delayedInsertionPointCalls.get( registry );
-		if ( previousCall ) {
-			clearTimeout( previousCall );
-		}
+	( { select, dispatch } ) => {
 		if ( ! select.isBlockInsertionPointVisible() ) {
 			return;
 		}