DataViews: Add grid keyboard navigation

packages/dataviews/src/components/dataviews-bulk-actions/index.tsx

@@ -65,7 +65,7 @@ export function useHasAPossibleBulkAction< Item >(
 	item: Item
 ) {
 	return useMemo( () => {
-		return actions.some( ( action ) => {
+		return actions?.some( ( action ) => {
 			return (
 				action.supportsBulk &&
 				( ! action.isEligible || action.isEligible( item ) )

packages/dataviews/src/dataviews-layouts/grid/index.tsx

@@ -13,6 +13,7 @@ import {
 	Spinner,
 	Flex,
 	FlexItem,
+	Composite,
 	privateApis as componentsPrivateApis,
 } from '@wordpress/components';
 import { __ } from '@wordpress/i18n';
@@ -36,7 +37,7 @@ import type {
 } from '../../types';
 import type { SetSelection } from '../../private-types';
 import getClickableItemProps from '../utils/get-clickable-item-props';
-import { useUpdatedPreviewSizeOnViewportChange } from './preview-size-picker';
+import { usePreviewSize } from './preview-size-picker';
 const { Badge } = unlock( componentsPrivateApis );
 
 interface GridItemProps< Item > {
@@ -56,6 +57,14 @@ interface GridItemProps< Item > {
 	hasBulkActions: boolean;
 }
 
+function chunk( array: any, size: any ) {
+	const chunks = [];
+	for ( let i = 0, j = array.length; i < j; i += size ) {
+		chunks.push( array.slice( i, i + size ) );
+	}
+	return chunks;
+}
+
 function GridItem< Item >( {
 	view,
 	selection,
@@ -120,6 +129,7 @@ function GridItem< Item >( {
 		<VStack
 			spacing={ 0 }
 			key={ id }
+			role="gridcell"
 			className={ clsx( 'dataviews-view-grid__card', {
 				'is-selected': hasBulkAction && isSelected,
 			} ) }
@@ -267,47 +277,80 @@ export default function ViewGrid< Item >( {
 		{ regularFields: [], badgeFields: [] }
 	);
 	const hasData = !! data?.length;
-	const updatedPreviewSize = useUpdatedPreviewSizeOnViewportChange();
+	const previewSize = usePreviewSize();
 	const hasBulkActions = useSomeItemHasAPossibleBulkAction( actions, data );
-	const usedPreviewSize = updatedPreviewSize || view.layout?.previewSize;
-	const gridStyle = usedPreviewSize
-		? {
-				gridTemplateColumns: `repeat(${ usedPreviewSize }, minmax(0, 1fr))`,
-		  }
-		: {};
 	return (
 		<>
 			{ hasData && (
-				<Grid
-					gap={ 8 }
-					columns={ 2 }
-					alignment="top"
+				<Composite
+					role="grid"
 					className="dataviews-view-grid"
-					style={ gridStyle }
+					focusLoop
+					focusWrap
 					aria-busy={ isLoading }
 				>
-					{ data.map( ( item ) => {
-						return (
-							<GridItem
-								key={ getItemId( item ) }
-								view={ view }
-								selection={ selection }
-								onChangeSelection={ onChangeSelection }
-								onClickItem={ onClickItem }
-								isItemClickable={ isItemClickable }
-								getItemId={ getItemId }
-								item={ item }
-								actions={ actions }
-								mediaField={ mediaField }
-								titleField={ titleField }
-								descriptionField={ descriptionField }
-								regularFields={ regularFields }
-								badgeFields={ badgeFields }
-								hasBulkActions={ hasBulkActions }
-							/>
-						);
-					} ) }
-				</Grid>
+					<VStack spacing={ 8 }>
+						{ chunk( data, previewSize ).map( ( row, i ) => (
+							<Composite.Row
+								key={ i }
+								role="row"
+								className="dataviews-view-grid__row"
+							>
+								<Grid
+									templateColumns={ `repeat( ${ previewSize }, minmax(0, 1fr) )` }
+									gap={ 8 }
+								>
+									{ row.map( ( item: any ) => (
+										<Composite.Item
+											key={ getItemId( item ) }
+											render={
+												<div
+													id={ getItemId( item ) }
+													className="dataviews-view-grid__row__gridcell"
+												>
+													<GridItem
+														view={ view }
+														selection={ selection }
+														onChangeSelection={
+															onChangeSelection
+														}
+														onClickItem={
+															onClickItem
+														}
+														isItemClickable={
+															isItemClickable
+														}
+														getItemId={ getItemId }
+														item={ item }
+														actions={ actions }
+														mediaField={
+															mediaField
+														}
+														titleField={
+															titleField
+														}
+														descriptionField={
+															descriptionField
+														}
+														regularFields={
+															regularFields
+														}
+														badgeFields={
+															badgeFields
+														}
+														hasBulkActions={
+															hasBulkActions
+														}
+													/>
+												</div>
+											}
+										/>
+									) ) }
+								</Grid>
+							</Composite.Row>
+						) ) }
+					</VStack>
+				</Composite>
 			) }
 			{ ! hasData && (
 				<div

packages/dataviews/src/dataviews-layouts/grid/preview-size-picker.tsx

@@ -43,23 +43,24 @@ function useViewPortBreakpoint() {
 	return 'mobile';
 }
 
-export function useUpdatedPreviewSizeOnViewportChange() {
+export function usePreviewSize() {
 	const view = useContext( DataViewsContext ).view as ViewGrid;
 	const viewport = useViewPortBreakpoint();
 	return useMemo( () => {
 		const previewSize = view.layout?.previewSize;
-		let newPreviewSize;
 		if ( ! previewSize ) {
-			return;
+			// For mobile, we want the `min` value as the default.
+			const breakValueProp = viewport === 'mobile' ? 'min' : 'default';
+			return viewportBreaks[ viewport ][ breakValueProp ];
 		}
 		const breakValues = viewportBreaks[ viewport ];
 		if ( previewSize < breakValues.min ) {
-			newPreviewSize = breakValues.min;
+			return breakValues.min;
 		}
 		if ( previewSize > breakValues.max ) {
-			newPreviewSize = breakValues.max;
+			return breakValues.max;
 		}
-		return newPreviewSize;
+		return previewSize;
 	}, [ viewport, view ] );
 }
 

packages/dataviews/src/dataviews-layouts/grid/style.scss

@@ -1,6 +1,5 @@
 .dataviews-view-grid {
 	margin-bottom: auto;
-	grid-template-rows: max-content;
 	padding: 0 $grid-unit-60 $grid-unit-30;
 	transition: padding ease-out 0.1s;
 	container-type: inline-size;
@@ -116,27 +115,10 @@
 }
 
 .dataviews-view-grid.dataviews-view-grid {
-	/**
-	 * Breakpoints were adjusted from media queries breakpoints to account for
-	 * the sidebar width. This was done to match the existing styles we had.
-	 */
-	@container (max-width: 480px) {
-		grid-template-columns: repeat(1, minmax(0, 1fr));
+	@container (max-width: 430px) {
 		padding-left: $grid-unit-30;
 		padding-right: $grid-unit-30;
 	}
-	@container (min-width: 480px) {
-		grid-template-columns: repeat(2, minmax(0, 1fr));
-	}
-	@container (min-width: 780px) {
-		grid-template-columns: repeat(3, minmax(0, 1fr));
-	}
-	@container (min-width: 1140px) {
-		grid-template-columns: repeat(4, minmax(0, 1fr));
-	}
-	@container (min-width: 1520px) {
-		grid-template-columns: repeat(5, minmax(0, 1fr));
-	}
 }
 
 .dataviews-view-grid__field-value:empty,
@@ -160,3 +142,23 @@
 .dataviews-view-grid__media--clickable {
 	cursor: pointer;
 }
+
+
+.dataviews-view-grid__row {
+	.dataviews-view-grid__row__gridcell {
+		outline: 0;
+		outline-style: solid;
+		// outline-offset: calc(-1 * var(--wp-admin-border-width-focus));
+		border-radius: $grid-unit-10;
+
+		&[data-focus-visible] {
+			outline-color: var(--wp-admin-theme-color);
+			outline-width: var(--wp-admin-border-width-focus);
+		}
+
+		&:hover {
+			outline-color: rgba($black, 0.3);
+			outline-width: var(--wp-admin-border-width-focus);
+		}
+	}
+}