@wordpress/block-editor 15.10.1-next.ba3aee3a2.0 → 15.11.0

package/src/store/selectors.js

@@ -2154,24 +2154,22 @@ const buildBlockTypeItem =
 			'inserter'
 		);
 		const blockVariations = getBlockVariations( blockType.name, 'block' );
-		// Combine inserter and block variations. Block-scope variations without
-		// inserter scope are searchable via slash commands but hidden from browse.
-		const inserterVariationNames = new Set(
-			inserterVariations.map( ( variation ) => variation.name )
-		);
 		const allVariations = [
 			...inserterVariations,
+			// Built-in heading level variations have block scope but allow
+			// insertion via slash inserter.
+			// See https://github.com/WordPress/gutenberg/issues/74233.
 			...blockVariations
 				.filter(
 					( variation ) =>
-						! inserterVariationNames.has( variation.name )
+						blockType.name === 'core/heading' &&
+						[ 'h1', 'h2', 'h3', 'h4', 'h5', 'h6' ].includes(
+							variation.name
+						)
 				)
 				.map( ( variation ) => ( {
 					...variation,
 					isSearchOnly: true,
-					// Block-scope `isDefault` is for the placeholder picker,
-					// not for the inserter, so don't carry it over.
-					isDefault: false,
 				} ) ),
 		];
 		return {
@@ -2281,13 +2279,21 @@ export const getInserterItems = createRegistrySelector( ( select ) =>
 						)
 				);
 			} else {
+				const { getClosestAllowedInsertionPoint } = unlock(
+					select( STORE_NAME )
+				);
 				blockTypeInserterItems = blockTypeInserterItems
-					.filter( ( blockType ) =>
-						isBlockVisibleInTheInserter(
-							state,
-							blockType,
-							rootClientId
-						)
+					.filter(
+						( blockType ) =>
+							isBlockVisibleInTheInserter(
+								state,
+								blockType,
+								rootClientId
+							) &&
+							getClosestAllowedInsertionPoint(
+								blockType.name,
+								rootClientId
+							) !== null
 					)
 					.map( ( blockType ) => ( {
 						...blockType,

package/src/store/test/private-selectors.js

@@ -20,6 +20,7 @@ import {
 	isRemoveLockedBlock,
 	isLockedBlock,
 	isBlockHidden,
+	areBlocksHiddenAnywhere,
 } from '../private-selectors';
 import { getBlockEditingMode } from '../selectors';
 import { deviceTypeKey } from '../private-keys';
@@ -1244,4 +1245,83 @@ describe( 'private selectors', () => {
 			expect( result ).toBe( false );
 		} );
 	} );
+
+	describe( 'areBlocksHiddenAnywhere', () => {
+		it( 'should return false when clientIds array is empty', () => {
+			const state = {
+				blocks: {
+					attributes: new Map(),
+				},
+			};
+			expect( areBlocksHiddenAnywhere( state, [] ) ).toBe( false );
+			expect( areBlocksHiddenAnywhere( state, null ) ).toBe( false );
+			expect( areBlocksHiddenAnywhere( state, undefined ) ).toBe( false );
+		} );
+
+		it( 'should return false when no blocks are hidden', () => {
+			const state = {
+				blocks: {
+					attributes: new Map( [
+						[ 'block-1', { metadata: { blockVisibility: true } } ],
+						[ 'block-2', { metadata: {} } ],
+					] ),
+				},
+			};
+			expect(
+				areBlocksHiddenAnywhere( state, [ 'block-1', 'block-2' ] )
+			).toBe( false );
+		} );
+
+		it( 'should return true when a block is hidden everywhere', () => {
+			const state = {
+				blocks: {
+					attributes: new Map( [
+						[ 'block-1', { metadata: { blockVisibility: false } } ],
+						[ 'block-2', { metadata: { blockVisibility: true } } ],
+					] ),
+				},
+			};
+			expect(
+				areBlocksHiddenAnywhere( state, [ 'block-1', 'block-2' ] )
+			).toBe( true );
+		} );
+
+		it( 'should return true when a block is hidden in any viewport', () => {
+			const state = {
+				blocks: {
+					attributes: new Map( [
+						[
+							'block-1',
+							{
+								metadata: {
+									blockVisibility: {
+										mobile: false,
+										tablet: true,
+									},
+								},
+							},
+						],
+						[ 'block-2', { metadata: { blockVisibility: true } } ],
+					] ),
+				},
+			};
+			expect(
+				areBlocksHiddenAnywhere( state, [ 'block-1', 'block-2' ] )
+			).toBe( true );
+		} );
+
+		it( 'should return false when clientId is null or undefined', () => {
+			const state = {
+				blocks: {
+					attributes: new Map( [
+						[ 'block-1', { metadata: { blockVisibility: false } } ],
+					] ),
+				},
+			};
+			expect( areBlocksHiddenAnywhere( state, [ null ] ) ).toBe( false );
+			expect( areBlocksHiddenAnywhere( state, [ undefined ] ) ).toBe(
+				false
+			);
+		} );
+	} );
 } );

package/src/style.scss

@@ -67,3 +67,4 @@
 @use "./components/block-toolbar/style.scss" as *;
 @use "./components/inserter/style.scss" as *;
 @use "./components/spacing-sizes-control/style.scss" as *;
+@use "./components/block-visibility/style.scss" as *;

package/src/components/block-visibility/styles.scss

@@ -1,10 +0,0 @@
-@use "@wordpress/base-styles/variables" as *;
-
-.block-editor-block-visibility-info {
-	padding-top: $grid-unit-05;
-	padding-bottom: $grid-unit-05;
-	margin: 0 $grid-unit-20 $grid-unit-20;
-	display: flex;
-	align-items: center;
-	justify-content: start;
-}