mirror of
https://github.com/supabase/supabase.git
synced 2026-10-09 03:15:06 +03:00
## Problem On the organization home page: - you can't tab to a project card and navigate to the project - the status filter popover cannot be open with keyboard - the feedback popover cannot be open with keyboard ## Solution - make the project card (which is a link) accessible with Tab - fix the popover trigger buttons <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Bug Fixes** * Improved keyboard accessibility so project cards can be focused with Tab navigation. * Updated dropdown and filter popover trigger wiring for more consistent click behavior. * Reset the feedback flow to its starting step whenever the trigger is clicked. * **Bug Fixes** * Made the home icon link explicitly focusable via keyboard navigation. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
122 lines
4.8 KiB
TypeScript
122 lines
4.8 KiB
TypeScript
import { IS_PLATFORM } from 'common'
|
|
import { Lightbulb, TriangleAlert } from 'lucide-react'
|
|
import { useRouter } from 'next/router'
|
|
import { useState } from 'react'
|
|
import { Button, Popover, PopoverContent, PopoverSeparator, PopoverTrigger } from 'ui'
|
|
|
|
import { FeedbackWidget } from './FeedbackWidget'
|
|
import { SIDEBAR_KEYS } from '@/components/layouts/ProjectLayout/LayoutSidebar/LayoutSidebarProvider'
|
|
import { ASSISTANT_SUGGESTIONS } from '@/components/ui/HelpPanel/HelpPanel.constants'
|
|
import { getSupportLinkQueryParams } from '@/components/ui/HelpPanel/HelpPanel.utils'
|
|
import { HelpSection } from '@/components/ui/HelpPanel/HelpSection'
|
|
import { useSelectedOrganizationQuery } from '@/hooks/misc/useSelectedOrganization'
|
|
import { useSelectedProjectQuery } from '@/hooks/misc/useSelectedProject'
|
|
import { useTrack } from '@/lib/telemetry/track'
|
|
import { useAiAssistantStateSnapshot } from '@/state/ai-assistant-state'
|
|
import { useSidebarManagerSnapshot } from '@/state/sidebar-manager-state'
|
|
|
|
export const FeedbackDropdown = ({ className }: { className?: string }) => {
|
|
const router = useRouter()
|
|
const { data: project } = useSelectedProjectQuery()
|
|
const { data: org } = useSelectedOrganizationQuery()
|
|
const snap = useAiAssistantStateSnapshot()
|
|
const { openSidebar } = useSidebarManagerSnapshot()
|
|
const [isOpen, setIsOpen] = useState(false)
|
|
const [stage, setStage] = useState<'select' | 'issue-options' | 'widget'>('select')
|
|
const track = useTrack()
|
|
|
|
const projectRef = project?.parent_project_ref ?? (router.query.ref as string | undefined)
|
|
const supportLinkQueryParams = getSupportLinkQueryParams(
|
|
project,
|
|
org,
|
|
router.query.ref as string | undefined
|
|
)
|
|
|
|
return (
|
|
<Popover
|
|
modal={false}
|
|
open={isOpen}
|
|
onOpenChange={(e) => {
|
|
if (e) track('header_feedback_dropdown_opened')
|
|
setIsOpen(e)
|
|
if (!e) setStage('select')
|
|
}}
|
|
>
|
|
<PopoverTrigger asChild>
|
|
<Button
|
|
onClick={() => {
|
|
setIsOpen((isOpen) => !isOpen)
|
|
setStage('select')
|
|
}}
|
|
variant="text"
|
|
className="rounded-full h-[32px] text-foreground-light hover:text-foreground"
|
|
>
|
|
<span className={className}>Feedback</span>
|
|
</Button>
|
|
</PopoverTrigger>
|
|
<PopoverContent
|
|
side="bottom"
|
|
align="end"
|
|
className="p-0 flex flex-col w-96"
|
|
id="feedback-widget"
|
|
>
|
|
{stage === 'select' && (
|
|
<div className="flex flex-col gap-4 p-4">
|
|
<div className="font-medium text-sm">What would you like to share?</div>
|
|
<div className="grid grid-cols-2 gap-3">
|
|
<Button variant="default" className="h-32" onClick={() => setStage('issue-options')}>
|
|
<div className="grid gap-1.5 text-center">
|
|
<TriangleAlert size="28" className="mx-auto text-destructive-600" />
|
|
<div className="flex flex-col items-center">
|
|
<span className="text-base">Issue</span>
|
|
<span className="text-xs text-foreground-lighter">with my project</span>
|
|
</div>
|
|
</div>
|
|
</Button>
|
|
<Button variant="default" className="h-32" onClick={() => setStage('widget')}>
|
|
<div className="grid gap-1.5 text-center">
|
|
<Lightbulb size="28" className="mx-auto text-warning" />
|
|
<div className="flex flex-col items-center">
|
|
<span className="text-base">Idea</span>
|
|
<span className="text-xs text-foreground-lighter">to improve Supabase</span>
|
|
</div>
|
|
</div>
|
|
</Button>
|
|
</div>
|
|
</div>
|
|
)}
|
|
{stage === 'issue-options' && (
|
|
<>
|
|
<div className="flex flex-col gap-4 p-4">
|
|
<HelpSection
|
|
excludeIds={[]}
|
|
isPlatform={IS_PLATFORM}
|
|
projectRef={projectRef}
|
|
supportLinkQueryParams={supportLinkQueryParams}
|
|
onAssistantClick={() => {
|
|
openSidebar(SIDEBAR_KEYS.AI_ASSISTANT)
|
|
snap.newChat(ASSISTANT_SUGGESTIONS)
|
|
setIsOpen(false)
|
|
}}
|
|
onSupportClick={() => setIsOpen(false)}
|
|
/>
|
|
</div>
|
|
<PopoverSeparator />
|
|
<div className="px-4 pt-4 pb-4">
|
|
<Button variant="default" size="tiny" onClick={() => setStage('widget')}>
|
|
Leave feedback instead
|
|
</Button>
|
|
</div>
|
|
</>
|
|
)}
|
|
{stage === 'widget' && (
|
|
<FeedbackWidget
|
|
onClose={() => setIsOpen(false)}
|
|
onSwitchToIssueOptions={() => setStage('issue-options')}
|
|
/>
|
|
)}
|
|
</PopoverContent>
|
|
</Popover>
|
|
)
|
|
}
|