Merge pull request #70 from CodandoTV/fix/weird-scroll-behavior

Refactor TasksListWidget and update version

Author: gabrielbmoro

lib/ui/components/widgets/checklist/checklist_full_widget.dart

@@ -5,7 +5,7 @@ import 'package:todoapp/data/model/tasks_complete_status.dart';
 import 'package:todoapp/ui/components/remove_task_dialog_builder.dart';
 import 'package:todoapp/ui/components/tasks_view_model/tasks_viewmodel.dart';
 import 'package:todoapp/ui/components/widgets/checklist/checklist_item_widget.dart';
-import 'package:todoapp/ui/components/widgets/task/taskslist/tasks_list_widget.dart';
+import 'package:todoapp/ui/components/widgets/tasks_list_widget.dart';
 import 'package:todoapp/ui/l10n/app_localizations.dart';
 import 'package:todoapp/ui/todo_app_router_config.gr.dart';
 import 'package:todoapp/util/di/dependency_startup_launcher.dart';

…ts/task/taskslist/tasks_list_widget.dart …omponents/widgets/tasks_list_widget.dartlib/ui/components/widgets/task/taskslist/tasks_list_widget.dart renamed to lib/ui/components/widgets/tasks_list_widget.dart lib/ui/components/widgets/task/taskslist/tasks_list_widget.dart renamed to lib/ui/components/widgets/tasks_list_widget.dart

@@ -49,7 +49,7 @@ class TasksListWidget extends StatelessWidget {
         ),
       );
     } else {
-      child = Column(
+      Widget header = Column(
         crossAxisAlignment: CrossAxisAlignment.start,
         children: [
           ProgressWidget(
@@ -60,21 +60,21 @@ class TasksListWidget extends StatelessWidget {
             onClick: () {
               onCompleteButtonAction();
             },
-          ),
-          Expanded(
-            child: ReorderableListView.builder(
-              onReorder: onReorder,
-              padding: const EdgeInsets.only(
-                bottom: 120.0,
-              ),
-              itemBuilder: (context, index) => _buildTaskCellWidget(
-                tasks[index],
-              ),
-              itemCount: tasks.length,
-            ),
-          ),
+          )
         ],
       );
+
+      child = ReorderableListView.builder(
+        header: header,
+        onReorder: onReorder,
+        padding: const EdgeInsets.only(
+          bottom: 120.0,
+        ),
+        itemBuilder: (context, index) => _buildTaskCellWidget(
+          tasks[index],
+        ),
+        itemCount: tasks.length,
+      );
     }
 
     return child;

lib/ui/screens/tasks/tasks_screen.dart

@@ -7,7 +7,7 @@ import 'package:todoapp/ui/components/remove_task_dialog_builder.dart';
 import 'package:todoapp/ui/components/tasks_view_model/tasks_screen_state.dart';
 import 'package:todoapp/ui/components/tasks_view_model/tasks_viewmodel.dart';
 import 'package:todoapp/ui/components/widgets/custom_app_bar_widget.dart';
-import 'package:todoapp/ui/components/widgets/task/taskslist/tasks_list_widget.dart';
+import 'package:todoapp/ui/components/widgets/tasks_list_widget.dart';
 import 'package:todoapp/ui/l10n/app_localizations.dart';
 import 'package:todoapp/ui/screens/tasks/tasks_screen_callbacks.dart';
 import 'package:todoapp/ui/todo_app_router_config.gr.dart';

pubspec.yaml

@@ -16,7 +16,7 @@ publish_to: 'none' # Remove this line if you wish to publish to pub.dev
 # https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
 # In Windows, build-name is used as the major, minor, and patch parts
 # of the product and file versions while build-number is used as the build suffix.
-version: 1.2.2
+version: 1.2.3
 
 environment:
   sdk: ^3.5.0