feat(ref: no-ref): fix comments

Author: andriikamaldinov1

lib/common/button/button.widget.dart

@@ -1,43 +1,37 @@
 import 'package:flutter/material.dart';
 
 class ButtonWidget extends StatelessWidget {
-  const ButtonWidget({
-    required this.label,
-    required this.onTap,
-    this.isActive = false,
-    super.key,
-  });
+  const ButtonWidget(
+      {
+        required this.title,
+        required this.onTap,
+        this.isActive = false,
+        super.key});
 
-  final String label;
+  final String title;
   final bool isActive;
   final VoidCallback onTap;
 
   @override
-  Widget build(BuildContext context) => Padding(
-      padding: const EdgeInsets.symmetric(horizontal: 8),
-      child: TextButton(
-        onPressed: onTap,
-        style: ButtonStyle(
-          side: WidgetStateProperty.all(
-            BorderSide(
-              color: isActive ? Colors.red : Colors.transparent,
-              width: 2,
-            ),
-          ),
-          foregroundColor: WidgetStateProperty.all(
-            isActive ? Colors.red : Colors.grey,
-          ),
-          padding: WidgetStateProperty.all(
-            const EdgeInsets.symmetric(horizontal: 16),
-          ),
-        ),
-        child: Text(
-          label,
-          style: TextStyle(
-            fontSize: 14,
-            fontWeight: isActive ? FontWeight.bold : FontWeight.normal,
-          ),
-        ),
-      ),
-    );
-}
+  Widget build(BuildContext context) => InkWell(
+      onTap: onTap,
+      borderRadius: BorderRadius.circular(14),
+      child: Container(
+          height: 40,
+          padding: const EdgeInsets.symmetric(horizontal: 8),
+          decoration: BoxDecoration(
+              border: Border.all(
+                  color: isActive ? Colors.red : Colors.transparent,
+                  width: 2,
+              ),
+              borderRadius: BorderRadius.circular(14)),
+          child: Center(
+              child: Text(title,
+                style: const TextStyle(
+                  fontSize: 14,
+                ),
+              )
+          )
+      )
+  );
+}

lib/screens/home/todo/filter-panel/filter-panel.controller.dart

@@ -2,20 +2,15 @@ import 'package:get/get.dart';
 import '../todo.model.dart';
 import '../todo.service.dart';
 import 'filter-panel.enum.dart';
-import 'filter-panel.model.dart';
 
 class FilterPanelController extends GetxController {
   FilterPanelController(this._todoService);
 
   final TodoService _todoService;
   final Rx<FilterType> currentFilter = FilterType.all.obs;
-  final List<FilterModel> filters = const <FilterModel>[
-    FilterModel(type: FilterType.all),
-    FilterModel(type: FilterType.active),
-    FilterModel(type: FilterType.completed),
-  ];
+  final List<FilterType> filters = FilterType.values;
 
-  List<TaskModel> get filteredTasks => _todoService.filteredTasks(currentFilter.value.label);
+  List<TaskModel> get filteredTasks => _todoService.filteredTasks(currentFilter.value);
   List<TaskModel> get tasks => _todoService.tasks;
 
   int get activeCount => _todoService.activeCount;

lib/screens/home/todo/filter-panel/filter-panel.enum.dart

@@ -1,16 +1,9 @@
 enum FilterType {
-  all,
-  active,
-  completed;
+  all('All'),
+  active('Active'),
+  completed('Completed');
 
-  String get label {
-    switch (this) {
-      case FilterType.all:
-        return 'All';
-      case FilterType.active:
-        return 'Active';
-      case FilterType.completed:
-        return 'Completed';
-    }
-  }
-}
+  const FilterType(this.type);
+
+  final String type;
+}

lib/screens/home/todo/filter-panel/filter-panel.model.dart

@@ -2,7 +2,7 @@ import 'package:flutter/material.dart';
 import 'package:get/get.dart';
 import '../../../../common/button/button.widget.dart';
 import 'filter-panel.controller.dart';
-import 'filter-panel.model.dart';
+import 'filter-panel.enum.dart';
 
 class FilterPanelWidget extends GetView<FilterPanelController> {
   const FilterPanelWidget({super.key});
@@ -22,18 +22,19 @@ class FilterPanelWidget extends GetView<FilterPanelController> {
             '${controller.activeCount} items left',
             style: const TextStyle(fontSize: 14, color: Colors.grey),
           ),
-          Row(
+          Wrap(
+            spacing: 10,
             children: controller.filters
-                .map<Widget>((FilterModel filter) => ButtonWidget(
-              label: filter.label,
-              isActive: controller.currentFilter.value == filter.type,
-              onTap: () => controller.setFilter(filter.type),
+                .map<Widget>((FilterType filter) => ButtonWidget(
+              title: filter.type,
+              isActive: controller.currentFilter.value == filter,
+              onTap: () => controller.setFilter(filter),
             ))
                 .toList(),
           ),
           ButtonWidget(
             onTap: controller.clearCompleted,
-            label: 'Clear completed',
+            title: 'Clear completed',
           ),
         ],
       ),

lib/screens/home/todo/filter-panel/filter-panel.widget.dart

@@ -9,19 +9,23 @@ class TaskListWidget extends GetView<FilterPanelController> {
   const TaskListWidget({super.key});
 
   @override
-  Widget build(BuildContext context) => Obx(() => Column(
-      children: <Widget>[
-        if (controller.filteredTasks.isNotEmpty)
-          Column(
+  Widget build(BuildContext context) => Column(
+    children: <Widget>[
+      Obx(() {
+        if (controller.filteredTasks.isNotEmpty) {
+          return Column(
             children: controller.filteredTasks
-                .map(
+                .map<Widget>(
                   (TaskModel task) => TaskListItemWidget(
                 task: task,
               ),
             )
                 .toList(),
-          ),
-        const FilterPanelWidget(),
-      ],
-    ));
+          );
+        }
+        return const SizedBox();
+      }),
+      const FilterPanelWidget(),
+    ],
+  );
 }

lib/screens/home/todo/task-list/task-list.widget.dart

@@ -1,4 +1,5 @@
 import 'package:get/get.dart';
+import 'filter-panel/filter-panel.enum.dart';
 import 'todo.model.dart';
 
 class TodoService extends GetxService {
@@ -14,11 +15,11 @@ class TodoService extends GetxService {
   }
 
 
-  List<TaskModel> filteredTasks(String filter) {
+  List<TaskModel> filteredTasks(FilterType filter) {
     switch (filter) {
-      case 'active':
+      case FilterType.active:
         return tasks.where((TaskModel task) => !task.isFinished).toList();
-      case 'completed':
+      case FilterType.completed:
         return tasks.where((TaskModel task) => task.isFinished).toList();
       default:
         return tasks;

lib/screens/home/todo/todo.service.dart

@@ -13,7 +13,7 @@ import 'package:todo_mvc/main.dart';
 void main() {
   testWidgets('Counter increments smoke test', (WidgetTester tester) async {
     // Build our app and trigger a frame.
-    await tester.pumpWidget(TodoApp());
+    await tester.pumpWidget(const MyApp());
 
     // Verify that our counter starts at 0.
     expect(find.text('0'), findsOneWidget);