make refresh on comeback optional

Lukas-Heiligenbrunner · Dec 2, 2024 · 14fcc3d · 14fcc3d
1 parent 9bd3a9d
commit 14fcc3d
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 26 deletions.
diff --git a/frontend/lib/components/api/ApiBuilder.dart b/frontend/lib/components/api/ApiBuilder.dart
@@ -26,9 +26,12 @@ class APIBuilder<T> extends StatefulWidget {
       required this.onLoad,
       required this.onData,
       required this.api,
-      this.controller});
+      this.controller,
+      this.refreshOnComeback = false});
 
   final Duration? interval;
+  final bool refreshOnComeback;
+
   final Widget Function() onLoad;
   final Widget Function(T data) onData;
   final Future<T> Function() api;
@@ -59,32 +62,37 @@ class _APIBuilderState<T> extends State<APIBuilder<T>> {
 
   @override
   Widget build(BuildContext context) {
-    return VisibilityDetector(
-      key: widget.key ?? Key(hashCode.toString()),
-      onVisibilityChanged: (VisibilityInfo info) {
-        if (info.visibleFraction > 0) {
-          _refreshData();
+    final builder = FutureBuilder<T>(
+      future: _futureData,
+      builder: (context, snapshot) {
+        if (snapshot.hasError) {
+          print(snapshot.error);
+          WidgetsBinding.instance
+              .addPostFrameCallback((_) => toastification.show(
+                    title: Text('API Request failed! ${snapshot.error}'),
+                    autoCloseDuration: const Duration(seconds: 5),
+                    type: ToastificationType.error,
+                  ));
+        }
+        if (snapshot.hasData) {
+          return widget.onData(snapshot.data!);
+        } else {
+          return widget.onLoad();
         }
       },
-      child: FutureBuilder<T>(
-        future: _futureData,
-        builder: (context, snapshot) {
-          if (snapshot.hasError) {
-            print(snapshot.error);
-            WidgetsBinding.instance
-                .addPostFrameCallback((_) => toastification.show(
-                      title: Text('API Request failed! ${snapshot.error}'),
-                      autoCloseDuration: const Duration(seconds: 5),
-                      type: ToastificationType.error,
-                    ));
-          }
-          if (snapshot.hasData) {
-            return widget.onData(snapshot.data!);
-          } else {
-            return widget.onLoad();
-          }
-        },
-      ),
     );
+
+    if (widget.refreshOnComeback) {
+      return VisibilityDetector(
+          key: widget.key ?? Key(hashCode.toString()),
+          onVisibilityChanged: (VisibilityInfo info) {
+            if (info.visibleFraction > 0) {
+              _refreshData();
+            }
+          },
+          child: builder);
+    } else {
+      return builder;
+    }
   }
 }
diff --git a/frontend/lib/components/dashboard/recent_builds.dart b/frontend/lib/components/dashboard/recent_builds.dart
@@ -33,6 +33,7 @@ class RecentBuilds extends StatelessWidget {
           APIBuilder(
             onLoad: () => const Text("no data"),
             controller: apiController,
+            refreshOnComeback: true,
             onData: (t) {
               if (t.isEmpty) {
                 return const TableInfo(title: "You have no builds yet");

diff --git a/frontend/lib/components/dashboard/your_packages.dart b/frontend/lib/components/dashboard/your_packages.dart
@@ -10,7 +10,7 @@ import '../api/ApiBuilder.dart';
 import '../table_info.dart';
 
 class YourPackages extends StatelessWidget {
-  YourPackages({super.key});
+  const YourPackages({super.key});
 
   @override
   Widget build(BuildContext context) {
@@ -32,6 +32,7 @@ class YourPackages extends StatelessWidget {
           ),
           APIBuilder(
             controller: apiController,
+            refreshOnComeback: true,
             onData: (data) {
               if (data.isEmpty) {
                 return const TableInfo(title: "You have no packages yet");