mirror of
https://github.com/supabase/supabase.git
synced 2026-10-06 18:05:11 +03:00
## What kind of change does this PR introduce? Bug fix. Resolves [FE-4192](https://linear.app/supabase/issue/FE-4192/org-and-project-selectors-sometimes-dont-register-selections). ## What is the current behavior? Navigation actions sometimes nest links inside command or dropdown menu items. Closing the menu during selection can prevent the nested link navigation from registering. ## What is the new behavior? - Adds a documented Studio CommandItemLink composition that wraps command items with their navigation link. - Migrates all Studio command-item links, including organisation, project, function, database, branch, and integration actions. - Uses the dropdown menu asChild composition for both infrastructure-diagram Manage replica actions. - Preserves native link behaviour and leaves disabled command items non-navigable. ## To test - [ ] [Organisation and project selectors](https://studio-staging-git-dnywh-fe-4192-selector-links-supabase.vercel.app/dashboard/org): open the organisation selector and try an organisation, All Organizations, and New organization. Open a project, then use the project selector to switch projects and open New project. Confirm every action navigates on the first click. - [ ] [Branch selector](https://studio-staging-git-dnywh-fe-4192-selector-links-supabase.vercel.app/dashboard/project/_): in a project with branching enabled, open the branch selector. Switch branches and select Manage branches. Confirm both navigate on the first click. - [ ] [Database selector](https://studio-staging-git-dnywh-fe-4192-selector-links-supabase.vercel.app/dashboard/project/_/observability/query-performance): open the Source selector. Switch between the primary database and a read replica if available, then select Create a new read replica. Confirm selections apply and the footer action navigates on the first click. - [ ] [Function selector](https://studio-staging-git-dnywh-fe-4192-selector-links-supabase.vercel.app/dashboard/project/_/auth/hooks): select Add a new hook, choose a hook, select Postgres, then open the Postgres function selector and select New function. Confirm it navigates on the first click. - [ ] [Infrastructure diagram](https://studio-staging-git-dnywh-fe-4192-selector-links-supabase.vercel.app/dashboard/project/_/settings/infrastructure): for a project with a read replica, select Manage replica from both diagram variants. Confirm the replica settings open on the first click. - [ ] On any navigational row above, modifier-click and confirm native link behaviour is preserved. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added consistent link navigation across organization, project, branch, function, replica, and integration menus. * Added project-specific destinations to organization and project selectors. * Preserved disabled-item behavior while improving accessible command-menu link semantics. * **Bug Fixes** * Improved navigation and menu-closing behavior for command items and dropdown actions. * **Tests** * Added coverage for link destinations, accessibility roles, disabled states, and route preservation. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
139 lines
3.8 KiB
TypeScript
139 lines
3.8 KiB
TypeScript
import { Plus } from 'lucide-react'
|
|
import Link from 'next/link'
|
|
import {
|
|
Button,
|
|
cn,
|
|
Command,
|
|
CommandEmpty,
|
|
CommandGroup,
|
|
CommandInput,
|
|
CommandList,
|
|
CommandSeparator,
|
|
ScrollArea,
|
|
} from 'ui'
|
|
|
|
import { OrgCommandItem } from './OrgCommandItem'
|
|
import { CommandItemLink } from '@/components/ui/CommandItemLink'
|
|
import type { Organization } from '@/types'
|
|
|
|
export interface OrganizationDropdownCommandContentProps {
|
|
embedded: boolean
|
|
className?: string
|
|
organizations: Organization[]
|
|
selectedSlug: string | undefined
|
|
routePathname: string
|
|
hasRouteSlug: boolean
|
|
organizationCreationEnabled: boolean
|
|
onClose: () => void
|
|
}
|
|
|
|
export function OrganizationDropdownCommandContent({
|
|
embedded,
|
|
className,
|
|
organizations,
|
|
selectedSlug,
|
|
routePathname,
|
|
hasRouteSlug,
|
|
organizationCreationEnabled,
|
|
onClose,
|
|
}: OrganizationDropdownCommandContentProps) {
|
|
const orgList = (
|
|
<>
|
|
{organizations?.map((org) => (
|
|
<OrgCommandItem
|
|
key={org.slug}
|
|
org={org}
|
|
selectedSlug={selectedSlug}
|
|
routePathname={routePathname}
|
|
hasRouteSlug={hasRouteSlug}
|
|
onClose={onClose}
|
|
compactPadding={!embedded}
|
|
/>
|
|
))}
|
|
</>
|
|
)
|
|
|
|
if (embedded) {
|
|
return (
|
|
<Command className={cn(className, 'flex flex-col flex-1 min-h-0 overflow-hidden')}>
|
|
<div className="flex items-center gap-2 shrink-0 border-b p-2">
|
|
<Button variant="text" block size="small" asChild>
|
|
<Link
|
|
href="/organizations"
|
|
className="text-xs text-foreground-light hover:text-foreground"
|
|
onClick={onClose}
|
|
>
|
|
All Organizations
|
|
</Link>
|
|
</Button>
|
|
{organizationCreationEnabled && (
|
|
<Button
|
|
variant="default"
|
|
block
|
|
size="small"
|
|
asChild
|
|
icon={<Plus size={14} strokeWidth={1.5} />}
|
|
>
|
|
<Link
|
|
href="/new"
|
|
className="text-xs text-foreground-light hover:text-foreground"
|
|
onClick={onClose}
|
|
>
|
|
New organization
|
|
</Link>
|
|
</Button>
|
|
)}
|
|
</div>
|
|
<CommandInput
|
|
placeholder="Find organization..."
|
|
wrapperClassName="shrink-0"
|
|
className="text-base sm:text-sm"
|
|
/>
|
|
<CommandList className="flex flex-col flex-1 min-h-0 overflow-y-auto p-1 max-h-none!">
|
|
<CommandEmpty>No organizations found</CommandEmpty>
|
|
<CommandGroup className="min-h-0">{orgList}</CommandGroup>
|
|
</CommandList>
|
|
</Command>
|
|
)
|
|
}
|
|
|
|
return (
|
|
<Command className={className}>
|
|
<CommandInput placeholder="Find organization..." />
|
|
<CommandList>
|
|
<CommandEmpty>No organizations found</CommandEmpty>
|
|
<CommandGroup>
|
|
<ScrollArea className={(organizations || []).length > 7 ? 'md:h-[210px]' : ''}>
|
|
{orgList}
|
|
</ScrollArea>
|
|
</CommandGroup>
|
|
<CommandSeparator />
|
|
<CommandGroup>
|
|
<CommandItemLink
|
|
href="/organizations"
|
|
className="cursor-pointer w-full gap-2"
|
|
onSelect={onClose}
|
|
>
|
|
All Organizations
|
|
</CommandItemLink>
|
|
</CommandGroup>
|
|
{organizationCreationEnabled && (
|
|
<>
|
|
<CommandSeparator />
|
|
<CommandGroup>
|
|
<CommandItemLink
|
|
href="/new"
|
|
className="cursor-pointer w-full gap-2"
|
|
onSelect={onClose}
|
|
>
|
|
<Plus size={14} strokeWidth={1.5} />
|
|
<p>New organization</p>
|
|
</CommandItemLink>
|
|
</CommandGroup>
|
|
</>
|
|
)}
|
|
</CommandList>
|
|
</Command>
|
|
)
|
|
}
|