removed the extra boz coming for the dashboard chart

lib/view/dashboard/dashboard_chart.dart

@@ -71,19 +71,8 @@ class AttendanceDashboardChart extends StatelessWidget {
     final isLoading = controller.isLoading.value;
 
     return Container(
-        decoration: const BoxDecoration(
+      // flat white background
-          gradient: LinearGradient(
-            colors: [Color(0xfff0f4f8), Color(0xffe2ebf0)],
-            begin: Alignment.topLeft,
-            end: Alignment.bottomRight,
-          ),
-        ),
-        child: Card(
       color: Colors.white,
-          elevation: 6,
-          shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(16)),
-          shadowColor: Colors.black12,
-          child: Padding(
       padding: const EdgeInsets.all(10),
       child: Column(
         crossAxisAlignment: CrossAxisAlignment.start,
@@ -102,8 +91,6 @@ class AttendanceDashboardChart extends StatelessWidget {
           ),
         ],
       ),
-          ),
-        ),
     );
   });
 }

lib/view/dashboard/dashboard_screen.dart

@@ -5,7 +5,6 @@ import 'package:marco/controller/dashboard/dashboard_controller.dart';
 import 'package:marco/controller/project_controller.dart';
 import 'package:marco/helpers/services/storage/local_storage.dart';
 import 'package:marco/helpers/utils/mixins/ui_mixin.dart';
-import 'package:marco/helpers/utils/my_shadow.dart';
 import 'package:marco/helpers/widgets/my_card.dart';
 import 'package:marco/helpers/widgets/my_container.dart';
 import 'package:marco/helpers/widgets/my_spacing.dart';
@@ -26,14 +25,12 @@ class DashboardScreen extends StatefulWidget {
   static const String directoryMainPageRoute = "/dashboard/directory-main-page";
   static const String expenseMainPageRoute = "/dashboard/expense-main-page";
 
-
   @override
   State<DashboardScreen> createState() => _DashboardScreenState();
 }
 
 class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
-  final DashboardController dashboardController =
+  final DashboardController dashboardController = Get.put(DashboardController());
-      Get.put(DashboardController());
   bool hasMpin = true;
 
   @override
@@ -43,10 +40,8 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
   }
 
   Future<void> _checkMpinStatus() async {
-    final bool mpinStatus = await LocalStorage.getIsMpin();
+    hasMpin = await LocalStorage.getIsMpin();
-    setState(() {
+    if (mounted) setState(() {});
-      hasMpin = mpinStatus;
-    });
   }
 
   @override
@@ -57,79 +52,88 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
         child: Column(
           crossAxisAlignment: CrossAxisAlignment.start,
           children: [
-            _buildDashboardStats(),
+            _buildDashboardStats(context),
             MySpacing.height(24),
-            GetBuilder<ProjectController>(
+            _buildAttendanceChartSection(),
-              id: 'dashboard_controller',
-              builder: (projectController) {
-                final bool isProjectSelected =
-                    projectController.selectedProject != null;
-
-                return Opacity(
-                  opacity: isProjectSelected ? 1.0 : 0.4,
-                  child: IgnorePointer(
-                    ignoring: !isProjectSelected,
-                    child: Column(
-                      crossAxisAlignment: CrossAxisAlignment.start,
-                      children: [
-                        AttendanceDashboardChart(),
-                      ],
-                    ),
-                  ),
-                );
-              },
-            ),
           ],
         ),
       ),
     );
   }
 
-  Widget _buildDashboardStats() {
+  /// Dashboard Statistics Section with ProjectController
+  Widget _buildDashboardStats(BuildContext context) {
     final stats = [
-      _StatItem(LucideIcons.scan_face, "Attendance", contentTheme.success,
+      _StatItem(LucideIcons.scan_face, "Attendance", contentTheme.success, DashboardScreen.attendanceRoute),
-          DashboardScreen.attendanceRoute),
+      _StatItem(LucideIcons.users, "Employees", contentTheme.warning, DashboardScreen.employeesRoute),
-      _StatItem(LucideIcons.users, "Employees", contentTheme.warning,
+      _StatItem(LucideIcons.logs, "Daily Task Planing", contentTheme.info, DashboardScreen.dailyTasksRoute),
-          DashboardScreen.employeesRoute),
+      _StatItem(LucideIcons.list_todo, "Daily Task Progress", contentTheme.info, DashboardScreen.dailyTasksProgressRoute),
-      _StatItem(LucideIcons.logs, "Daily Task Planing", contentTheme.info,
+      _StatItem(LucideIcons.folder, "Directory", contentTheme.info, DashboardScreen.directoryMainPageRoute),
-          DashboardScreen.dailyTasksRoute),
+      _StatItem(LucideIcons.badge_dollar_sign, "Expense", contentTheme.info, DashboardScreen.expenseMainPageRoute),
-      _StatItem(LucideIcons.list_todo, "Daily Task Progress", contentTheme.info,
-          DashboardScreen.dailyTasksProgressRoute),
-      _StatItem(LucideIcons.folder, "Directory", contentTheme.info,
-          DashboardScreen.directoryMainPageRoute),
-      _StatItem(LucideIcons.badge_dollar_sign, "Expense", contentTheme.info,
-          DashboardScreen.expenseMainPageRoute),
     ];
 
     return GetBuilder<ProjectController>(
       id: 'dashboard_controller',
       builder: (controller) {
-        final bool isLoading = controller.isLoading.value;
+        if (controller.isLoading.value) {
-        final bool isProjectSelected = controller.selectedProject != null;
+          return _buildLoadingSkeleton(context);
-
-        if (isLoading) {
-          return Wrap(
-            spacing: 10,
-            runSpacing: 10,
-            children: List.generate(
-              4,
-              (index) =>
-                  _buildStatCardSkeleton(MediaQuery.of(context).size.width / 3),
-            ),
-          );
         }
 
+        final isProjectSelected = controller.selectedProject != null;
+
         return Column(
           crossAxisAlignment: CrossAxisAlignment.start,
           children: [
-            if (!isProjectSelected)
+            if (!isProjectSelected) _buildNoProjectMessage(),
-              Padding(
+            LayoutBuilder(
+              builder: (context, constraints) {
+                final maxWidth = constraints.maxWidth;
+                final crossAxisCount = (maxWidth / 100).floor().clamp(2, 4);
+                final cardWidth = (maxWidth - (crossAxisCount - 1) * 10) / crossAxisCount;
+
+                return Wrap(
+                  spacing: 10,
+                  runSpacing: 10,
+                  children: stats
+                      .map((stat) => _buildStatCard(stat, cardWidth, isProjectSelected))
+                      .toList(),
+                );
+              },
+            ),
+          ],
+        );
+      },
+    );
+  }
+
+  /// Attendance Chart Section
+  Widget _buildAttendanceChartSection() {
+    return GetBuilder<ProjectController>(
+      id: 'dashboard_controller',
+      builder: (projectController) {
+        final isProjectSelected = projectController.selectedProject != null;
+        return Opacity(
+          opacity: isProjectSelected ? 1.0 : 0.4,
+          child: IgnorePointer(
+            ignoring: !isProjectSelected,
+            child: ClipRRect(
+              borderRadius: BorderRadius.circular(12),
+              child:  AttendanceDashboardChart(),
+            ),
+          ),
+        );
+      },
+    );
+  }
+
+  /// No Project Assigned Message
+  Widget _buildNoProjectMessage() {
+    return Padding(
       padding: const EdgeInsets.symmetric(vertical: 12),
       child: MyCard(
         color: Colors.orange.withOpacity(0.1),
         paddingAll: 12,
         child: Row(
-                    crossAxisAlignment: CrossAxisAlignment.start,
           children: [
             const Icon(Icons.info_outline, color: Colors.orange),
             MySpacing.width(8),
@@ -144,30 +148,22 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
           ],
         ),
       ),
-              ),
-            LayoutBuilder(
-              builder: (context, constraints) {
-                double maxWidth = constraints.maxWidth;
-                int crossAxisCount = (maxWidth / 100).floor().clamp(2, 4);
-                double cardWidth =
-                    (maxWidth - (crossAxisCount - 1) * 10) / crossAxisCount;
-
-                return Wrap(
-                  spacing: 10,
-                  runSpacing: 10,
-                  children: stats
-                      .map((stat) =>
-                          _buildStatCard(stat, cardWidth, isProjectSelected))
-                      .toList(),
-                );
-              },
-            ),
-          ],
-        );
-      },
     );
   }
 
+  /// Loading Skeletons
+  Widget _buildLoadingSkeleton(BuildContext context) {
+    return Wrap(
+      spacing: 10,
+      runSpacing: 10,
+      children: List.generate(
+        4,
+        (index) => _buildStatCardSkeleton(MediaQuery.of(context).size.width / 3),
+      ),
+    );
+  }
+
+  /// Skeleton Card
   Widget _buildStatCardSkeleton(double width) {
     return MyCard.bordered(
       width: width,
@@ -175,7 +171,6 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
       paddingAll: 5,
       borderRadiusAll: 10,
       border: Border.all(color: Colors.grey.withOpacity(0.15)),
-      shadow: MyShadow(elevation: 1.5, position: MyShadowPosition.bottom),
       child: Column(
         mainAxisAlignment: MainAxisAlignment.center,
         children: [
@@ -195,27 +190,14 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
     );
   }
 
+  /// Stat Card
   Widget _buildStatCard(_StatItem statItem, double width, bool isEnabled) {
     return Opacity(
       opacity: isEnabled ? 1.0 : 0.4,
       child: IgnorePointer(
         ignoring: !isEnabled,
         child: InkWell(
-          onTap: () {
+          onTap: () => _handleStatCardTap(statItem, isEnabled),
-            if (!isEnabled) {
-              Get.defaultDialog(
-                title: "No Project Selected",
-                middleText:
-                    "You need to select a project before accessing this section.",
-                confirm: ElevatedButton(
-                  onPressed: () => Get.back(),
-                  child: const Text("OK"),
-                ),
-              );
-            } else {
-              Get.toNamed(statItem.route);
-            }
-          },
           borderRadius: BorderRadius.circular(10),
           child: MyCard.bordered(
             width: width,
@@ -223,7 +205,6 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
             paddingAll: 5,
             borderRadiusAll: 10,
             border: Border.all(color: Colors.grey.withOpacity(0.15)),
-            shadow: MyShadow(elevation: 1.5, position: MyShadowPosition.bottom),
             child: Column(
               mainAxisAlignment: MainAxisAlignment.center,
               children: [
@@ -243,6 +224,23 @@ class _DashboardScreenState extends State<DashboardScreen> with UIMixin {
     );
   }
 
+  /// Handle Tap
+  void _handleStatCardTap(_StatItem statItem, bool isEnabled) {
+    if (!isEnabled) {
+      Get.defaultDialog(
+        title: "No Project Selected",
+        middleText: "You need to select a project before accessing this section.",
+        confirm: ElevatedButton(
+          onPressed: () => Get.back(),
+          child: const Text("OK"),
+        ),
+      );
+    } else {
+      Get.toNamed(statItem.route);
+    }
+  }
+
+  /// Stat Icon
   Widget _buildStatCardIcon(_StatItem statItem) {
     return MyContainer.rounded(
       paddingAll: 10,