fix(detailRoute): improve navigation logic and debug messages for unsaved changes

Author: Pavel401

lib/app/modules/detailRoute/views/detail_route_view.dart

@@ -28,11 +28,15 @@ class DetailRouteView extends GetView<DetailRouteController> {
     return WillPopScope(
       onWillPop: () async {
         if (!controller.onEdit.value) {
+          debugPrint(
+              'DetailRouteView: No edits made, navigating back without prompt.');
           // Get.offAll(() => const HomeView());
-          Get.back();
+          Navigator.of(context).pop();
           // Get.toNamed(Routes.HOME);
           return false;
         }
+        debugPrint(
+            'DetailRouteView: Unsaved edits detected, prompting user for action.');
 
         bool? save = await showDialog(
           context: context,
@@ -50,10 +54,14 @@ class DetailRouteView extends GetView<DetailRouteController> {
               actions: [
                 TextButton(
                   onPressed: () {
-                    controller.saveChanges();
-                    // Get.offAll(() => const HomeView());
+                    // Get.back(); // Close the dialog first
+                    // // Wait for dialog to fully close before showing snackbar
+                    // Future.delayed(const Duration(milliseconds: 100), () {
+                    //   controller.saveChanges();
+                    // });
 
-                    Get.back();
+                    controller.saveChanges();
+                    Navigator.of(context).pop(true);
                   },
                   child: Text(
                     SentenceManager(
@@ -69,7 +77,7 @@ class DetailRouteView extends GetView<DetailRouteController> {
                   onPressed: () {
                     // Get.offAll(() => const HomeView());
 
-                    Get.back();
+                    Navigator.of(context).pop();
                   },
                   child: Text(
                     SentenceManager(
@@ -83,7 +91,7 @@ class DetailRouteView extends GetView<DetailRouteController> {
                 ),
                 TextButton(
                   onPressed: () {
-                    Get.back();
+                    Navigator.of(context).pop(false);
                   },
                   child: Text(
                     SentenceManager(
@@ -251,7 +259,7 @@ class AttributeWidget extends StatelessWidget {
         Theme.of(context).extension<TaskwarriorColorTheme>()!;
     // Get the controller to check if the task is read-only
     final DetailRouteController controller = Get.find<DetailRouteController>();
-    
+
     // Always allow status to be edited, but respect read-only for other attributes
     final bool isEditable = !controller.isReadOnly.value || name == 'status';
 
@@ -323,9 +331,10 @@ class AttributeWidget extends StatelessWidget {
           isEditable: isEditable,
         );
       default:
-        final Color? textColor = (isEditable && !['entry', 'modified', 'urgency'].contains(name))
-            ? tColors.primaryTextColor
-            : tColors.primaryDisabledTextColor;
+        final Color? textColor =
+            (isEditable && !['entry', 'modified', 'urgency'].contains(name))
+                ? tColors.primaryTextColor
+                : tColors.primaryDisabledTextColor;
 
         return Card(
           color: tColors.secondaryBackgroundColor,

lib/app/modules/home/views/tasks_builder.dart

@@ -53,7 +53,8 @@ class TasksBuilder extends StatelessWidget {
 
   void saveChanges(
       BuildContext context, Modify modify, String id, String newValue) async {
-    TaskwarriorColorTheme tColors = Theme.of(context).extension<TaskwarriorColorTheme>()!;
+    TaskwarriorColorTheme tColors =
+        Theme.of(context).extension<TaskwarriorColorTheme>()!;
     var now = DateTime.now().toUtc();
     modify.save(
       modified: () => now,
@@ -110,32 +111,37 @@ class TasksBuilder extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     // print(taskData);
-    TaskwarriorColorTheme tColors = Theme.of(context).extension<TaskwarriorColorTheme>()!;
+    TaskwarriorColorTheme tColors =
+        Theme.of(context).extension<TaskwarriorColorTheme>()!;
     var storageWidget = Get.find<HomeController>();
     return Scaffold(
         floatingActionButtonLocation:
             FloatingActionButtonLocation.miniStartFloat,
-        floatingActionButton: showbtn ? AnimatedOpacity(
-          duration: const Duration(milliseconds: 100), //show/hide animation
-          opacity: showbtn ? 1.0 : 0.0, //set obacity to 1 on visible, or hide
-          child: FloatingActionButton(
-            heroTag: "btn2",
-            onPressed: () {
-              scrollController.animateTo(
-                  //go to top of scroll
-                  0, //scroll offset to go
-                  duration:
-                      const Duration(milliseconds: 500), //duration of scroll
-                  curve: Curves.fastLinearToSlowEaseIn //scroll type
-                  );
-            },
-            backgroundColor: tColors.primaryTextColor,
-            child: Icon(
-              Icons.arrow_upward,
-              color: tColors.secondaryBackgroundColor,
-            ),
-          ),
-        ) : null,
+        floatingActionButton: showbtn
+            ? AnimatedOpacity(
+                duration:
+                    const Duration(milliseconds: 100), //show/hide animation
+                opacity:
+                    showbtn ? 1.0 : 0.0, //set obacity to 1 on visible, or hide
+                child: FloatingActionButton(
+                  heroTag: "btn2",
+                  onPressed: () {
+                    scrollController.animateTo(
+                        //go to top of scroll
+                        0, //scroll offset to go
+                        duration: const Duration(
+                            milliseconds: 500), //duration of scroll
+                        curve: Curves.fastLinearToSlowEaseIn //scroll type
+                        );
+                  },
+                  backgroundColor: tColors.primaryTextColor,
+                  child: Icon(
+                    Icons.arrow_upward,
+                    color: tColors.secondaryBackgroundColor,
+                  ),
+                ),
+              )
+            : null,
         backgroundColor: Colors.transparent,
         body: Obx(
           () => taskData.isEmpty
@@ -152,8 +158,7 @@ class TasksBuilder extends StatelessWidget {
                       style: TextStyle(
                           fontFamily: FontFamily.poppins,
                           fontSize: TaskWarriorFonts.fontSizeLarge,
-                          color: tColors.primaryTextColor
-                      ),
+                          color: tColors.primaryTextColor),
                       // style: GoogleFonts.poppins(
                       //   fontSize: TaskWarriorFonts.fontSizeLarge,
                       //   color: AppSettings.isDarkMode
@@ -193,7 +198,8 @@ class TasksBuilder extends StatelessWidget {
                                     }
                                   },
                                   icon: Icons.done,
-                                  label: SentenceManager(currentLanguage: selectedLanguage)
+                                  label: SentenceManager(
+                                          currentLanguage: selectedLanguage)
                                       .sentences
                                       .complete,
                                   backgroundColor: TaskWarriorColors.green,
@@ -222,7 +228,8 @@ class TasksBuilder extends StatelessWidget {
                                     }
                                   },
                                   icon: Icons.delete,
-                                  label: SentenceManager(currentLanguage: selectedLanguage)
+                                  label: SentenceManager(
+                                          currentLanguage: selectedLanguage)
                                       .sentences
                                       .delete,
                                   backgroundColor: TaskWarriorColors.red,