Imports Rearranged,Reduced Redundant Widgets and Updated Tests.

Author: Tilakraj-B

lib/screens/home_page/collection_pane.dart

@@ -1,10 +1,10 @@
-import 'package:apidash/consts.dart';
-import 'package:apidash/models/models.dart';
+import 'package:flutter/material.dart';
+import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:apidash/providers/providers.dart';
 import 'package:apidash/widgets/overlay_widget.dart';
 import 'package:apidash/widgets/widgets.dart';
-import 'package:flutter/material.dart';
-import 'package:flutter_riverpod/flutter_riverpod.dart';
+import 'package:apidash/models/models.dart';
+import 'package:apidash/consts.dart';
 
 class CollectionPane extends ConsumerWidget {
   const CollectionPane({
@@ -35,14 +35,16 @@ class CollectionPane extends ConsumerWidget {
                   onPressed: savingData
                       ? null
                       : () async {
-                          overlayWidget.show(widget: const SavingOverlay());
+                          overlayWidget.show(
+                              widget:
+                                  const SavingOverlay(saveCompleted: false));
 
                           await ref
                               .read(collectionStateNotifierProvider.notifier)
                               .saveData();
-
                           overlayWidget.hide();
-                          overlayWidget.show(widget: const CompletedOverlay());
+                          overlayWidget.show(
+                              widget: const SavingOverlay(saveCompleted: true));
                           await Future.delayed(const Duration(seconds: 1));
                           overlayWidget.hide();
                         },

lib/widgets/overlay_widget.dart

@@ -1,6 +1,5 @@
 import 'package:flutter/material.dart';
 import 'package:lottie/lottie.dart';
-
 import '../consts.dart';
 
 class OverlayWidgetTemplate {
@@ -32,7 +31,8 @@ class OverlayWidgetTemplate {
 }
 
 class SavingOverlay extends StatelessWidget {
-  const SavingOverlay({super.key});
+  final bool saveCompleted;
+  const SavingOverlay({super.key, required this.saveCompleted});
 
   @override
   Widget build(BuildContext context) {
@@ -43,11 +43,16 @@ class SavingOverlay extends StatelessWidget {
           child: Column(
             mainAxisSize: MainAxisSize.min,
             children: [
-              Lottie.asset("assets/saving.json", width: 100, height: 100),
+              Lottie.asset(
+                  saveCompleted
+                      ? "assets/completed.json"
+                      : "assets/saving.json",
+                  width: 100,
+                  height: 100),
               kHSpacer20,
-              const Text(
-                kLabelSaving,
-                style: TextStyle(
+              Text(
+                saveCompleted ? kLabelSaved : kLabelSaving,
+                style: const TextStyle(
                   fontSize: kDefaultFontSize,
                 ),
               )
@@ -58,30 +63,3 @@ class SavingOverlay extends StatelessWidget {
     );
   }
 }
-
-class CompletedOverlay extends StatelessWidget {
-  const CompletedOverlay({super.key});
-
-  @override
-  Widget build(BuildContext context) {
-    return Center(
-      child: Card(
-        child: Padding(
-          padding: kPh60v60,
-          child: Column(
-            mainAxisSize: MainAxisSize.min,
-            children: [
-              Lottie.asset("assets/completed.json",
-                  width: 100, height: 100, repeat: false),
-              kHSpacer20,
-              const Text(
-                kLabelSaved,
-                style: TextStyle(fontSize: kDefaultFontSize),
-              )
-            ],
-          ),
-        ),
-      ),
-    );
-  }
-}

test/widgets/overlay_widget_test.dart

@@ -32,25 +32,25 @@ void main() {
     await tester.pumpWidget(
       const MaterialApp(
         home: Scaffold(
-          body: SavingOverlay(),
+          body: SavingOverlay(
+            saveCompleted: false,
+          ),
         ),
       ),
     );
-
     expect(find.byType(Card), findsOneWidget);
     expect(find.byType(Lottie), findsOneWidget);
     expect(find.text(kLabelSaving), findsOneWidget);
-  });
 
-  testWidgets('CompletedOverlay Test', (WidgetTester tester) async {
     await tester.pumpWidget(
       const MaterialApp(
         home: Scaffold(
-          body: CompletedOverlay(),
+          body: SavingOverlay(
+            saveCompleted: true,
+          ),
         ),
       ),
     );
-
     expect(find.byType(Card), findsOneWidget);
     expect(find.byType(Lottie), findsOneWidget);
     expect(find.text(kLabelSaved), findsOneWidget);