From 616cfabd70a5cc875ee2859ea26f1e26d1bddfaa Mon Sep 17 00:00:00 2001
From: THIS ONE IS A LITTLE BIT TRICKY KRUB <pattadon2546pn@gmail.com>
Date: Fri, 25 Apr 2025 18:54:45 +0700
Subject: [PATCH] feat: enhance button accessibility by adding cursor-pointer
 class and type attributes in CreatePipelinePage and AddDataSource component

---
 .../(sidebar)/data-pipeline/create/page.tsx   |  2 +-
 frontend/app/(sidebar)/data-pipeline/page.tsx |  2 +-
 .../components/pipeline/add-data-source.tsx   | 25 ++++++++++++++-----
 3 files changed, 21 insertions(+), 8 deletions(-)

diff --git a/frontend/app/(sidebar)/data-pipeline/create/page.tsx b/frontend/app/(sidebar)/data-pipeline/create/page.tsx
index 7f20f31..a191786 100644
--- a/frontend/app/(sidebar)/data-pipeline/create/page.tsx
+++ b/frontend/app/(sidebar)/data-pipeline/create/page.tsx
@@ -8,7 +8,7 @@ export default function CreatePipelinePage() {
     <div className="container mx-auto p-6">
       <div className="mt-6">
         <Link href="/data-pipeline">
-          <Button variant="outline" className="mb-6">
+          <Button variant="outline" className="mb-6 cursor-pointer">
             <ArrowLeft className="mr-2 h-4 w-4" />
             Back to Pipelines
           </Button>
diff --git a/frontend/app/(sidebar)/data-pipeline/page.tsx b/frontend/app/(sidebar)/data-pipeline/page.tsx
index 1e36353..a5bf45d 100644
--- a/frontend/app/(sidebar)/data-pipeline/page.tsx
+++ b/frontend/app/(sidebar)/data-pipeline/page.tsx
@@ -48,7 +48,7 @@ export default function DataPipelinePage() {
 
           <div className="flex justify-end mt-4">
             <Link href="/data-pipeline/create">
-              <Button className="gap-2">
+              <Button className="gap-2 cursor-pointer">
                 <Plus className="h-4 w-4" />
                 Create Pipeline
               </Button>
diff --git a/frontend/components/pipeline/add-data-source.tsx b/frontend/components/pipeline/add-data-source.tsx
index cc655cb..90ea018 100644
--- a/frontend/components/pipeline/add-data-source.tsx
+++ b/frontend/components/pipeline/add-data-source.tsx
@@ -78,6 +78,7 @@ export function AddDataSource() {
                     variant="outline"
                     size="sm"
                     className="text-destructive"
+                    type="button"
                   >
                     <Trash2 className="mr-2 h-4 w-4" />
                     Remove Source
@@ -106,9 +107,7 @@ export function AddDataSource() {
                       <p className="text-sm text-muted-foreground">
                         Drag and drop your file here, or click to browse
                       </p>
-                      <Button variant="outline" className="mt-2">
-                        Select File
-                      </Button>
+                      <Input type="file" className="mt-2 cursor-pointer" />
                     </div>
                   </div>
                   <p className="text-xs text-muted-foreground mt-1">
@@ -121,6 +120,7 @@ export function AddDataSource() {
                     variant="outline"
                     size="sm"
                     className="text-destructive"
+                    type="button"
                   >
                     <Trash2 className="mr-2 h-4 w-4" />
                     Remove Source
@@ -167,6 +167,7 @@ export function AddDataSource() {
                   <Button
                     variant="outline"
                     size="sm"
+                    type="button"
                     className="text-destructive"
                   >
                     <Trash2 className="mr-2 h-4 w-4" />
@@ -179,15 +180,27 @@ export function AddDataSource() {
         </Accordion>
 
         <div className="flex flex-col gap-2 mt-4">
-          <Button variant="outline" className="w-full justify-start gap-2">
+          <Button
+            variant="outline"
+            type="button"
+            className="w-full justify-start gap-2"
+          >
             <Plus className="h-4 w-4" />
             Add Website Source
           </Button>
-          <Button variant="outline" className="w-full justify-start gap-2">
+          <Button
+            variant="outline"
+            type="button"
+            className="w-full justify-start gap-2"
+          >
             <Plus className="h-4 w-4" />
             Add File Upload Source
           </Button>
-          <Button variant="outline" className="w-full justify-start gap-2">
+          <Button
+            variant="outline"
+            type="button"
+            className="w-full justify-start gap-2"
+          >
             <Plus className="h-4 w-4" />
             Add API Source
           </Button>