diff options
| author | Shivam Dua <trugamr@live.in> | 2024-09-30 01:23:00 +0530 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-09-29 20:53:00 +0100 |
| commit | 8fe1927323b5e44354f43a5c1243fcd49c02f762 (patch) | |
| tree | 710eacc7f41d88962edf8f505048f0bb1c975272 /apps/web/components/dashboard/sidebar/SidebarItem.tsx | |
| parent | 36fb5a4c63aada8e8107b8e9d97a6ba128d13494 (diff) | |
| download | karakeep-8fe1927323b5e44354f43a5c1243fcd49c02f762.tar.zst | |
fix(web): Improve SidebarItem tap target accessibility (#409)
Diffstat (limited to 'apps/web/components/dashboard/sidebar/SidebarItem.tsx')
| -rw-r--r-- | apps/web/components/dashboard/sidebar/SidebarItem.tsx | 26 |
1 files changed, 18 insertions, 8 deletions
diff --git a/apps/web/components/dashboard/sidebar/SidebarItem.tsx b/apps/web/components/dashboard/sidebar/SidebarItem.tsx index 262fd9ae..83ce776e 100644 --- a/apps/web/components/dashboard/sidebar/SidebarItem.tsx +++ b/apps/web/components/dashboard/sidebar/SidebarItem.tsx @@ -10,6 +10,7 @@ export default function SidebarItem({ logo, path, className, + linkClassName, style, collapseButton, right = null, @@ -19,6 +20,7 @@ export default function SidebarItem({ path: string; style?: React.CSSProperties; className?: string; + linkClassName?: string; right?: React.ReactNode; collapseButton?: React.ReactNode; }) { @@ -26,20 +28,28 @@ export default function SidebarItem({ return ( <li className={cn( - "relative rounded-lg px-3 py-2 hover:bg-accent", + "relative rounded-lg hover:bg-accent", path == currentPath ? "bg-accent/50" : "", className, )} style={style} > {collapseButton} - <div className="flex justify-between"> - <Link href={path} className="flex w-full gap-x-2"> - {logo} - <span className="my-auto"> {name} </span> - </Link> - {right} - </div> + <Link + href={path} + className={cn( + "flex w-full items-center rounded-[inherit] px-3 py-2", + linkClassName, + )} + > + <div className="flex w-full justify-between"> + <div className="flex items-center gap-x-2"> + {logo} + <span>{name}</span> + </div> + {right} + </div> + </Link> </li> ); } |
