mirror of
https://github.com/supabase/supabase.git
synced 2026-09-22 13:37:53 +08:00
476d4a5851
## 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
57 lines
1.4 KiB
TypeScript
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' } }}
|
|
/>
|
|
)
|
|
}
|