@wordpress/components 39.0.0 → 40.1.1-next.v.202608281122.0

package/src/menu-items-choice/stories/index.story.tsx

@@ -2,6 +2,7 @@ import type { Meta, StoryFn } from '@storybook/react-vite';
 import { useState } from '@wordpress/element';
 import MenuItemsChoice from '..';
 import MenuGroup from '../../menu-group';
+import { NavigableMenu } from '../../navigable-container';
 
 const meta: Meta< typeof MenuItemsChoice > = {
 	tags: [ 'manifest' ],
@@ -35,17 +36,19 @@ const Template: StoryFn< typeof MenuItemsChoice > = ( {
 	const [ choice, setChoice ] = useState( choices[ 0 ]?.value ?? '' );
 
 	return (
-		<MenuGroup label="Editor">
-			<MenuItemsChoice
-				choices={ choices }
-				value={ choice }
-				onSelect={ ( ...selectArgs ) => {
-					onSelect( ...selectArgs );
-					setChoice( ...selectArgs );
-				} }
-				onHover={ onHover }
-			/>
-		</MenuGroup>
+		<NavigableMenu>
+			<MenuGroup label="Editor">
+				<MenuItemsChoice
+					choices={ choices }
+					value={ choice }
+					onSelect={ ( ...selectArgs ) => {
+						onSelect( ...selectArgs );
+						setChoice( ...selectArgs );
+					} }
+					onHover={ onHover }
+				/>
+			</MenuGroup>
+		</NavigableMenu>
 	);
 };
 

package/src/modal/index.tsx

@@ -195,6 +195,7 @@ function UnforwardedModal(
 			! event.defaultPrevented
 		) {
 			event.preventDefault();
+			event.stopPropagation();
 			closeModal().then( () => onRequestClose( event ) );
 		}
 	}

package/src/modal/test/index.tsx

@@ -82,6 +82,25 @@ describe( 'Modal', () => {
 		expect( onRequestClose ).toHaveBeenCalled();
 	} );
 
+	it( 'should stop an Escape key press from propagating when it dismisses the modal', async () => {
+		const user = userEvent.setup();
+		const documentKeyDownHandler = jest.fn();
+		document.addEventListener( 'keydown', documentKeyDownHandler );
+
+		try {
+			render(
+				<Modal onRequestClose={ noop }>
+					<p>Modal content</p>
+				</Modal>
+			);
+
+			await user.keyboard( '[Escape]' );
+			expect( documentKeyDownHandler ).not.toHaveBeenCalled();
+		} finally {
+			document.removeEventListener( 'keydown', documentKeyDownHandler );
+		}
+	} );
+
 	it( 'should return focus when dismissed by clicking outside', async () => {
 		const user = userEvent.setup();
 		const ReturnDemo = () => {

package/src/navigator/stories/index.story.tsx

@@ -12,11 +12,6 @@ const meta: Meta< typeof Navigator > = {
 		Button: Navigator.Button,
 		BackButton: Navigator.BackButton,
 	},
-	// Temporary: Due to an upstream bug, render the root explicitly so the
-	// components manifest extractor can resolve props from the JSX.
-	//
-	// See: https://github.com/storybookjs/storybook/issues/34877
-	render: ( args ) => <Navigator { ...args } />,
 	title: 'Components/Navigation/Navigator',
 	id: 'components-navigator',
 	argTypes: {

package/src/number-control/stories/index.story.tsx

@@ -14,14 +14,14 @@ const meta: Meta< typeof NumberControl > = {
 		type: { control: { type: 'text' } },
 		value: { control: false },
 	},
-	tags: [ 'status-experimental', 'manifest' ],
+	tags: [ 'status-experimental' ],
 	parameters: {
 		controls: { expanded: true },
 		docs: { canvas: { sourceState: 'shown' } },
 		componentStatus: {
-			status: 'recommended',
+			status: 'not-recommended',
 			whereUsed: 'global',
-			notes: 'Will be superseded by [`InputControl`](?path=/docs/design-system-components-form-inputcontrol--docs) with `type="number"` in `@wordpress/ui`, but continue using for now.',
+			notes: 'For new use cases, use [`InputControl`](?path=/docs/design-system-components-form-inputcontrol--docs) with `type="number"` from `@wordpress/ui` instead. Existing usages should migrate with caution, due to significant differences in behavior. See the [migration guide](?path=/docs/components-inputcontrol--migration-guide).',
 		},
 	},
 };