Skip to content

Commit bd4401f

Browse files
Media Modal Experiment: Remove default value for allowedTypes so that the file block can accept all types (WordPress#75159)
Co-authored-by: andrewserong <andrewserong@git.wordpress.org> Co-authored-by: talldan <talldanwp@git.wordpress.org>
1 parent 81017b3 commit bd4401f

File tree

1 file changed

+4
-5
lines changed
  • packages/media-utils/src/components/media-upload-modal

1 file changed

+4
-5
lines changed

packages/media-utils/src/components/media-upload-modal/index.tsx

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,6 @@ const NOTICE_ID_UPLOAD_PROGRESS = 'media-modal-upload-progress';
5858
interface MediaUploadModalProps {
5959
/**
6060
* Array of allowed media types.
61-
* @default ['image']
6261
*/
6362
allowedTypes?: string[];
6463

@@ -153,7 +152,7 @@ interface MediaUploadModalProps {
153152
* @return JSX element or null
154153
*/
155154
export function MediaUploadModal( {
156-
allowedTypes = [ 'image' ],
155+
allowedTypes,
157156
multiple = false,
158157
value,
159158
onSelect,
@@ -236,7 +235,7 @@ export function MediaUploadModal( {
236235

237236
// Base media type on allowedTypes if no filter is set
238237
if ( ! filters.media_type ) {
239-
filters.media_type = allowedTypes.includes( '*' )
238+
filters.media_type = allowedTypes?.includes( '*' )
240239
? undefined
241240
: allowedTypes;
242241
}
@@ -467,10 +466,10 @@ export function MediaUploadModal( {
467466

468467
// Build accept attribute from allowedTypes
469468
const acceptTypes = useMemo( () => {
470-
if ( allowedTypes.includes( '*' ) ) {
469+
if ( allowedTypes?.includes( '*' ) ) {
471470
return undefined;
472471
}
473-
return allowedTypes.join( ',' );
472+
return allowedTypes?.join( ',' );
474473
}, [ allowedTypes ] );
475474

476475
if ( ! isOpen ) {

0 commit comments

Comments
 (0)