Files
Danny White 476d4a5851 refactor(ui): drop redundant Button variant="default" props (#50161)
## What kind of change does this PR introduce?

Mechanical cleanup on top of the Button default-variant change (#50160).

## What is the current behavior?

Many callsites still pass `variant="default"` even though that is now
the component default.

## What is the new behavior?

Removes redundant static `variant="default"` from legacy `Button` and
`ButtonTooltip` callsites. Keeps explicit defaults where they document
the API:

- `button-default.tsx` and `button-sizes.tsx` demos
- `DocsButton`, which pins neutral styling at the wrapper boundary

## To test

Studio:

- [Auth → Rate
Limits](https://studio-staging-2s957kwc4-supabase.vercel.app/dashboard/project/_/auth/rate-limits):
dirty the form so Cancel appears; Cancel stays neutral, Save stays green
- [Project Settings → API
Keys](https://studio-staging-2s957kwc4-supabase.vercel.app/dashboard/project/_/settings/api-keys):
`DocsButton` in the header actions stays neutral

Design system:

- [Design system →
Button](https://design-system-git-dnywh-dc924ac1-supabase.vercel.app/design-system/docs/components/button):
`button-default` / `button-sizes` still show explicit default styling;
Primary (green) is restricted to the Primary section (and `asChild`)

WWW:

- [www → Brand
assets](https://zone-www-dot-com-git-dnywh-dc924ac1-supabase.vercel.app/brand-assets):
Download logo kit / Download button kit stay neutral
2026-09-11 17:05:26 +10:00

274 lines
9.5 KiB
TypeScript

import { zodResolver } from '@hookform/resolvers/zod'
import { useMemo } from 'react'
import { useForm, useWatch } from 'react-hook-form'
import { toast } from 'sonner'
import {
Badge,
Button,
Dialog,
DialogContent,
DialogFooter,
DialogHeader,
DialogSection,
DialogSectionSeparator,
DialogTitle,
Form,
FormControl,
FormField,
Input,
Select,
SelectContent,
SelectItem,
SelectSeparator,
SelectTrigger,
SelectValue,
} from 'ui'
import { Admonition } from 'ui-patterns/Admonition'
import { FormItemLayout } from 'ui-patterns/form/FormItemLayout/FormItemLayout'
import { ShimmeringLoader } from 'ui-patterns/ShimmeringLoader'
import * as z from 'zod'
import { extensionsWithRecommendedSchemas } from './Extensions.constants'
import { DocsButton } from '@/components/ui/DocsButton'
import { useDatabaseExtensionEnableMutation } from '@/data/database-extensions/database-extension-enable-mutation'
import { type DatabaseExtension } from '@/data/database-extensions/database-extensions-query'
import { useSchemasQuery } from '@/data/database/schemas-query'
import { useSchemasFilteredForHighAvailability } from '@/hooks/misc/useHighAvailability'
import { useIsOrioleDb, useSelectedProjectQuery } from '@/hooks/misc/useSelectedProject'
import { useProtectedSchemas } from '@/hooks/useProtectedSchemas'
import { DOCS_URL } from '@/lib/constants'
const orioleExtCallOuts = ['vector', 'postgis']
const FormSchema = z.object({ name: z.string(), schema: z.string() }).superRefine((val, ctx) => {
if (val.schema === 'custom' && val.name.length === 0) {
ctx.addIssue({
code: z.ZodIssueCode.custom,
path: ['name'],
message: 'Please provide a name for the schema',
})
}
})
interface EnableExtensionModalProps {
visible: boolean
extension: DatabaseExtension
onCancel: () => void
}
export const EnableExtensionModal = ({
visible,
extension,
onCancel,
}: EnableExtensionModalProps) => {
const isOrioleDb = useIsOrioleDb()
const { data: project } = useSelectedProjectQuery()
const { data: protectedSchemas } = useProtectedSchemas({ excludeSchemas: ['extensions'] })
const recommendedSchema = extensionsWithRecommendedSchemas[extension.name]
const { data: schemas = [], isPending: isLoading } = useSchemasQuery(
{
projectRef: project?.ref,
connectionString: project?.connectionString,
},
{ enabled: visible }
)
const visibleSchemas = useSchemasFilteredForHighAvailability(schemas)
const availableSchemas = useMemo(
() =>
visibleSchemas.filter(
(schema) =>
schema.name === recommendedSchema ||
!protectedSchemas.some((protectedSchema) => protectedSchema.name === schema.name)
),
[visibleSchemas, recommendedSchema, protectedSchemas]
)
// [Joshen] Hard-coding pg_cron here as this is enforced on our end (Not via pg_available_extension_versions)
const defaultSchema =
extension.name === 'pg_cron' ? 'pg_catalog' : extension.default_version_schema
const { mutate: enableExtension, isPending: isEnabling } = useDatabaseExtensionEnableMutation({
onSuccess: () => {
toast.success(`Extension "${extension.name}" is now enabled`)
onCancel()
},
onError: (error) => {
toast.error(`Failed to enable ${extension.name}: ${error.message}`)
},
})
const defaultValues = { name: extension.name, schema: recommendedSchema ?? 'extensions' }
const form = useForm<z.infer<typeof FormSchema>>({
mode: 'onBlur',
reValidateMode: 'onBlur',
resolver: zodResolver(FormSchema),
defaultValues,
})
const schema = useWatch({ control: form.control, name: 'schema' })
const onSubmit = async (values: z.infer<typeof FormSchema>) => {
if (project === undefined) return console.error('Project is required')
const schema =
defaultSchema !== undefined && defaultSchema !== null
? defaultSchema
: values.schema === 'custom'
? values.name
: values.schema
enableExtension({
projectRef: project.ref,
connectionString: project?.connectionString,
schema,
name: extension.name,
version: extension.default_version,
cascade: true,
createSchema: !schema.startsWith('pg_'),
})
}
return (
<Dialog
open={visible}
onOpenChange={(open: boolean) => {
if (!open) onCancel()
}}
>
<DialogContent size="small" aria-describedby={undefined}>
<DialogHeader>
<DialogTitle>Enable {extension.name}</DialogTitle>
</DialogHeader>
<DialogSectionSeparator />
{isOrioleDb && orioleExtCallOuts.includes(extension.name) && (
<Admonition
type="default"
title="Extension is limited by OrioleDB"
className="border-x-0 border-t-0 rounded-none"
>
<span className="block">
{extension.name} cannot be accelerated by indexes on tables that are using the
OrioleDB access method
</span>
<DocsButton abbrev={false} className="mt-2" href={`${DOCS_URL}`} />
</Admonition>
)}
<DialogSection>
<Form {...form}>
<form id="enable-extensions-form" onSubmit={form.handleSubmit(onSubmit)}>
{isLoading ? (
<div className="space-y-2">
<ShimmeringLoader />
<div className="w-3/4">
<ShimmeringLoader />
</div>
</div>
) : !!defaultSchema ? (
<div className="flex flex-col gap-y-2">
<FormItemLayout
isReactForm={false}
label="Select a schema to enable the extension for"
>
<Input disabled value={defaultSchema} />
</FormItemLayout>
<p className="text-sm text-foreground-light">
Extension must be installed in the "{defaultSchema}" schema.
</p>
</div>
) : (
<div className="flex flex-col gap-y-2">
<FormField
key="schema"
name="schema"
control={form.control}
render={({ field }) => (
<FormItemLayout
name="schema"
label="Select a schema to enable the extension for"
>
<FormControl>
<Select
value={field.value}
onValueChange={field.onChange}
disabled={!!defaultSchema}
>
<SelectTrigger>
<SelectValue placeholder="Select a schema" />
</SelectTrigger>
<SelectContent>
<SelectItem value="custom">
Create a new schema{' '}
<code className="text-code-inline">{extension.name}</code>
</SelectItem>
<SelectSeparator />
{availableSchemas.map((schema) => {
return (
<SelectItem key={schema.id} value={schema.name}>
{schema.name}
{schema.name === recommendedSchema ? (
<Badge className="ml-2" variant="success">
Recommended
</Badge>
) : !defaultSchema && schema.name === 'extensions' ? (
<Badge className="ml-2">Default</Badge>
) : null}
</SelectItem>
)
})}
</SelectContent>
</Select>
</FormControl>
</FormItemLayout>
)}
/>
{!!recommendedSchema && (
<p className="text-sm text-foreground-light">
Use the "{recommendedSchema}" schema for full compatibility with related
features.
</p>
)}
{schema === 'custom' && (
<FormField
key="name"
name="name"
control={form.control}
render={({ field }) => (
<FormItemLayout label="Schema name">
<FormControl>
<Input {...field} />
</FormControl>
</FormItemLayout>
)}
/>
)}
</div>
)}
</form>
</Form>
</DialogSection>
<DialogFooter>
<Button disabled={isEnabling} onClick={() => onCancel()}>
Cancel
</Button>
<Button
variant="primary"
type="submit"
form="enable-extensions-form"
loading={isEnabling}
disabled={isLoading || isEnabling}
>
Enable extension
</Button>
</DialogFooter>
</DialogContent>
</Dialog>
)
}