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

57 lines
1.4 KiB
TypeScript

import { useState } from 'react'
import { type DateRange } from 'react-day-picker'
import {
Button,
Calendar,
Dialog,
DialogContent,
DialogFooter,
DialogHeader,
DialogTitle,
} from 'ui'
export const LogsCustomRangeDialog = ({
open,
onOpenChange,
onApply,
}: {
open: boolean
onOpenChange: (open: boolean) => void
onApply: (range: { from: Date; to: Date }) => void
}) => {
const [range, setRange] = useState<DateRange | undefined>()
const canApply = range?.from !== undefined && range?.to !== undefined
const handleApply = () => {
if (range?.from === undefined || range?.to === undefined) return
onApply({ from: range.from, to: range.to })
onOpenChange(false)
}
return (
<Dialog open={open} onOpenChange={onOpenChange}>
<DialogContent size="large">
<DialogHeader>
<DialogTitle>Custom time range</DialogTitle>
</DialogHeader>
<div className="flex items-center justify-center overflow-x-auto px-2 py-2">
<Calendar
mode="range"
selected={range}
onSelect={setRange}
numberOfMonths={2}
hidden={{ after: new Date() }}
/>
</div>
<DialogFooter>
<Button onClick={() => onOpenChange(false)}>Cancel</Button>
<Button variant="primary" disabled={!canApply} onClick={handleApply}>
Apply
</Button>
</DialogFooter>
</DialogContent>
</Dialog>
)
}