From 5130ced36ed36d3ec37c012ba013a8c2faae4080 Mon Sep 17 00:00:00 2001
From: Liangdi <ligsnf@gmail.com>
Date: Wed, 25 Dec 2024 13:02:45 +1100
Subject: [PATCH] refactor: improve UI clarity and consistency - Remove
 redundant CSV upload alert - Add descriptive text to CSV information button -
 Update delete all button styling

---
 src/components/csv/csv-information-dialog.tsx | 3 ++-
 src/components/results/delete-all-dialog.tsx  | 4 ++--
 src/routes/index.lazy.tsx                     | 6 ++----
 3 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/src/components/csv/csv-information-dialog.tsx b/src/components/csv/csv-information-dialog.tsx
index 64cdb3b..c4da241 100644
--- a/src/components/csv/csv-information-dialog.tsx
+++ b/src/components/csv/csv-information-dialog.tsx
@@ -15,8 +15,9 @@ export function CSVInformationDialog() {
   return (
     <Dialog>
       <DialogTrigger asChild>
-        <Button variant="outline" size="icon" className="min-w-10">
+        <Button variant="outline" className="min-w-10">
           <Info className="text-primary" strokeWidth={2.5} />
+          <span className="hidden sm:inline">Import Results Guide</span>
         </Button>
       </DialogTrigger>
       <DialogContent className="max-h-[80vh]">
diff --git a/src/components/results/delete-all-dialog.tsx b/src/components/results/delete-all-dialog.tsx
index 7334847..7b699e7 100644
--- a/src/components/results/delete-all-dialog.tsx
+++ b/src/components/results/delete-all-dialog.tsx
@@ -22,8 +22,8 @@ export function DeleteAllDialog({ onConfirm }: DeleteAllDialogProps) {
       <AlertDialogTrigger asChild>
         <Button
           type="button" 
-          variant="destructive"
-          className="opacity-80 h-8 px-2 md:h-10 md:px-3"
+          variant="outline"
+          className="border-destructive text-destructive hover:text-destructive h-8 px-2 md:h-10 md:px-3"
         >
           <X strokeWidth={3} />
         </Button>
diff --git a/src/routes/index.lazy.tsx b/src/routes/index.lazy.tsx
index 85c6c01..6777a05 100644
--- a/src/routes/index.lazy.tsx
+++ b/src/routes/index.lazy.tsx
@@ -8,9 +8,8 @@ import { calculateWAM, calculateGPA, calculateTotalCredits } from "@/lib/calcula
 import { ProcessingResult } from '@/lib/csv-parser'
 
 import { StatCard } from "@/components/stat-card"
-import { CSVUploadAlert } from "@/components/csv/csv-upload-alert"
-import { CSVUploadDialog } from '@/components/csv/csv-upload-dialog'
 import { CSVInformationDialog } from '@/components/csv/csv-information-dialog'
+import { CSVUploadDialog } from '@/components/csv/csv-upload-dialog'
 import { ResultTable } from "@/components/results/result-table"
 import { Button } from "@/components/ui/button"
 import {
@@ -159,13 +158,12 @@ function Index() {
                 <span className="font-mono font-semibold text-lg sm:text-xl">{totalCredits}</span>
               </div>
               <div className="flex items-center gap-2">
-                <CSVUploadAlert className="hidden md:flex" />
+                <CSVInformationDialog />
                 <CSVUploadDialog 
                   onCSVUpload={handleCSVUpload}
                   open={uploadDialogOpen}
                   onOpenChange={setUploadDialogOpen}
                 />
-                <CSVInformationDialog />
               </div>
             </div>
           </CardHeader>