fix: maintain same ui across all the pages

frontend/src/pages/Appointment.tsx

@@ -1,7 +1,7 @@
-import {useState, useEffect, useCallback} from "react";
+import { useState, useEffect, useCallback } from "react";
 
-import {getAppointmentsApi, deleteAppointmentApi} from "@/api/appointment";
-import {exportToExcel} from "@/utils/exportToExcel";
+import { getAppointmentsApi, deleteAppointmentApi } from "@/api/appointment";
+import { exportToExcel } from "@/utils/exportToExcel";
 
 import {
 	Table,
@@ -12,12 +12,26 @@ import {
 	TableRow,
 } from "@/components/ui/table";
 
-import {Card, CardContent, CardHeader, CardTitle} from "@/components/ui/card";
+import { Card, CardContent, CardHeader, CardTitle } from "@/components/ui/card";
+import { Button } from "@/components/ui/button";
+import { Input } from "@/components/ui/input";
+import {
+	Dialog,
+	DialogContent,
+	DialogHeader,
+	DialogTitle,
+	DialogFooter,
+} from "@/components/ui/dialog";
 
-import {Button} from "@/components/ui/button";
-import {Input} from "@/components/ui/input";
-
-import {Loader2, Trash, RefreshCw, Download} from "lucide-react";
+import {
+	Loader2,
+	Trash,
+	RefreshCw,
+	Download,
+	ChevronLeft,
+	ChevronRight,
+	Eye,
+} from "lucide-react";
 
 export default function AppointmentPage() {
 	const [appointments, setAppointments] = useState<any[]>([]);
@@ -25,9 +39,14 @@ export default function AppointmentPage() {
 
 	const [searchText, setSearchText] = useState("");
 	const [filterDoctor, setFilterDoctor] = useState("");
-	const [filterDepartment, setFilterDepartment] = useState("");
 	const [filterDate, setFilterDate] = useState("");
 
+	const [viewOpen, setViewOpen] = useState(false);
+	const [viewData, setViewData] = useState<any>(null);
+
+	const [currentPage, setCurrentPage] = useState(1);
+	const itemsPerPage = 10;
+
 	const fetchAll = useCallback(async () => {
 		setLoading(true);
 		try {
@@ -54,19 +73,30 @@ export default function AppointmentPage() {
 			? item.doctor?.name?.toLowerCase().includes(filterDoctor.toLowerCase())
 			: true;
 
-		const matchesDepartment = filterDepartment
-			? item.department?.name
-					?.toLowerCase()
-					.includes(filterDepartment.toLowerCase())
-			: true;
-
 		const matchesDate = filterDate
 			? new Date(item.date).toISOString().split("T")[0] === filterDate
 			: true;
 
-		return matchesSearch && matchesDoctor && matchesDepartment && matchesDate;
+		return matchesSearch && matchesDoctor && matchesDate;
 	});
 
+	useEffect(() => {
+		setCurrentPage(1);
+	}, [searchText, filterDoctor, filterDate]);
+
+	const totalPages = Math.ceil(filteredAppointments.length / itemsPerPage);
+	const indexOfLastItem = currentPage * itemsPerPage;
+	const indexOfFirstItem = indexOfLastItem - itemsPerPage;
+	const currentItems = filteredAppointments.slice(
+		indexOfFirstItem,
+		indexOfLastItem,
+	);
+
+	function openView(item: any) {
+		setViewData(item);
+		setViewOpen(true);
+	}
+
 	async function handleDelete(id: number) {
 		if (!confirm("Delete appointment?")) return;
 		await deleteAppointmentApi(id);
@@ -84,51 +114,41 @@ export default function AppointmentPage() {
 			Date: new Date(item.date).toLocaleDateString(),
 			Message: item.message,
 		}));
-
 		exportToExcel(exportData, "appointments");
 	};
 
 	return (
 		<div className="p-6 space-y-6">
-			<div className="flex justify-between items-center gap-3 flex-wrap">
-				<h1 className="text-2xl font-bold">Appointments</h1>
+			<div className="flex flex-col md:flex-row md:justify-between md:items-center gap-4">
+				<h1 className="text-3xl font-bold">Appointments</h1>
 
-				<div className="flex flex-wrap gap-2">
+				<div className="flex flex-wrap gap-3">
 					<Input
-						placeholder="Search name / phone / email..."
+						placeholder="Search name / phone..."
 						value={searchText}
 						onChange={(e) => setSearchText(e.target.value)}
-						className="w-[220px]"
-					/>
-
-					<Input
-						placeholder="Filter Doctor"
-						value={filterDoctor}
-						onChange={(e) => setFilterDoctor(e.target.value)}
-						className="w-[180px]"
-					/>
-
-					<Input
-						placeholder="Filter Department"
-						value={filterDepartment}
-						onChange={(e) => setFilterDepartment(e.target.value)}
-						className="w-[200px]"
+						className="w-[220px] text-base"
 					/>
 
 					<Input
 						type="date"
 						value={filterDate}
 						onChange={(e) => setFilterDate(e.target.value)}
-						className="w-[180px]"
+						className="w-[160px] text-base"
 					/>
 
-					<Button variant="outline" onClick={fetchAll} disabled={loading}>
-						<RefreshCw className="mr-2 h-4 w-4" />
+					<Button
+						variant="outline"
+						onClick={fetchAll}
+						disabled={loading}
+						className="text-base"
+					>
+						<RefreshCw className="mr-2 h-5 w-5" />
 						Refresh
 					</Button>
 
-					<Button variant="outline" onClick={handleExport}>
-						<Download className="mr-2 h-4 w-4" />
+					<Button onClick={handleExport} className="text-base">
+						<Download className="mr-2 h-5 w-5" />
 						Export
 					</Button>
 				</div>
@@ -136,72 +156,102 @@ export default function AppointmentPage() {
 
 			<Card>
 				<CardHeader>
-					<CardTitle>Appointment List</CardTitle>
+					<CardTitle className="text-xl">Appointment List</CardTitle>
 				</CardHeader>
 
-				<CardContent>
-					<div className="overflow-x-auto">
-						<Table className="min-w-[700px]">
-							<TableHeader>
+				<CardContent className="p-0 sm:p-6 space-y-4">
+					<div className="rounded-md border overflow-x-auto overflow-y-auto max-h-[650px] relative">
+						<Table className="w-full min-w-[1000px] table-fixed border-separate border-spacing-0">
+							<TableHeader className="sticky top-0 z-20 bg-background shadow-sm">
 								<TableRow>
-									<TableHead>ID</TableHead>
-									<TableHead>Name</TableHead>
-									<TableHead>Phone</TableHead>
-									<TableHead>Email</TableHead>
-									<TableHead>Doctor</TableHead>
-									<TableHead>Department</TableHead>
-									<TableHead>Appointment Date</TableHead>
-									<TableHead>Message</TableHead>
-									<TableHead>Generated on</TableHead>
-
-									<TableHead>Actions</TableHead>
+									<TableHead className="w-[60px] bg-background font-bold text-sm">
+										ID
+									</TableHead>
+									<TableHead className="w-[200px] bg-background font-bold text-sm">
+										Patient
+									</TableHead>
+									<TableHead className="w-[180px] bg-background font-bold text-sm">
+										Doctor
+									</TableHead>
+									<TableHead className="w-[150px] bg-background font-bold text-sm">
+										Date
+									</TableHead>
+									<TableHead className="w-[250px] bg-background font-bold text-sm">
+										Message
+									</TableHead>
+									<TableHead className="w-[120px] bg-background font-bold text-right text-sm">
+										Actions
+									</TableHead>
 								</TableRow>
 							</TableHeader>
 
 							<TableBody>
 								{loading ? (
 									<TableRow>
-										<TableCell colSpan={9} className="text-center">
-											<Loader2 className="h-6 w-6 animate-spin mx-auto" />
+										<TableCell colSpan={6} className="text-center py-10">
+											<Loader2 className="h-8 w-8 animate-spin mx-auto" />
 										</TableCell>
 									</TableRow>
-								) : filteredAppointments.length === 0 ? (
+								) : currentItems.length === 0 ? (
 									<TableRow>
-										<TableCell colSpan={9} className="text-center">
+										<TableCell
+											colSpan={6}
+											className="text-center text-muted-foreground py-10 text-base"
+										>
 											No appointments found
 										</TableCell>
 									</TableRow>
 								) : (
-									filteredAppointments.map((item) => (
-										<TableRow key={item.id}>
-											<TableCell>{item.id}</TableCell>
-											<TableCell>{item.name}</TableCell>
-											<TableCell>{item.mobileNumber}</TableCell>
-											<TableCell>{item.email}</TableCell>
-											<TableCell>{item.doctor?.name}</TableCell>
-											<TableCell>{item.department?.name}</TableCell>
-
-											{/* ✅ DATE ONLY */}
-											<TableCell>
-												{new Date(item.date).toLocaleDateString()}
-											</TableCell>
-
-											<TableCell className="max-w-[250px] whitespace-normal">
-												{item.message}
+									currentItems.map((item) => (
+										<TableRow key={item.id} className="hover:bg-muted/50">
+											<TableCell className="font-mono text-xs">
+												{item.id}
 											</TableCell>
 											<TableCell>
-												{" "}
-												{new Date(item.createdAt).toLocaleDateString()}
+												<div className="font-semibold text-base truncate">
+													{item.name}
+												</div>
+												<div className="text-xs text-muted-foreground">
+													{item.mobileNumber}
+												</div>
 											</TableCell>
-
 											<TableCell>
-												<Button
-													size="sm"
-													variant="destructive"
-													onClick={() => handleDelete(item.id)}
-												>
-													<Trash className="h-4 w-4" />
-												</Button>
+												<div className="text-sm font-medium">
+													{item.doctor?.name || "-"}
+												</div>
+												<div className="text-[10px] text-muted-foreground truncate">
+													{item.department?.name}
+												</div>
+											</TableCell>
+											<TableCell>
+												<div className="text-sm">
+													{new Date(item.date).toLocaleDateString()}
+												</div>
+											</TableCell>
+											<TableCell>
+												<div className="text-sm line-clamp-2 text-muted-foreground italic">
+													{item.message || "-"}
+												</div>
+											</TableCell>
+											<TableCell className="text-right">
+												<div className="flex justify-end gap-2">
+													<Button
+														size="icon"
+														variant="ghost"
+														className="h-9 w-9"
+														onClick={() => openView(item)}
+													>
+														<Eye className="h-4 w-4" />
+													</Button>
+													<Button
+														size="icon"
+														variant="ghost"
+														className="h-9 w-9 text-destructive hover:text-destructive hover:bg-destructive/10"
+														onClick={() => handleDelete(item.id)}
+													>
+														<Trash className="h-4 w-4" />
+													</Button>
+												</div>
 											</TableCell>
 										</TableRow>
 									))
@@ -209,8 +259,123 @@ export default function AppointmentPage() {
 							</TableBody>
 						</Table>
 					</div>
+
+					{!loading && filteredAppointments.length > 0 && (
+						<div className="flex items-center justify-between px-2 py-6 border-t">
+							<div className="text-base text-muted-foreground">
+								Showing{" "}
+								<span className="font-semibold">{indexOfFirstItem + 1}</span> to{" "}
+								<span className="font-semibold">
+									{Math.min(indexOfLastItem, filteredAppointments.length)}
+								</span>{" "}
+								of{" "}
+								<span className="font-semibold">
+									{filteredAppointments.length}
+								</span>
+							</div>
+							<div className="flex items-center gap-6">
+								<div className="text-base font-semibold">
+									Page {currentPage} of {totalPages}
+								</div>
+								<div className="flex gap-2">
+									<Button
+										variant="outline"
+										size="icon"
+										className="h-10 w-10"
+										onClick={() =>
+											setCurrentPage((prev) => Math.max(prev - 1, 1))
+										}
+										disabled={currentPage === 1}
+									>
+										<ChevronLeft className="h-5 w-5" />
+									</Button>
+									<Button
+										variant="outline"
+										size="icon"
+										className="h-10 w-10"
+										onClick={() =>
+											setCurrentPage((prev) => Math.min(prev + 1, totalPages))
+										}
+										disabled={currentPage === totalPages || totalPages === 0}
+									>
+										<ChevronRight className="h-5 w-5" />
+									</Button>
+								</div>
+							</div>
+						</div>
+					)}
 				</CardContent>
 			</Card>
+
+			<Dialog open={viewOpen} onOpenChange={setViewOpen}>
+				<DialogContent className="w-full !max-w-3xl max-h-[85vh] overflow-y-auto">
+					<DialogHeader>
+						<DialogTitle className="text-2xl border-b pb-2">
+							Appointment Details
+						</DialogTitle>
+					</DialogHeader>
+					{viewData && (
+						<div className="space-y-6 py-4">
+							<div className="grid grid-cols-1 md:grid-cols-2 gap-6">
+								<div className="space-y-4">
+									<div>
+										<p className="text-xs uppercase font-bold text-muted-foreground">
+											Patient Information
+										</p>
+										<p className="text-lg font-bold text-primary">
+											{viewData.name}
+										</p>
+										<p className="text-sm">{viewData.mobileNumber}</p>
+										<p className="text-sm">
+											{viewData.email || "No email provided"}
+										</p>
+									</div>
+									<div>
+										<p className="text-xs uppercase font-bold text-muted-foreground">
+											Appointment Date
+										</p>
+										<p className="text-base font-semibold">
+											{new Date(viewData.date).toLocaleDateString()}
+										</p>
+										<p className="text-[10px] text-muted-foreground">
+											Booked on: {new Date(viewData.createdAt).toLocaleString()}
+										</p>
+									</div>
+								</div>
+								<div className="space-y-4">
+									<div>
+										<p className="text-xs uppercase font-bold text-muted-foreground">
+											Doctor / Department
+										</p>
+										<p className="text-base font-bold">
+											{viewData.doctor?.name || "Not Assigned"}
+										</p>
+										<p className="text-sm text-muted-foreground">
+											{viewData.department?.name || "General"}
+										</p>
+									</div>
+									<div className="p-4 bg-muted/30 rounded-lg">
+										<p className="text-xs uppercase font-bold text-muted-foreground mb-2">
+											Message from Patient
+										</p>
+										<p className="text-sm italic leading-relaxed whitespace-pre-wrap">
+											{viewData.message || "No message provided."}
+										</p>
+									</div>
+								</div>
+							</div>
+						</div>
+					)}
+					<DialogFooter>
+						<Button
+							onClick={() => setViewOpen(false)}
+							className="w-full md:w-auto"
+						>
+							Close
+						</Button>
+					</DialogFooter>
+				</DialogContent>
+			</Dialog>
 		</div>
 	);
 }