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 { LoaderCircle, RefreshCcw } from 'lucide-react'
import { Button } from 'ui'
import { ButtonTooltip } from '../ButtonTooltip'
import { Shortcut } from '@/components/ui/Shortcut'
import type { ShortcutId } from '@/state/shortcuts/registry'
interface RefreshButtonProps {
isLoading: boolean
onRefresh: () => void
/**
* When provided, the button binds this registered shortcut (hotkey + a
* tooltip that surfaces the keybind). Otherwise it falls back to a plain
* "Refresh logs" tooltip with no keybind.
*/
shortcutId?: ShortcutId
}
export const RefreshButton = ({ isLoading, onRefresh, shortcutId }: RefreshButtonProps) => {
const icon = isLoading ? (
<LoaderCircle className="text-foreground animate-spin" />
) : (
<RefreshCcw className="text-foreground" />
)
if (shortcutId) {
return (
<Shortcut
id={shortcutId}
onTrigger={onRefresh}
options={{ enabled: !isLoading, registerInCommandMenu: true }}
side="bottom"
>
<Button
size="tiny"
disabled={isLoading}
onClick={onRefresh}
className="w-[26px]"
icon={icon}
aria-label="Refresh logs"
/>
</Shortcut>
)
}
return (
<ButtonTooltip
size="tiny"
disabled={isLoading}
onClick={onRefresh}
className="w-[26px]"
icon={icon}
tooltip={{ content: { side: 'bottom', text: 'Refresh logs' } }}
/>
)
}