Fix nav active state hidden by conflicting hover class

[Copilot]

bg-blue-200 on active NavLinks was silently overridden by the base hover:bg-accent class on hover, making the active indicator vanish. Also replaced the non-theme bg-blue-200 with the proper bg-accent token.

Changes

• Navigation.tsx
  • Swapped bg-blue-200 → bg-accent for active state (theme-correct variable)
  • Moved hover classes out of the base string into the isActive conditional to eliminate Tailwind class conflicts
  • Active hover now uses hover:bg-accent/80 (dims the active bg) instead of hover:bg-accent (which was replacing it)
  • Extracted shared getNavLinkClasses(isActive: boolean) helper to deduplicate identical logic across Report and Settings NavLinks

// Before — hover:bg-accent in base overwrites bg-blue-200 on active links
`... bg-white hover:bg-accent hover:accent-foreground ... ${isActive ? 'bg-blue-200 hover:bg-accent hover:text-accent-foreground' : 'bg-white'}`

// After — state fully owned by the conditional; active hover is additive
`... border border-gray-200 hover:border-input ${isActive ? 'bg-accent hover:bg-accent/80 hover:text-accent-foreground' : 'bg-white hover:bg-accent hover:text-accent-foreground'}`

[cloudflare-workers-and-pages]

Deploying timetrackerpro with    Cloudflare Pages

Latest commit:	4004974
Status:	✅  Deploy successful!
Preview URL:	https://2cc4a4c6.timetrackerpro.pages.dev
Branch Preview URL:	https://copilot-fix-conflicting-colo.timetrackerpro.pages.dev

View logs