mirror of
https://github.com/supabase/supabase.git
synced 2026-10-06 01:45:10 +03:00
Danny White
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
90 lines
3.0 KiB
TypeScript
90 lines
3.0 KiB
TypeScript
import { SupportCategories } from '@supabase/shared-types/out/constants'
|
|
import { Blocks, ExternalLink } from 'lucide-react'
|
|
import { useRouter } from 'next/router'
|
|
import { Button } from 'ui'
|
|
|
|
import { SupportLink } from '@/components/interfaces/Support/SupportLink'
|
|
import { detectBrowser } from '@/lib/helpers'
|
|
|
|
interface InsertBeforeRemoveChildErrorHandlerProps {
|
|
message: string
|
|
sentryIssueId: string
|
|
urlMessage: string
|
|
isRemoveChildError: boolean
|
|
isInsertBeforeError?: boolean
|
|
}
|
|
|
|
export const InsertBeforeRemoveChildErrorHandler = ({
|
|
message,
|
|
sentryIssueId,
|
|
urlMessage,
|
|
isRemoveChildError,
|
|
isInsertBeforeError,
|
|
}: InsertBeforeRemoveChildErrorHandlerProps) => {
|
|
const router = useRouter()
|
|
const browser = detectBrowser()
|
|
|
|
return (
|
|
<>
|
|
<div className="flex flex-col gap-y-4 text-left py-2 w-full">
|
|
<div className="flex items-center gap-x-3">
|
|
<p className="text-lg font-bold">Sorry! A browser extension may have caused an error.</p>
|
|
<Blocks className="text-foreground-lighter" />
|
|
</div>
|
|
|
|
<div className="flex flex-col gap-y-2">
|
|
<p className="text-sm text-foreground-light">
|
|
Browser translation tools (like Chrome's built-in Translate) or some third-party browser
|
|
extensions are known to cause errors when using the Supabase Dashboard.
|
|
</p>
|
|
|
|
<p className="text-sm text-foreground-light">
|
|
We highly recommend{' '}
|
|
<span className="text-foreground">
|
|
{browser === 'Chrome'
|
|
? 'disabling Chrome Translate or certain browser extensions'
|
|
: 'avoiding the use of browser translation tools or disabling certain extensions'}
|
|
</span>{' '}
|
|
while using the Supabase Dashboard to avoid running into this error. Try to refresh the
|
|
browser to see if it occurs again.
|
|
</p>
|
|
</div>
|
|
|
|
<p className="text-foreground-lighter text-sm">Error: {message}</p>
|
|
</div>
|
|
|
|
<div className="flex gap-x-2 justify-center items-center">
|
|
<Button asChild icon={<ExternalLink />}>
|
|
<a
|
|
target="_blank"
|
|
rel="noreferrer"
|
|
href={
|
|
isRemoveChildError
|
|
? 'https://github.com/facebook/react/issues/17256'
|
|
: isInsertBeforeError
|
|
? 'https://github.com/facebook/react/issues/24865'
|
|
: '/'
|
|
}
|
|
>
|
|
More information
|
|
</a>
|
|
</Button>
|
|
<Button variant="outline" onClick={() => router.reload()}>
|
|
Refresh page
|
|
</Button>
|
|
</div>
|
|
<SupportLink
|
|
className="text-center text-xs text-foreground-lighter hover:text-foreground-light transition"
|
|
queryParams={{
|
|
category: SupportCategories.DASHBOARD_BUG,
|
|
subject: `Client error: Failed to execute '${isRemoveChildError ? 'removeChild' : 'insertBefore'}' on 'Node'`,
|
|
sid: sentryIssueId,
|
|
error: urlMessage,
|
|
}}
|
|
>
|
|
Still stuck?
|
|
</SupportLink>
|
|
</>
|
|
)
|
|
}
|