diff --git a/.Jules/palette.md b/.Jules/palette.md new file mode 100644 index 00000000..4565fc06 --- /dev/null +++ b/.Jules/palette.md @@ -0,0 +1,3 @@ +## 2024-06-15 - Missing Tooltips on Icon-Only Buttons in Topbar +**Learning:** The Topbar component has icon-only buttons for Notifications, User Menu, and Settings. They use `aria-label`, which is good for screen readers, but sighted keyboard and mouse users don't get any visual context. The lack of tooltips is an accessibility and UX miss for sighted users who may not immediately recognize the icons, or who navigate with a keyboard and need visual confirmation of where their focus is. +**Action:** Add tooltips to the icon-only buttons in the Topbar component, wrapping the `Button` components with `Tooltip` and `TooltipTrigger`, and providing a `TooltipContent` with a clear description of the button's action. This improves UX for all users. diff --git a/src/components/nav/topbar.tsx b/src/components/nav/topbar.tsx index 9d1d921c..7d607535 100644 --- a/src/components/nav/topbar.tsx +++ b/src/components/nav/topbar.tsx @@ -4,6 +4,7 @@ import React from 'react' import { Search, Bell, User, Settings as SettingsIcon, Command } from 'lucide-react' import { cn } from '@/lib/utils' import { Button } from '@/components/ui/button' +import { Tooltip, TooltipContent, TooltipTrigger } from '@/components/ui/tooltip' import { DropdownMenu, DropdownMenuContent, @@ -53,27 +54,37 @@ export function Topbar() { {/* Right Side - Actions */}
{/* Notifications */} - - - {/* User Menu */} - - + + - + + Notifications + + + {/* User Menu */} + + + + + + + + User menu + {/* Settings Link */} - + + + + + Settings +
)