From 7e55bcc5012ba9f50de1424b45d0fecc9a22ed52 Mon Sep 17 00:00:00 2001
From: Zakariye Mohamed <168684030+zakiscoding@users.noreply.github.com>
Date: Thu, 23 Apr 2026 16:53:16 -0400
Subject: [PATCH] chore: simplify account nav and signup role

Remove redundant Order History link (Past Orders covers it) and make signup role selection a clear Customer/Driver button toggle.

Made-with: Cursor
---
 app/components/AccountNav.tsx |  7 ------
 app/login/page.tsx            | 44 ++++++++++++++++++++---------------
 2 files changed, 25 insertions(+), 26 deletions(-)

diff --git a/app/components/AccountNav.tsx b/app/components/AccountNav.tsx
index ccd4d5a..38f9048 100644
--- a/app/components/AccountNav.tsx
+++ b/app/components/AccountNav.tsx
@@ -70,13 +70,6 @@ export function AccountNav({ userName, sidebarView, onSelectView, onClosePanel,
         <Separator className="my-4" />
 
         <div className="space-y-2">
-          <Link
-            href="/orders/history"
-            onClick={onClosePanel}
-            className="flex w-full items-center gap-3 rounded-lg px-4 py-3 text-sm font-medium hover:bg-muted transition-all"
-          >
-            🧾 Order History
-          </Link>
           <Link
             href="/rewards"
             onClick={onClosePanel}
diff --git a/app/login/page.tsx b/app/login/page.tsx
index 86c5399..ef5e1f6 100644
--- a/app/login/page.tsx
+++ b/app/login/page.tsx
@@ -239,6 +239,28 @@ export default function LoginPage() {
 
               <TabsContent value="signup" className="pt-4">
                 <form onSubmit={handleSubmit} className="space-y-4">
+                  <div className="rounded-lg border bg-background/50 p-3">
+                    <p className="text-xs font-semibold uppercase tracking-wider text-muted-foreground mb-2">
+                      Role
+                    </p>
+                    <div className="grid grid-cols-2 gap-2">
+                      <Button
+                        type="button"
+                        variant={role === "customer" ? "default" : "outline"}
+                        onClick={() => setRole("customer")}
+                      >
+                        🛒 Customer
+                      </Button>
+                      <Button
+                        type="button"
+                        variant={role === "driver" ? "default" : "outline"}
+                        onClick={() => setRole("driver")}
+                      >
+                        🚗 Driver
+                      </Button>
+                    </div>
+                  </div>
+
                   <div>
                     <label className="block text-sm font-semibold mb-2">Full name</label>
                     <Input
@@ -296,25 +318,9 @@ export default function LoginPage() {
                   </div>
 
                   <div className="pt-1">
-                    <p className="text-sm font-semibold mb-2">I am a:</p>
-                    <div className="grid grid-cols-2 gap-2">
-                      <Button
-                        type="button"
-                        variant={role === "customer" ? "default" : "outline"}
-                        onClick={() => setRole("customer")}
-                        className="justify-between"
-                      >
-                        🛒 Customer
-                      </Button>
-                      <Button
-                        type="button"
-                        variant={role === "driver" ? "default" : "outline"}
-                        onClick={() => setRole("driver")}
-                        className="justify-between"
-                      >
-                        🚗 Driver
-                      </Button>
-                    </div>
+                    <p className="text-xs text-muted-foreground">
+                      You can switch roles above before creating the account.
+                    </p>
                   </div>
 
                   {(error || localError) && (