From 08e5e17910a6b3e56c207300ba9952b701a42159 Mon Sep 17 00:00:00 2001
From: Faris Armoush <farmoush@uba.com.jo>
Date: Tue, 15 Apr 2025 15:23:36 +0300
Subject: [PATCH] Extracted `Add Devices` button into a private method.

---
 .../widgets/tag_chips_display_widget.dart     | 58 ++++++++++---------
 1 file changed, 31 insertions(+), 27 deletions(-)

diff --git a/lib/pages/spaces_management/space_model/widgets/tag_chips_display_widget.dart b/lib/pages/spaces_management/space_model/widgets/tag_chips_display_widget.dart
index 1472a05b..1f75a047 100644
--- a/lib/pages/spaces_management/space_model/widgets/tag_chips_display_widget.dart
+++ b/lib/pages/spaces_management/space_model/widgets/tag_chips_display_widget.dart
@@ -53,18 +53,18 @@ class TagChipDisplay extends StatelessWidget {
       return SizedBox(
         width: screenWidth * 0.25,
         child: Container(
-          padding: const EdgeInsets.all(8.0),
+          padding: const EdgeInsets.all(8),
           decoration: BoxDecoration(
             color: ColorsManager.textFieldGreyColor,
             borderRadius: BorderRadius.circular(15),
             border: Border.all(
               color: ColorsManager.textFieldGreyColor,
-              width: 3.0,
+              width: 3,
             ),
           ),
           child: Wrap(
-            spacing: 8.0,
-            runSpacing: 8.0,
+            spacing: 8,
+            runSpacing: 8,
             children: [
               ..._groupedTags.entries.map((entry) {
                 return Chip(
@@ -120,29 +120,33 @@ class TagChipDisplay extends StatelessWidget {
         ),
       );
     } else {
-      return TextButton(
-        onPressed: () => showDialog<void>(
-          context: context,
-          builder: (context) => AddDeviceTypeModelWidget(
-            products: products,
-            subspaces: subspaces,
-            allTags: allTags,
-            spaceName: spaceNameController.text,
-            pageContext: pageContext,
-            isCreate: true,
-            spaceModel: spaceModel,
-            otherSpaceModels: otherSpaceModels,
-            projectTags: projectTags,
-          ),
-        ),
-        style: TextButton.styleFrom(
-          padding: EdgeInsets.zero,
-        ),
-        child: const ButtonContentWidget(
-          icon: Icons.add,
-          label: 'Add Devices',
-        ),
-      );
+      return _buildAddDevicesButton(context);
     }
   }
+
+  Widget _buildAddDevicesButton(BuildContext context) {
+    return TextButton(
+      onPressed: () => showDialog<void>(
+        context: context,
+        builder: (context) => AddDeviceTypeModelWidget(
+          products: products,
+          subspaces: subspaces,
+          allTags: allTags,
+          spaceName: spaceNameController.text,
+          pageContext: pageContext,
+          isCreate: true,
+          spaceModel: spaceModel,
+          otherSpaceModels: otherSpaceModels,
+          projectTags: projectTags,
+        ),
+      ),
+      style: TextButton.styleFrom(
+        padding: EdgeInsets.zero,
+      ),
+      child: const ButtonContentWidget(
+        icon: Icons.add,
+        label: 'Add Devices',
+      ),
+    );
+  }
 }