Remove redundant hover background color style in sidebar-item.tsx.

MartinBraquet · MartinBraquet · commit 1cae2510babc · 2026-04-10T14:27:37.000+02:00
diff --git a/web/components/nav/sidebar-item.tsx b/web/components/nav/sidebar-item.tsx
@@ -28,9 +28,7 @@ export function SidebarItem(props: {item: Item; currentPage?: string}) {
   }
 
   const sidebarClass = clsx(
-    isCurrentPage
-      ? 'bg-ink-100 text-primary-700'
-      : 'text-ink-600 hover:bg-ink-100 hover:text-primary-700',
+    isCurrentPage ? 'bg-ink-100 text-primary-700' : 'text-ink-600 hover:text-primary-700',
     'group flex items-center rounded-md px-3 py-2 text-sm font-medium',
     'focus-visible:bg-ink-100 outline-none transition-all',
   )

Original file line number	Diff line number	Diff line change
`@@ -28,9 +28,7 @@ export function SidebarItem(props: {item: Item; currentPage?: string}) {`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`const sidebarClass = clsx(`
`31`		`- isCurrentPage`
`32`		`- ? 'bg-ink-100 text-primary-700'`
`33`		`- : 'text-ink-600 hover:bg-ink-100 hover:text-primary-700',`
	`31`	`+ isCurrentPage ? 'bg-ink-100 text-primary-700' : 'text-ink-600 hover:text-primary-700',`
`34`	`32`	`'group flex items-center rounded-md px-3 py-2 text-sm font-medium',`
`35`	`33`	`'focus-visible:bg-ink-100 outline-none transition-all',`
`36`	`34`	`)`