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

87 lines
2.5 KiB
TypeScript

import { ChevronDown, Terminal } from 'lucide-react'
import { useState } from 'react'
import {
Button,
Command,
CommandGroup,
CommandItem,
CommandList,
Popover,
PopoverContent,
PopoverTrigger,
} from 'ui'
import { BASE_PATH } from '@/lib/constants'
import { useAppStateSnapshot } from '@/state/app-state'
interface LanguageSelectorProps {
/**
* Only show icons to represent the language
*/
simplifiedVersion?: boolean
}
const LanguageSelector = ({ simplifiedVersion = false }: LanguageSelectorProps) => {
const snap = useAppStateSnapshot()
const [showLanguage, setShowLanguage] = useState(false)
const updateLanguage = (value: 'js' | 'bash') => {
snap.setDocsLanguage(value)
setShowLanguage(false)
}
return (
<div className="flex items-center gap-x-2">
<Popover modal={false} open={showLanguage} onOpenChange={setShowLanguage}>
<PopoverTrigger asChild>
<Button
className={simplifiedVersion ? 'px-1' : ''}
icon={
simplifiedVersion ? (
snap.docsLanguage === 'js' ? (
<img
src={`${BASE_PATH}/img/libraries/javascript-icon.svg`}
alt={`javascript logo`}
width="14"
/>
) : (
<Terminal size={14} strokeWidth={2.5} />
)
) : undefined
}
iconRight={!simplifiedVersion && <ChevronDown size={14} strokeWidth={2} />}
>
{!simplifiedVersion
? `Language: ${snap.docsLanguage === 'js' ? 'Javascript' : 'Bash'}`
: undefined}
</Button>
</PopoverTrigger>
<PopoverContent className="p-0 w-24" side="bottom" align="end">
<Command>
<CommandList>
<CommandGroup>
<CommandItem
className="cursor-pointer"
onSelect={() => updateLanguage('js')}
onClick={() => updateLanguage('js')}
>
<p>Javascript</p>
</CommandItem>
<CommandItem
className="cursor-pointer"
onSelect={() => updateLanguage('bash')}
onClick={() => updateLanguage('bash')}
>
<p>Bash</p>
</CommandItem>
</CommandGroup>
</CommandList>
</Command>
</PopoverContent>
</Popover>
</div>
)
}
export default LanguageSelector