feat: Replace Navbar tooltip with native title attribute and introduce onToggleOpen prop for Drawer close functionality.

This commit is contained in:
2025-12-05 02:36:57 -05:00
parent 2294f0a631
commit 132283e216
3 changed files with 17 additions and 23 deletions

View File

@@ -42,6 +42,7 @@ export default function HomePage() {
onSelect={setSelectedShop}
onClose={() => setSelectedShop(null)}
isOpen={isDiscoveryOpen}
onToggleOpen={() => setIsDiscoveryOpen(false)}
/>
<WelcomeModal />
</main>

View File

@@ -25,9 +25,10 @@ interface DrawerProps {
onSelect: (shop: CoffeeShop) => void;
onClose: () => void;
isOpen: boolean;
onToggleOpen?: () => void;
}
export default function Drawer({ shop, shops, onSelect, onClose, isOpen }: DrawerProps) {
export default function Drawer({ shop, shops, onSelect, onClose, isOpen, onToggleOpen }: DrawerProps) {
const [searchQuery, setSearchQuery] = useState("");
const [imageLoading, setImageLoading] = useState(true);
@@ -149,7 +150,7 @@ export default function Drawer({ shop, shops, onSelect, onClose, isOpen }: Drawe
<Button
variant="ghost"
size="icon"
onClick={onClose}
onClick={onToggleOpen}
className="absolute top-4 right-4 bg-background/20 hover:bg-background/40 text-foreground rounded-full h-8 w-8 backdrop-blur-md border border-border/50"
>
<X className="w-4 h-4" />

View File

@@ -72,12 +72,10 @@ export default function Navbar({ isDiscoveryOpen, onToggleDiscovery }: NavbarPro
</TooltipProvider>
</div>
<TooltipProvider>
<Tooltip>
<TooltipTrigger asChild>
<div
className="flex items-center gap-3 px-2 cursor-pointer group absolute left-1/2 top-1/2 -translate-x-1/2 -translate-y-1/2"
onClick={handleHeaderClick}
title="About This Map"
>
<div className="p-2 rounded-lg group-hover:bg-primary/10 transition-colors">
<Coffee className="h-6 w-6 text-primary" />
@@ -87,12 +85,6 @@ export default function Navbar({ isDiscoveryOpen, onToggleDiscovery }: NavbarPro
<p className="text-xs text-muted-foreground font-serif mt-0.5">Find&nbsp;your&nbsp;perfect&nbsp;brew</p>
</div>
</div>
</TooltipTrigger>
<TooltipContent side="bottom" className="bg-background/80 backdrop-blur-xl border-border/50 text-foreground font-semibold shadow-2xl">
<p>About This Map</p>
</TooltipContent>
</Tooltip>
</TooltipProvider>
<div className="w-10" /> {/* Spacer to balance the toggle button */}
</div>