Creating a new action type, so we can update the state with one actio…

…n call.
WordPress · Nov 1, 2021 · 5cac465 · 5cac465
1 parent e7cec87
commit 5cac465
Showing 1 changed file with 14 additions and 7 deletions.
diff --git a/packages/block-editor/src/components/list-view/index.js b/packages/block-editor/src/components/list-view/index.js
@@ -35,6 +35,17 @@ const expanded = ( state, action ) => {
 			return { ...state, ...{ [ action.clientId ]: true } };
 		case 'collapse':
 			return { ...state, ...{ [ action.clientId ]: false } };
+		case 'expandAll':
+			return {
+				...state,
+				...action.clientIds.reduce(
+					( newState, id ) => ( {
+						...newState,
+						[ id ]: true,
+					} ),
+					{}
+				),
+			};
 		default:
 			return state;
 	}
@@ -182,13 +193,9 @@ function ListView(
 			Array.isArray( selectedBlockParentClientIds ) &&
 			selectedBlockParentClientIds.length
 		) {
-			selectedBlockParentClientIds.forEach( ( clientId ) => {
-				if ( ! expandedState[ clientId ] ) {
-					setExpandedState( {
-						type: 'expand',
-						clientId,
-					} );
-				}
+			setExpandedState( {
+				type: 'expandAll',
+				clientIds: selectedBlockParentClientIds,
 			} );
 		}
 	}, [ hasFocus, selectedBlockParentClientIds ] );