revert: focusOnMount prop should use firstContentElement

WordPress · Mar 4, 2025 · 78ad11a · 78ad11a
1 parent a77319c
commit 78ad11a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 13 deletions.
diff --git a/packages/dataviews/src/components/dataviews-item-actions/index.tsx b/packages/dataviews/src/components/dataviews-item-actions/index.tsx
@@ -110,7 +110,7 @@ export function ActionModal< Item >( {
 			title={ action.modalHeader || label }
 			__experimentalHideHeader={ !! action.hideModalHeader }
 			onRequestClose={ closeModal }
-			focusOnMount={ action.modalFocusOnMount }
+			focusOnMount="firstContentElement"
 			size={ action.modalSize || 'medium' }
 			overlayClassName={ `dataviews-action-modal dataviews-action-modal__${ kebabCase(
 				action.id

diff --git a/packages/dataviews/src/types.ts b/packages/dataviews/src/types.ts
@@ -3,11 +3,6 @@
  */
 import type { ReactElement, ComponentType } from 'react';
 
-/**
- * WordPress dependencies
- */
-import type { useFocusOnMount } from '@wordpress/compose';
-
 /**
  * Internal dependencies
  */
@@ -473,13 +468,6 @@ export interface ActionModal< Item > extends ActionBase< Item > {
 	 * The size of the modal.
 	 */
 	modalSize?: 'small' | 'medium' | 'large' | 'fill';
-
-	/**
-	 * The focus on mount of the modal.
-	 */
-	modalFocusOnMount?:
-		| Parameters< typeof useFocusOnMount >[ 0 ]
-		| 'firstContentElement';
 }
 
 export interface ActionButton< Item > extends ActionBase< Item > {