From 9d4658c526a80dcce2614127bf0c4ac425b443b4 Mon Sep 17 00:00:00 2001
From: KabootarGod <aryansinha1908@gmail.com>
Date: Thu, 22 Oct 2026 01:00:46 +0530
Subject: [PATCH] fix: use Next.js Link for methodology CTA with typed route

Replace the plain anchor with Next.js Link for client-side navigation,
type the methodology href as a Route to satisfy typedRoutes, and use
theme-aware primary-foreground text color.

Closes #259

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 src/features/comparison/components/result-dashboard.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/features/comparison/components/result-dashboard.tsx b/src/features/comparison/components/result-dashboard.tsx
index b7875b9..b6cdd66 100644
--- a/src/features/comparison/components/result-dashboard.tsx
+++ b/src/features/comparison/components/result-dashboard.tsx
@@ -93,7 +93,7 @@ export function ResultDashboard({
     }
   };
 
-  const methodologyHref = useMemo(() => {
+  const methodologyHref = useMemo<Route>(() => {
     const query = searchParams.toString();
     return query ? `/scoring-methodology?${query}` : "/scoring-methodology";
   }, [searchParams]);
@@ -640,12 +640,12 @@ export function ResultDashboard({
         </CardHeader>
         <CardContent className="space-y-3">
           <p className="text-sm text-muted-foreground">{t("methodology.cta.description")}</p>
-          <a
+          <Link
             href={methodologyHref}
-            className="inline-flex items-center rounded-lg bg-primary px-3 py-2 text-sm font-semibold text-white transition-colors hover:bg-blue-600 focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 focus-visible:outline-primary"
+            className="inline-flex items-center rounded-lg bg-primary px-3 py-2 text-sm font-semibold text-primary-foreground transition-colors hover:bg-primary focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 focus-visible:outline-primary"
           >
             {t("methodology.cta.button")}
-          </a>
+          </Link>
         </CardContent>
       </Card>
     </div>