From 3ffc446824453cc37898d805b3700839ee2bf1fe Mon Sep 17 00:00:00 2001 From: Gildas Garcia <1122076+djhi@users.noreply.github.com> Date: Fri, 26 Jun 2026 11:06:18 +0200 Subject: [PATCH] chore: delete unused bucket picker dialogs (#47331) ## Problem Cleaning up dead code. Those two dialogs are not used. Other components in their directories are though. ## Solution Remove them ## Summary by CodeRabbit * **Chores** * Removed two storage picker dialog components from the app. * This may affect how bucket and file selection screens are presented in the Studio interface. --- .../Storage/BucketFilePickerDialog/index.tsx | 81 ------------------- .../Storage/BucketsPickerDialog/index.tsx | 71 ---------------- 2 files changed, 152 deletions(-) delete mode 100644 apps/studio/components/interfaces/Storage/BucketFilePickerDialog/index.tsx delete mode 100644 apps/studio/components/interfaces/Storage/BucketsPickerDialog/index.tsx diff --git a/apps/studio/components/interfaces/Storage/BucketFilePickerDialog/index.tsx b/apps/studio/components/interfaces/Storage/BucketFilePickerDialog/index.tsx deleted file mode 100644 index f829967b1ac..00000000000 --- a/apps/studio/components/interfaces/Storage/BucketFilePickerDialog/index.tsx +++ /dev/null @@ -1,81 +0,0 @@ -import { useBreakpoint } from 'common' -import { - Dialog, - DialogContent, - DialogHeader, - DialogTitle, - Sheet, - SheetContent, - SheetHeader, - SheetTitle, -} from 'ui' - -import { BucketFilePickerExplorer } from './BucketFilePickerExplorer' -import { BucketFilePickerStateContextProvider } from './BucketFilePickerState' -import type { Bucket } from '@/data/storage/buckets-query' - -export type BucketsFilePickerDialogProps = { - open: boolean - onOpenChange: (open: boolean) => void - selectedBucket: Bucket - acceptedFileExtensions?: string[] - onSelect: (value: string) => void -} - -export function BucketsFilePickerDialog({ - open, - onOpenChange, - selectedBucket, - acceptedFileExtensions, - onSelect, -}: BucketsFilePickerDialogProps) { - const isMobileLayout = useBreakpoint('lg') - - const handleSelect = (value: string) => { - onSelect(value) - onOpenChange(false) - } - - return ( - <> - {isMobileLayout ? ( - - - - Choose a file - -
- - - -
-
-
- ) : ( - - - - Choose a file - -
- - - -
-
-
- )} - - ) -} diff --git a/apps/studio/components/interfaces/Storage/BucketsPickerDialog/index.tsx b/apps/studio/components/interfaces/Storage/BucketsPickerDialog/index.tsx deleted file mode 100644 index 5c5fcab384e..00000000000 --- a/apps/studio/components/interfaces/Storage/BucketsPickerDialog/index.tsx +++ /dev/null @@ -1,71 +0,0 @@ -import { useBreakpoint } from 'common' -import { - Button, - Dialog, - DialogContent, - DialogHeader, - DialogTitle, - Sheet, - SheetContent, - SheetHeader, - SheetTitle, -} from 'ui' - -import { BucketsPicker } from './BucketsPicker' -import type { AllowedBucketType } from './types' -import type { Bucket } from '@/data/storage/buckets-query' - -export type BucketsPickerDialogProps = { - open: boolean - onOpenChange: (open: boolean) => void - acceptedFileExtensions?: string[] - hideUnsupportedFiles?: boolean - onSelect: (bucket: Bucket) => void - allowedBucketType: AllowedBucketType -} - -export function BucketsPickerDialog({ - open, - onOpenChange, - onSelect, - allowedBucketType, -}: BucketsPickerDialogProps) { - const isMobileLayout = useBreakpoint('lg') - - return ( - <> - {isMobileLayout ? ( - - - - - Select a bucket - -
- -
-
-
- ) : ( - - - - - Select a bucket - -
- -
-
-
- )} - - ) -}