make it less broken

WordPress · Jan 10, 2024 · c9e8c11 · c9e8c11
1 parent 125e1f2
commit c9e8c11
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/packages/block-editor/src/components/block-popover/inbetween.js b/packages/block-editor/src/components/block-popover/inbetween.js
@@ -68,7 +68,7 @@ function BlockPopoverInbetween( {
 	);
 	const previousElement = useBlockElement( previousClientId );
 	const nextElement = useBlockElement( nextClientId );
-	const isVertical = operation !== 'group' && orientation === 'vertical';
+	const isVertical = orientation === 'vertical';
 
 	const popoverAnchor = useMemo( () => {
 		if (
@@ -99,7 +99,14 @@ function BlockPopoverInbetween( {
 				let width = 0;
 				let height = 0;
 
-				if ( isVertical ) {
+				if ( operation === 'group' ) {
+					// If the operation is group, nextRect is the target to be grouped.
+					// Not sure if previousRect is needed here.
+					top = nextRect ? nextRect.top : previousRect.top;
+					width = nextRect ? nextRect.width : previousRect.width;
+					height = nextRect ? nextRect.bottom - nextRect.top : 0;
+					left = nextRect ? nextRect.left : previousRect.left;
+				} else if ( isVertical ) {
 					// vertical
 					top = previousRect ? previousRect.bottom : nextRect.top;
 					width = previousRect ? previousRect.width : nextRect.width;