From 1ed6d27ca71cdfdee77becc5d10b696b337ded11 Mon Sep 17 00:00:00 2001
From: Achintha Isuru <achinthaisuru444@gmail.com>
Date: Fri, 27 Feb 2026 21:45:43 -0500
Subject: [PATCH] refactor: Centralize bottom bar visibility paths into a list
 and simplify the conditional check.

---
 .../lib/src/presentation/blocs/client_main_cubit.dart    | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/apps/mobile/packages/features/client/client_main/lib/src/presentation/blocs/client_main_cubit.dart b/apps/mobile/packages/features/client/client_main/lib/src/presentation/blocs/client_main_cubit.dart
index 3822645c..1b6683fd 100644
--- a/apps/mobile/packages/features/client/client_main/lib/src/presentation/blocs/client_main_cubit.dart
+++ b/apps/mobile/packages/features/client/client_main/lib/src/presentation/blocs/client_main_cubit.dart
@@ -9,6 +9,11 @@ class ClientMainCubit extends Cubit<ClientMainState> implements Disposable {
     _onRouteChanged();
   }
 
+  static const List<String> _hideBottomBarPaths = <String>[
+    ClientPaths.completionReview,
+    ClientPaths.awaitingApproval,
+  ];
+
   void _onRouteChanged() {
     final String path = Modular.to.path;
     int newIndex = state.currentIndex;
@@ -26,9 +31,7 @@ class ClientMainCubit extends Cubit<ClientMainState> implements Disposable {
       newIndex = 4;
     }
 
-    final bool showBottomBar =
-        !path.contains(ClientPaths.completionReview) &&
-        !path.contains(ClientPaths.awaitingApproval);
+    final bool showBottomBar = !_hideBottomBarPaths.any(path.contains);
 
     if (newIndex != state.currentIndex ||
         showBottomBar != state.showBottomBar) {