Skip to content

Commit 2a46419

Browse files
Copilotrajbos
andcommitted
Fix power user button functionality by restructuring tooltip/sheet interaction
Co-authored-by: rajbos <6085745+rajbos@users.noreply.github.com>
1 parent d5d40aa commit 2a46419

1 file changed

Lines changed: 18 additions & 18 deletions

File tree

src/App.tsx

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -356,25 +356,19 @@ function App() {
356356
</span>
357357
</div>
358358
{powerUserSummary && (
359-
<Sheet>
360-
<SheetTrigger asChild>
361-
<UITooltip>
362-
<TooltipTrigger asChild>
359+
<UITooltip>
360+
<TooltipTrigger asChild>
361+
<Sheet>
362+
<SheetTrigger asChild>
363363
<Button variant="outline" className="flex items-center gap-2">
364364
<span className="text-sm">Power Users:</span>
365365
<span className="font-bold">{powerUserSummary.totalPowerUsers}</span>
366366
</Button>
367-
</TooltipTrigger>
368-
<TooltipContent>
369-
<p>Power users are the top 10% of users by request count.<br/>
370-
These users make the most requests to GitHub Copilot.</p>
371-
</TooltipContent>
372-
</UITooltip>
373-
</SheetTrigger>
374-
<SheetContent className="w-[800px] sm:w-[1000px] lg:w-[1200px] overflow-y-auto">
375-
<SheetHeader>
376-
<SheetTitle>Power Users Analysis</SheetTitle>
377-
</SheetHeader>
367+
</SheetTrigger>
368+
<SheetContent className="w-[800px] sm:w-[1000px] lg:w-[1200px] overflow-y-auto">
369+
<SheetHeader>
370+
<SheetTitle>Power Users Analysis</SheetTitle>
371+
</SheetHeader>
378372
<div className="mt-6 space-y-6">
379373
{/* Power User Summary */}
380374
<div className="grid grid-cols-1 lg:grid-cols-2 gap-4">
@@ -491,9 +485,15 @@ function App() {
491485
</Table>
492486
</div>
493487
</Card>
494-
</div>
495-
</SheetContent>
496-
</Sheet>
488+
</div>
489+
</SheetContent>
490+
</Sheet>
491+
</TooltipTrigger>
492+
<TooltipContent>
493+
<p>Power users are the top 10% of users by request count.<br/>
494+
These users make the most requests to GitHub Copilot.</p>
495+
</TooltipContent>
496+
</UITooltip>
497497
)}
498498
</div>
499499
</div>

0 commit comments

Comments
 (0)