Files
supabase/apps/studio/components/layouts/Navigation/LayoutHeader/FeedbackDropdown/FeedbackDropdown.tsx
Gildas Garcia 0eeeb758d8 fix: homepage accessibility fixes (#47729)
## 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 -->
2026-07-08 11:44:48 +02:00

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>
)
}