From 4d22daefa722a4931ca39373b8759fa7451cbc27 Mon Sep 17 00:00:00 2001
From: balibabu <cike8899@users.noreply.github.com>
Date: Mon, 24 Feb 2025 13:50:21 +0800
Subject: [PATCH] Feat: Add PageHeader to DatasetWrapper #3221 (#5284)

### What problem does this PR solve?

Feat: Add PageHeader to DatasetWrapper #3221

### Type of change


- [x] New Feature (non-breaking change which adds functionality)
---
 web/src/pages/dataset/index.tsx | 19 ++++++++++++++-----
 web/src/routes.ts               | 33 ++++++++++++++++-----------------
 2 files changed, 30 insertions(+), 22 deletions(-)

diff --git a/web/src/pages/dataset/index.tsx b/web/src/pages/dataset/index.tsx
index d1f65c7e..e6b82b4f 100644
--- a/web/src/pages/dataset/index.tsx
+++ b/web/src/pages/dataset/index.tsx
@@ -1,13 +1,22 @@
+import { PageHeader } from '@/components/page-header';
+import { useNavigatePage } from '@/hooks/logic-hooks/navigate-hooks';
 import { Outlet } from 'umi';
 import { SideBar } from './sidebar';
 
 export default function DatasetWrapper() {
+  const { navigateToDatasetList } = useNavigatePage();
   return (
-    <div className="flex flex-1">
-      <SideBar></SideBar>
-      <div className="flex-1">
-        <Outlet />
+    <section>
+      <PageHeader
+        title="Dataset details"
+        back={navigateToDatasetList}
+      ></PageHeader>
+      <div className="flex flex-1">
+        <SideBar></SideBar>
+        <div className="flex-1">
+          <Outlet />
+        </div>
       </div>
-    </div>
+    </section>
   );
 }
diff --git a/web/src/routes.ts b/web/src/routes.ts
index c7376e7b..0f70a21c 100644
--- a/web/src/routes.ts
+++ b/web/src/routes.ts
@@ -239,25 +239,24 @@ const routes = [
     path: Routes.DatasetBase,
     layout: false,
     component: '@/layouts/next',
+    routes: [{ path: Routes.DatasetBase, redirect: Routes.Dataset }],
+  },
+  {
+    path: Routes.DatasetBase,
+    layout: false,
+    component: `@/pages${Routes.DatasetBase}`,
     routes: [
-      { path: Routes.DatasetBase, redirect: Routes.Dataset },
       {
-        path: Routes.DatasetBase,
-        component: `@/pages${Routes.DatasetBase}`,
-        routes: [
-          {
-            path: `${Routes.Dataset}/:id`,
-            component: `@/pages${Routes.Dataset}`,
-          },
-          {
-            path: `${Routes.DatasetBase}${Routes.DatasetSetting}/:id`,
-            component: `@/pages${Routes.DatasetBase}${Routes.DatasetSetting}`,
-          },
-          {
-            path: `${Routes.DatasetBase}${Routes.DatasetTesting}/:id`,
-            component: `@/pages${Routes.DatasetBase}${Routes.DatasetTesting}`,
-          },
-        ],
+        path: `${Routes.Dataset}/:id`,
+        component: `@/pages${Routes.Dataset}`,
+      },
+      {
+        path: `${Routes.DatasetBase}${Routes.DatasetSetting}/:id`,
+        component: `@/pages${Routes.DatasetBase}${Routes.DatasetSetting}`,
+      },
+      {
+        path: `${Routes.DatasetBase}${Routes.DatasetTesting}/:id`,
+        component: `@/pages${Routes.DatasetBase}${Routes.DatasetTesting}`,
       },
     ],
   },