Clean up

sgcarstrends · Dec 2, 2024 · f09bca7 · f09bca7
1 parent f6f1735
commit f09bca7
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 8 deletions.
diff --git a/app/(dashboard)/layout.tsx b/app/(dashboard)/layout.tsx
@@ -5,7 +5,6 @@ import { GoogleAnalytics } from "@next/third-parties/google";
 import classNames from "classnames";
 import { Analytics } from "@/app/components/Analytics";
 import { Announcement } from "@/app/components/Announcement";
-// import { Footer } from "@/app/components/Footer";
 import { AppSidebar } from "@/components/AppSidebar";
 import { Header } from "@/components/Header";
 import { SidebarProvider, SidebarTrigger } from "@/components/ui/sidebar";
@@ -58,7 +57,7 @@ const RootLayout = async ({
         {ANNOUNCEMENT && <Announcement>{ANNOUNCEMENT}</Announcement>}
         <SidebarProvider>
           <AppSidebar />
-          <main className="w-full">
+          <main className="w-full bg-gray-50">
             <Header breadcrumbs={breadcrumbs}>
               <SidebarTrigger />
             </Header>

diff --git a/components/AppSidebar.tsx b/components/AppSidebar.tsx
@@ -1,7 +1,7 @@
 "use client";
 
 import Link from "next/link";
-import { usePathname } from "next/navigation";
+import { usePathname, useSearchParams } from "next/navigation";
 import {
   type IconType,
   SiBluesky,
@@ -69,6 +69,8 @@ type Nav = {
 
 export const AppSidebar = () => {
   const pathname = usePathname();
+  const searchParams = useSearchParams();
+
   const { setOpenMobile } = useSidebar();
 
   return (
@@ -130,7 +132,12 @@ export const AppSidebar = () => {
                             isActive={subItem.url === pathname}
                             onClick={() => setOpenMobile(false)}
                           >
-                            <Link href={subItem.url}>
+                            <Link
+                              href={{
+                                pathname: subItem.url,
+                                query: searchParams.toString(),
+                              }}
+                            >
                               {subItem.icon && <subItem.icon />}
                               <span>{subItem.title}</span>
                             </Link>
@@ -252,7 +259,7 @@ const data: Nav = {
         const title = item.title as VehicleType;
         return {
           ...item,
-          title: VEHICLE_TYPE_MAP[title] || title,
+          title: VEHICLE_TYPE_MAP[title] ?? title,
           url: `/cars/vehicle-types/${slugify(title)}`,
         };
       }),

diff --git a/components/EmptyData.tsx b/components/EmptyData.tsx
@@ -10,7 +10,7 @@ export const EmptyData = () => {
   const pathname = usePathname();
 
   return (
-    <div className="flex flex-col items-center justify-center gap-4">
+    <div className="container flex min-h-screen flex-col items-center justify-center gap-4">
       <Alert variant="destructive">
         <AlertCircle className="h-4 w-4" />
         <AlertTitle>Error</AlertTitle>

diff --git a/config/index.ts b/config/index.ts
@@ -88,14 +88,14 @@ export const VEHICLE_TYPE_LINKS: LinkItem[] = [
     const label = link.label as VehicleType;
     return {
       ...link,
-      label: VEHICLE_TYPE_MAP[label] || label,
+      label: VEHICLE_TYPE_MAP[label] ?? label,
       href: `/cars/vehicle-types/${slugify(link.label)}`,
     };
   })
   .sort((a, b) => a.label.localeCompare(b.label));
 
 export const COE_LINKS: LinkItem[] = [
-  { href: "/coe", label: "Dashboard" },
+  { href: "/coe", label: "COE Result" },
   // { href: "/coe/prices", label: "COE Prices" },
   // { href: "/coe/bidding", label: "COE Bidding" },
 ];

diff --git a/constants/index.ts b/constants/index.ts
@@ -25,4 +25,5 @@ export const VEHICLE_TYPE_MAP: Partial<Record<VehicleType, string>> = {
   "Multi-purpose Vehicle": "MPV",
   "Multi-purpose Vehicle/Station-wagon": "MPV",
   "Sports Utility Vehicle": "SUV",
+  "Station-wagon": "Station wagon",
 };