@wordpress/components 19.12.0 → 19.15.0

package/src/ui/form-group/form-group-content.js

@@ -26,10 +26,10 @@ function FormGroupContent( {
 	truncate,
 	...props
 } ) {
-	const contextProps = useMemo( () => ( { id, horizontal } ), [
-		id,
-		horizontal,
-	] );
+	const contextProps = useMemo(
+		() => ( { id, horizontal } ),
+		[ id, horizontal ]
+	);
 
 	const content = help ? (
 		<VStack expanded={ false } spacing={ spacing }>

package/src/ui/form-group/types.ts

@@ -7,7 +7,7 @@ import type { CSSProperties, ReactNode, ReactText } from 'react';
  * Internal dependencies
  */
 import type { Props as ControlLabelProps } from '../control-label/types';
-import type { Props as GridProps } from '../../grid/types';
+import type { GridProps } from '../../grid/types';
 
 export type FormGroupLabelProps = ControlLabelProps & {
 	labelHidden?: boolean;

package/src/ui/utils/get-valid-children.ts

@@ -1,7 +1,7 @@
 /**
  * External dependencies
  */
-import type { ReactNode, ReactNodeArray } from 'react';
+import type { ReactNode, ReactChild, ReactFragment, ReactPortal } from 'react';
 
 /**
  * WordPress dependencies
@@ -15,7 +15,9 @@ import { Children, isValidElement } from '@wordpress/element';
  *
  * @return An array of available children.
  */
-export function getValidChildren( children: ReactNode ): ReactNodeArray {
+export function getValidChildren(
+	children: ReactNode
+): Array< ReactChild | ReactFragment | ReactPortal > {
 	if ( typeof children === 'string' ) return [ children ];
 
 	return Children.toArray( children ).filter( ( child ) =>

package/src/unit-control/index.tsx

@@ -160,15 +160,13 @@ function UnforwardedUnitControl(
 			refParsedQuantity.current = undefined;
 			return;
 		}
-		const [
-			validParsedQuantity,
-			validParsedUnit,
-		] = getValidParsedQuantityAndUnit(
-			event.currentTarget.value,
-			units,
-			parsedQuantity,
-			unit
-		);
+		const [ validParsedQuantity, validParsedUnit ] =
+			getValidParsedQuantityAndUnit(
+				event.currentTarget.value,
+				units,
+				parsedQuantity,
+				unit
+			);
 
 		refParsedQuantity.current = validParsedQuantity;
 

package/src/unit-control/stories/index.tsx

@@ -67,9 +67,8 @@ const DefaultTemplate: ComponentStory< typeof UnitControl > = ( {
 	);
 };
 
-export const Default: ComponentStory<
-	typeof UnitControl
-> = DefaultTemplate.bind( {} );
+export const Default: ComponentStory< typeof UnitControl > =
+	DefaultTemplate.bind( {} );
 Default.args = {
 	label: 'Label',
 };
@@ -79,9 +78,8 @@ Default.args = {
  * will not fire while a new value is typed in the input field (you can verify this
  * behavior by inspecting the console's output).
  */
-export const PressEnterToChange: ComponentStory<
-	typeof UnitControl
-> = DefaultTemplate.bind( {} );
+export const PressEnterToChange: ComponentStory< typeof UnitControl > =
+	DefaultTemplate.bind( {} );
 PressEnterToChange.args = {
 	...Default.args,
 	isPressEnterToChange: true,
@@ -95,9 +93,8 @@ PressEnterToChange.args = {
  * Most of `NumberControl`'s props can be passed to `UnitControl`, and they will
  * affect its numeric input field.
  */
-export const TweakingTheNumberInput: ComponentStory<
-	typeof UnitControl
-> = DefaultTemplate.bind( {} );
+export const TweakingTheNumberInput: ComponentStory< typeof UnitControl > =
+	DefaultTemplate.bind( {} );
 TweakingTheNumberInput.args = {
 	...Default.args,
 	min: 0,
@@ -109,9 +106,8 @@ TweakingTheNumberInput.args = {
 /**
  * When only one unit is available, the unit selection dropdown becomes static text.
  */
-export const WithSingleUnit: ComponentStory<
-	typeof UnitControl
-> = DefaultTemplate.bind( {} );
+export const WithSingleUnit: ComponentStory< typeof UnitControl > =
+	DefaultTemplate.bind( {} );
 WithSingleUnit.args = {
 	...Default.args,
 	units: CSS_UNITS.slice( 0, 1 ),

package/src/unit-control/test/index.tsx

@@ -39,9 +39,8 @@ const ControlledSyncUnits = () => {
 		fieldName: 'valueA' | 'valueB',
 		newValue?: string | number
 	) => {
-		const parsedQuantityAndUnit = parseQuantityAndUnitFromRawValue(
-			newValue
-		);
+		const parsedQuantityAndUnit =
+			parseQuantityAndUnitFromRawValue( newValue );
 		const quantity = parsedQuantityAndUnit[ 0 ];
 
 		if ( ! Number.isFinite( quantity ) ) {
@@ -56,10 +55,8 @@ const ControlledSyncUnits = () => {
 		};
 
 		Object.entries( state ).forEach( ( [ stateProp, stateValue ] ) => {
-			const [
-				stateQuantity,
-				stateUnit,
-			] = parseQuantityAndUnitFromRawValue( stateValue );
+			const [ stateQuantity, stateUnit ] =
+				parseQuantityAndUnitFromRawValue( stateValue );
 
 			if ( stateProp !== fieldName && stateUnit !== newUnit ) {
 				nextState[

package/src/utils/hooks/stories/use-cx.js

@@ -33,19 +33,20 @@ const ExampleWithUseMemoWrong = ( { serializedStyles, children } ) => {
 	const cx = useCx();
 	// Wrong: using 'useMemo' without adding 'cx' to the dependency list.
 	// eslint-disable-next-line react-hooks/exhaustive-deps
-	const classes = useMemo( () => cx( serializedStyles ), [
-		serializedStyles,
-	] );
+	const classes = useMemo(
+		() => cx( serializedStyles ),
+		[ serializedStyles ]
+	);
 	return <span className={ classes }>{ children }</span>;
 };
 
 const ExampleWithUseMemoRight = ( { serializedStyles, children } ) => {
 	const cx = useCx();
 	// Right: using 'useMemo' with 'cx' listed as a dependency.
-	const classes = useMemo( () => cx( serializedStyles ), [
-		cx,
-		serializedStyles,
-	] );
+	const classes = useMemo(
+		() => cx( serializedStyles ),
+		[ cx, serializedStyles ]
+	);
 	return <span className={ classes }>{ children }</span>;
 };
 

package/src/utils/hooks/test/use-controlled-state.js

@@ -2,13 +2,14 @@
  * External dependencies
  */
 import { render, fireEvent, screen } from '@testing-library/react';
-import { noop } from 'lodash';
 
 /**
  * Internal dependencies
  */
 import useControlledState from '../use-controlled-state';
 
+const noop = () => {};
+
 describe( 'hooks', () => {
 	const getInput = () => screen.getByTestId( 'input' );
 

package/src/utils/math.js

@@ -1,8 +1,3 @@
-/**
- * External dependencies
- */
-import { clamp } from 'lodash';
-
 /**
  * Parses and retrieves a number value.
  *
@@ -61,6 +56,20 @@ function getPrecision( value ) {
 	return split[ 1 ] !== undefined ? split[ 1 ].length : 0;
 }
 
+/**
+ * Clamps a value based on a min/max range.
+ *
+ * @param {number} value The value.
+ * @param {number} min   The minimum range.
+ * @param {number} max   The maximum range.
+ *
+ * @return {number} The clamped value.
+ */
+export function clamp( value, min, max ) {
+	const baseValue = getNumber( value );
+	return Math.max( min, Math.min( baseValue, max ) );
+}
+
 /**
  * Clamps a value based on a min/max range with rounding
  *

package/src/utils/test/math.js

@@ -1,7 +1,7 @@
 /**
  * Internal dependencies
  */
-import { add, subtract, roundClamp } from '../math';
+import { add, clamp, subtract, roundClamp } from '../math';
 
 describe( 'add', () => {
 	it( 'should add string and number values', () => {
@@ -37,6 +37,27 @@ describe( 'subtract', () => {
 	} );
 } );
 
+describe( 'clamp', () => {
+	it( 'should clamp a value between min and max', () => {
+		expect( clamp( 10, 1, 10 ) ).toBe( 10 );
+		expect( clamp( 1, 1, 10 ) ).toBe( 1 );
+		expect( clamp( 0, 1, 10 ) ).toBe( 1 );
+
+		expect( clamp( 50, 1, 10 ) ).toBe( 10 );
+		expect( clamp( 50, -10, 10 ) ).toBe( 10 );
+		expect( clamp( -50, -10, 10 ) ).toBe( -10 );
+
+		expect( clamp( Infinity, -10, 10 ) ).toBe( 10 );
+		expect( clamp( -Infinity, -10, 10 ) ).toBe( -10 );
+	} );
+
+	it( 'should clamp number or string values', () => {
+		expect( clamp( '50', 1, 10 ) ).toBe( 10 );
+		expect( clamp( '50', -10, 10 ) ).toBe( 10 );
+		expect( clamp( -50, -10, '10' ) ).toBe( -10 );
+	} );
+} );
+
 describe( 'roundClamp', () => {
 	it( 'should clamp a value between min and max', () => {
 		expect( roundClamp( 10, 1, 10 ) ).toBe( 10 );

package/src/utils/unit-values.ts

@@ -1,4 +1,5 @@
-const UNITED_VALUE_REGEX = /^([\d.\-+]*)\s*(fr|cm|mm|Q|in|pc|pt|px|em|ex|ch|rem|lh|vw|vh|vmin|vmax|%|cap|ic|rlh|vi|vb|deg|grad|rad|turn|s|ms|Hz|kHz|dpi|dpcm|dppx)?$/;
+const UNITED_VALUE_REGEX =
+	/^([\d.\-+]*)\s*(fr|cm|mm|Q|in|pc|pt|px|em|ex|ch|rem|lh|vw|vh|vmin|vmax|%|cap|ic|rlh|vi|vb|deg|grad|rad|turn|s|ms|Hz|kHz|dpi|dpcm|dppx)?$/;
 
 /**
  * Parses a number and unit from a value.

package/src/utils/values.js

@@ -88,9 +88,8 @@ export function isValueNumeric( value, locale = window.navigator.language ) {
 		}
 	}
 
-	const [ delimiterRegexp, decimalRegexp ] = getDelimiterAndDecimalRegex(
-		locale
-	);
+	const [ delimiterRegexp, decimalRegexp ] =
+		getDelimiterAndDecimalRegex( locale );
 	const valueToCheck =
 		typeof value === 'string'
 			? value

package/src/v-stack/README.md

@@ -29,9 +29,7 @@ function Example() {
 
 ## Props
 
-##### alignment
-
-**Type**: `HStackAlignment`,`CSS['alignItems']`
+##### `alignment`: `HStackAlignment | CSSProperties['alignItems']`
 
 Determines how the child elements are aligned.
 
@@ -47,35 +45,25 @@ Determines how the child elements are aligned.
 -   `edge`: Aligns content to the edges of the container.
 -   `stretch`: Stretches content to the edges of the container.
 
-##### direction
-
-**Type**: `FlexDirection`
+##### `direction`: `FlexDirection`
 
 The direction flow of the children content can be adjusted with `direction`. `column` will align children vertically and `row` will align children horizontally.
 
-##### expanded
-
-**Type**: `boolean`
+##### `expanded`: `boolean`
 
 Expands to the maximum available width (if horizontal) or height (if vertical).
 
-##### justify
-
-**Type**: `CSS['justifyContent']`
+##### `justify`: `CSSProperties['justifyContent']`
 
 Horizontally aligns content if the `direction` is `row`, or vertically aligns content if the `direction` is `column`.
 In the example below, `flex-start` will align the children content to the left.
 
-##### spacing
-
-**Type**: `CSS['width']`
+##### `spacing`: `CSSProperties['width']`
 
 The amount of space between each child element. Spacing in between each child can be adjusted by using `spacing`.
 The value of `spacing` works as a multiplier to the library's grid system (base of `4px`).
 
-##### wrap
-
-**Type**: `boolean`
+##### `wrap`: `boolean`
 
 Determines if children should wrap.
 

package/src/v-stack/{component.js → component.tsx}

@@ -1,26 +1,31 @@
+/**
+ * External dependencies
+ */
+import type { ForwardedRef } from 'react';
+
 /**
  * Internal dependencies
  */
-import { contextConnect } from '../ui/context';
+import { contextConnect, WordPressComponentProps } from '../ui/context';
 import { View } from '../view';
 import { useVStack } from './hook';
+import type { VStackProps } from './types';
 
-/**
- * @param {import('../ui/context').WordPressComponentProps<import('./types').Props, 'div'>} props
- * @param {import('react').ForwardedRef<any>}                                               forwardedRef
- */
-function VStack( props, forwardedRef ) {
+function UnconnectedVStack(
+	props: WordPressComponentProps< VStackProps, 'div' >,
+	forwardedRef: ForwardedRef< any >
+) {
 	const vStackProps = useVStack( props );
 
 	return <View { ...vStackProps } ref={ forwardedRef } />;
 }
 
 /**
- * `VStack` (or Vertical Stack) is a layout component that arranges child elements in a vertical line.
+ * `VStack` (or Vertical Stack) is a layout component that arranges child
+ * elements in a vertical line.
  *
  * `VStack` can render anything inside.
  *
- * @example
  * ```jsx
  * import {
  * 	__experimentalText as Text,
@@ -38,6 +43,6 @@ function VStack( props, forwardedRef ) {
  * }
  * ```
  */
-const ConnectedVStack = contextConnect( VStack, 'VStack' );
+export const VStack = contextConnect( UnconnectedVStack, 'VStack' );
 
-export default ConnectedVStack;
+export default VStack;

package/src/v-stack/{hook.js → hook.ts}

@@ -1,14 +1,13 @@
 /**
  * Internal dependencies
  */
-import { useContextSystem } from '../ui/context';
+import { useContextSystem, WordPressComponentProps } from '../ui/context';
 import { useHStack } from '../h-stack';
+import type { VStackProps } from './types';
 
-/**
- *
- * @param {import('../ui/context').WordPressComponentProps<import('./types').Props, 'div'>} props
- */
-export function useVStack( props ) {
+export function useVStack(
+	props: WordPressComponentProps< VStackProps, 'div' >
+) {
 	const { expanded = false, ...otherProps } = useContextSystem(
 		props,
 		'VStack'

package/src/v-stack/stories/index.tsx

@@ -0,0 +1,41 @@
+/**
+ * External dependencies
+ */
+import type { ComponentMeta, ComponentStory } from '@storybook/react';
+
+/**
+ * Internal dependencies
+ */
+import { View } from '../../view';
+import { VStack } from '..';
+
+const meta: ComponentMeta< typeof VStack > = {
+	component: VStack,
+	title: 'Components (Experimental)/VStack',
+	argTypes: {
+		alignment: { control: { type: 'text' } },
+		as: { control: { type: 'text' } },
+		direction: { control: { type: 'text' } },
+		justify: { control: { type: 'text' } },
+		spacing: { control: { type: 'text' } },
+	},
+	parameters: {
+		controls: { expanded: true },
+		docs: { source: { state: 'open' } },
+	},
+};
+export default meta;
+
+const Template: ComponentStory< typeof VStack > = ( props ) => {
+	return (
+		<VStack { ...props }>
+			<View>One</View>
+			<View>Two</View>
+			<View>Three</View>
+			<View>Four</View>
+			<View>Five</View>
+		</VStack>
+	);
+};
+
+export const Default = Template.bind( {} );

package/src/v-stack/types.ts

@@ -8,7 +8,27 @@ import type { CSSProperties } from 'react';
  */
 import type { HStackAlignment, Props as HStackProps } from '../h-stack/types';
 
-export type Props = HStackProps & {
+export type VStackProps = HStackProps & {
+	/**
+	 * Determines how the child elements are aligned.
+	 *
+	 * -   `top`: Aligns content to the top.
+	 * -   `topLeft`: Aligns content to the top/left.
+	 * -   `topRight`: Aligns content to the top/right.
+	 * -   `left`: Aligns content to the left.
+	 * -   `center`: Aligns content to the center.
+	 * -   `right`: Aligns content to the right.
+	 * -   `bottom`: Aligns content to the bottom.
+	 * -   `bottomLeft`: Aligns content to the bottom/left.
+	 * -   `bottomRight`: Aligns content to the bottom/right.
+	 * -   `edge`: Aligns content to the edges of the container.
+	 * -   `stretch`: Stretches content to the edges of the container.
+	 */
 	alignment?: HStackAlignment | CSSProperties[ 'alignItems' ];
+	/**
+	 * The amount of space between each child element. Spacing in between each
+	 * child can be adjusted by using `spacing`. The value of `spacing` works as
+	 * a multiplier to the library's grid system (base of `4px`).
+	 */
 	spacing?: CSSProperties[ 'width' ];
 };

package/src/visually-hidden/{component.js → component.tsx}

@@ -1,15 +1,24 @@
+/**
+ * External dependencies
+ */
+import type { ForwardedRef } from 'react';
+
 /**
  * Internal dependencies
  */
-import { useContextSystem, contextConnect } from '../ui/context';
+import {
+	useContextSystem,
+	contextConnect,
+	WordPressComponentProps,
+} from '../ui/context';
 import { visuallyHidden } from './styles';
 import { View } from '../view';
+import type { VisuallyHiddenProps } from './types';
 
-/**
- * @param {import('../ui/context').WordPressComponentProps<{ children: import('react').ReactNode }, 'div'>} props
- * @param {import('react').ForwardedRef<any>}                                                               forwardedRef
- */
-function VisuallyHidden( props, forwardedRef ) {
+function UnconnectedVisuallyHidden(
+	props: WordPressComponentProps< VisuallyHiddenProps, 'div' >,
+	forwardedRef: ForwardedRef< any >
+) {
 	const { style: styleProp, ...contextProps } = useContextSystem(
 		props,
 		'VisuallyHidden'
@@ -27,22 +36,21 @@ function VisuallyHidden( props, forwardedRef ) {
  * `VisuallyHidden` is a component used to render text intended to be visually
  * hidden, but will show for alternate devices, for example a screen reader.
  *
- * @example
  * ```jsx
  * import { VisuallyHidden } from `@wordpress/components`;
  *
  * function Example() {
- * 	return (
- * 		<VisuallyHidden>
- * 			<label>Code is Poetry</label>
- * 		</VisuallyHidden>
- * 	);
+ *   return (
+ *     <VisuallyHidden>
+ *       <label>Code is Poetry</label>
+ *     </VisuallyHidden>
+ *   );
  * }
  * ```
  */
-const ConnectedVisuallyHidden = contextConnect(
-	VisuallyHidden,
+export const VisuallyHidden = contextConnect(
+	UnconnectedVisuallyHidden,
 	'VisuallyHidden'
 );
 
-export default ConnectedVisuallyHidden;
+export default VisuallyHidden;

package/src/visually-hidden/stories/index.tsx

@@ -0,0 +1,66 @@
+/**
+ * External dependencies
+ */
+import type { ComponentMeta, ComponentStory } from '@storybook/react';
+
+/**
+ * Internal dependencies
+ */
+import { VisuallyHidden } from '..';
+
+const meta: ComponentMeta< typeof VisuallyHidden > = {
+	component: VisuallyHidden,
+	title: 'Components/VisuallyHidden',
+	argTypes: {
+		children: { control: { type: null } },
+		as: { control: { type: 'text' } },
+	},
+	parameters: {
+		controls: {
+			expanded: true,
+		},
+		docs: { source: { state: 'open' } },
+	},
+};
+export default meta;
+
+export const Default: ComponentStory< typeof VisuallyHidden > = ( args ) => (
+	<>
+		<VisuallyHidden as="span" { ...args }>
+			This should not show.
+		</VisuallyHidden>
+		<div>
+			This text will{ ' ' }
+			<VisuallyHidden as="span" { ...args }>
+				but not inline{ ' ' }
+			</VisuallyHidden>{ ' ' }
+			always show.
+		</div>
+	</>
+);
+
+export const WithForwardedProps: ComponentStory< typeof VisuallyHidden > = (
+	args
+) => (
+	<>
+		Additional props can be passed to VisuallyHidden and are forwarded to
+		the rendered element.{ ' ' }
+		<VisuallyHidden as="span" data-id="test" { ...args }>
+			Check out my data attribute!{ ' ' }
+		</VisuallyHidden>
+		Inspect the HTML to see!
+	</>
+);
+
+export const WithAdditionalClassNames: ComponentStory<
+	typeof VisuallyHidden
+> = ( args ) => (
+	<>
+		Additional class names passed to VisuallyHidden extend the component
+		class name.{ ' ' }
+		<VisuallyHidden as="label" className="test-input" { ...args }>
+			Check out my class!{ ' ' }
+		</VisuallyHidden>
+		Inspect the HTML to see!
+	</>
+);

package/src/visually-hidden/{styles.js → styles.ts}

@@ -1,5 +1,9 @@
-/** @type {import('react').CSSProperties} */
-export const visuallyHidden = {
+/**
+ * External dependencies
+ */
+import type { CSSProperties } from 'react';
+
+export const visuallyHidden: CSSProperties = {
 	border: 0,
 	clip: 'rect(1px, 1px, 1px, 1px)',
 	WebkitClipPath: 'inset( 50% )',

package/src/visually-hidden/test/index.tsx

@@ -0,0 +1,17 @@
+/**
+ * External dependencies
+ */
+import { render, screen } from '@testing-library/react';
+
+/**
+ * Internal dependencies
+ */
+import { VisuallyHidden } from '..';
+
+describe( 'VisuallyHidden', () => {
+	it( 'should render correctly', () => {
+		const text = 'This is hidden';
+		render( <VisuallyHidden>{ text }</VisuallyHidden> );
+		expect( screen.getByText( text ) ).toMatchSnapshot();
+	} );
+} );

package/src/visually-hidden/types.ts

@@ -0,0 +1,11 @@
+/**
+ * External dependencies
+ */
+import type { ReactNode } from 'react';
+
+export type VisuallyHiddenProps = {
+	/**
+	 * The children elements.
+	 */
+	children: ReactNode;
+};