Files
Gildas Garcia d0fd4478c0 chore: migrate Popover usages to Shadcn components (#45980)
## Problem

We have multiple Popover components

## Solution

- [x] migrate Popover usages to Shadcn components
- Migrated JSON and text editor in the `TableEditor` (inline row
edition)
  - Migrated the template popover in the logs explorer templates page
- [x] remove `_Shadcn_` suffix from Popover components (renaming +
prettier)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Refactor**
* Unified popover implementation across the app and design system;
dropdowns, calendars, menus and tooltips now use a consistent popover
API with no visual or interaction changes.

* **Chores**
* Minor prop typing update for the logs date-picker to align with the
consolidated popover content type.

<!-- review_stack_entry_start -->

[![Review Change
Stack](https://storage.googleapis.com/coderabbit_public_assets/review-stack-in-coderabbit-ui.svg)](https://app.coderabbit.ai/change-stack/supabase/supabase/pull/45980)

<!-- review_stack_entry_end -->
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
2026-05-15 15:20:28 +02:00

123 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
asChild
onClick={() => {
setIsOpen((isOpen) => !isOpen)
setStage('select')
}}
type="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 type="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 type="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 type="default" size="tiny" onClick={() => setStage('widget')}>
Leave feedback instead
</Button>
</div>
</>
)}
{stage === 'widget' && (
<FeedbackWidget
onClose={() => setIsOpen(false)}
onSwitchToIssueOptions={() => setStage('issue-options')}
/>
)}
</PopoverContent>
</Popover>
)
}