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

115 lines
3.1 KiB
TypeScript

import { zodResolver } from '@hookform/resolvers/zod'
import { useEffect } from 'react'
import { SubmitHandler, useForm } from 'react-hook-form'
import {
Button,
Dialog,
DialogContent,
DialogFooter,
DialogHeader,
DialogSection,
DialogSectionSeparator,
DialogTitle,
Form,
FormControl,
FormField,
Input,
Textarea,
} from 'ui'
import { FormItemLayout } from 'ui-patterns/form/FormItemLayout/FormItemLayout'
import * as z from 'zod'
const formSchema = z.object({
name: z.string().min(1, 'Required'),
description: z.string().optional(),
})
type SavedQuery = z.infer<typeof formSchema>
export interface UpdateSavedQueryProps {
header: string
visible: boolean
onCancel: () => void
onSubmit: SubmitHandler<SavedQuery>
initialValues: SavedQuery
}
export const UpdateSavedQueryModal = ({
header,
visible,
onCancel,
onSubmit,
initialValues,
}: UpdateSavedQueryProps) => {
const form = useForm<SavedQuery>({
resolver: zodResolver(formSchema),
defaultValues: { ...initialValues, description: initialValues.description ?? '' },
})
const { reset, formState } = form
const { isDirty, isSubmitting } = formState
useEffect(() => {
if (isDirty) return
reset({ ...initialValues, description: initialValues.description ?? '' })
}, [isDirty, initialValues, reset])
const handleCancel = () => {
form.reset()
onCancel()
}
return (
<Dialog open={visible} onOpenChange={handleCancel}>
<DialogContent size="medium">
<DialogHeader>
<DialogTitle>{header}</DialogTitle>
</DialogHeader>
<DialogSectionSeparator />
<Form {...form}>
<form onSubmit={form.handleSubmit(onSubmit)} noValidate>
<DialogSection>
<FormField
control={form.control}
name="name"
render={({ field }) => (
<FormItemLayout layout="vertical" label="Name">
<FormControl>
<Input {...field} placeholder="Enter text" />
</FormControl>
</FormItemLayout>
)}
/>
</DialogSection>
<DialogSection>
<FormField
control={form.control}
name="description"
render={({ field }) => (
<FormItemLayout layout="vertical" label="Description">
<FormControl>
<Textarea {...field} placeholder="Describe query" className="resize-none" />
</FormControl>
</FormItemLayout>
)}
/>
</DialogSection>
<DialogFooter>
<Button type="reset" onClick={handleCancel} disabled={isSubmitting}>
Cancel
</Button>
<Button
variant="primary"
type="submit"
loading={isSubmitting}
disabled={isSubmitting || !isDirty}
>
Save query
</Button>
</DialogFooter>
</form>
</Form>
</DialogContent>
</Dialog>
)
}