mirror of
https://github.com/supabase/supabase.git
synced 2026-10-05 17:35:10 +03:00
## 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
70 lines
1.9 KiB
TypeScript
70 lines
1.9 KiB
TypeScript
import { useParams } from 'common'
|
|
import { useState } from 'react'
|
|
import { toast } from 'sonner'
|
|
import {
|
|
Button,
|
|
Dialog,
|
|
DialogContent,
|
|
DialogFooter,
|
|
DialogHeader,
|
|
DialogSection,
|
|
DialogSectionSeparator,
|
|
DialogTitle,
|
|
DialogTrigger,
|
|
} from 'ui'
|
|
|
|
import { LintInfo } from '../Linter/Linter.constants'
|
|
import { useLintRuleDeleteMutation } from '@/data/lint/delete-lint-rule-mutation'
|
|
import { LintException } from '@/data/lint/lint-rules-query'
|
|
|
|
interface EnableRuleModalProps {
|
|
lint: LintInfo
|
|
rule: LintException
|
|
}
|
|
|
|
export const EnableRuleModal = ({ lint, rule }: EnableRuleModalProps) => {
|
|
const { ref } = useParams()
|
|
|
|
const [open, setOpen] = useState(false)
|
|
|
|
const { mutate: deleteRule, isPending: isDeleting } = useLintRuleDeleteMutation({
|
|
onSuccess: () => {
|
|
toast.success(`Successfully enabled the "${lint.title}" rule`)
|
|
setOpen(false)
|
|
},
|
|
})
|
|
|
|
const onDeleteRule = () => {
|
|
if (!ref) return console.error('Project ref is required')
|
|
deleteRule({ projectRef: ref, ids: [rule.id] })
|
|
}
|
|
|
|
return (
|
|
<Dialog open={open} onOpenChange={setOpen}>
|
|
<DialogTrigger asChild>
|
|
<Button>Enable rule</Button>
|
|
</DialogTrigger>
|
|
<DialogContent size="small">
|
|
<DialogHeader>
|
|
<DialogTitle>Enable rule</DialogTitle>
|
|
</DialogHeader>
|
|
<DialogSectionSeparator />
|
|
<DialogSection>
|
|
<p className="text-sm">
|
|
The "{lint.title}" rule will be visible in the Advisor reports, and will be included in
|
|
email notifications for this project.
|
|
</p>
|
|
</DialogSection>
|
|
<DialogFooter>
|
|
<Button disabled={isDeleting} onClick={() => setOpen(false)}>
|
|
Cancel
|
|
</Button>
|
|
<Button loading={isDeleting} variant="primary" onClick={onDeleteRule}>
|
|
Enable
|
|
</Button>
|
|
</DialogFooter>
|
|
</DialogContent>
|
|
</Dialog>
|
|
)
|
|
}
|