Merge pull request #73 from SyncrowIOT/bugfix/edit-space

Bugfix/edit space

lib/pages/spaces_management/add_device_type/views/add_device_type_widget.dart

@@ -12,7 +12,6 @@ import 'package:syncrow_web/pages/spaces_management/all_spaces/model/product_mod
 import 'package:syncrow_web/pages/spaces_management/all_spaces/model/selected_product_model.dart';
 import 'package:syncrow_web/pages/spaces_management/assign_tag/views/assign_tag_dialog.dart';
 import 'package:syncrow_web/pages/spaces_management/helper/tag_helper.dart';
-import 'package:syncrow_web/pages/spaces_management/tag_model/widgets/action_button_widget.dart';
 import 'package:syncrow_web/utils/color_manager.dart';
 
 class AddDeviceTypeWidget extends StatelessWidget {
@@ -23,10 +22,12 @@ class AddDeviceTypeWidget extends StatelessWidget {
   final List<Tag>? spaceTags;
   final List<String>? allTags;
   final String spaceName;
+  final bool isCreate;
   final Function(List<Tag>, List<SubspaceModel>?)? onSave;
 
   const AddDeviceTypeWidget(
       {super.key,
+      required this.isCreate,
       this.products,
       this.initialSelectedProducts,
       this.onProductsSelected,
@@ -74,7 +75,9 @@ class AddDeviceTypeWidget extends StatelessWidget {
                             padding:
                                 const EdgeInsets.symmetric(horizontal: 20.0),
                             child: ScrollableGridViewWidget(
+                                initialProductCounts: state.selectedProducts,
                                 products: products,
+                                isCreate: isCreate,
                                 crossAxisCount: crossAxisCount),
                           ),
                         ),
@@ -131,10 +134,7 @@ class AddDeviceTypeWidget extends StatelessWidget {
                                       spaceName: spaceName,
                                       initialTags: initialTags,
                                       title: dialogTitle,
-                                    onSave: (tags, subspaces) {
+                                      onSave: onSave),
-                                      onSave!(tags, subspaces);
-                                    },
-                                  ),
                                 );
                               }
                             },
@@ -148,29 +148,4 @@ class AddDeviceTypeWidget extends StatelessWidget {
           ),
         ));
   }
-
-  List<Tag> generateInitialTags({
-    List<Tag>? spaceTags,
-    List<SubspaceModel>? subspaces,
-  }) {
-    final List<Tag> initialTags = [];
-
-    if (spaceTags != null) {
-      initialTags.addAll(spaceTags);
-    }
-
-    if (subspaces != null) {
-      for (var subspace in subspaces) {
-        if (subspace.tags != null) {
-          initialTags.addAll(
-            subspace.tags!.map(
-              (tag) => tag.copyWith(location: subspace.subspaceName),
-            ),
-          );
-        }
-      }
-    }
-
-    return initialTags;
-  }
 }

lib/pages/spaces_management/add_device_type/widgets/device_type_tile_widget.dart

@@ -13,12 +13,13 @@ import 'package:syncrow_web/utils/constants/assets.dart';
 class DeviceTypeTileWidget extends StatelessWidget {
   final ProductModel product;
   final List<SelectedProduct> productCounts;
+  final bool isCreate;
 
-  const DeviceTypeTileWidget({
+  const DeviceTypeTileWidget(
-    super.key,
+      {super.key,
       required this.product,
       required this.productCounts,
-  });
+      required this.isCreate});
 
   @override
   Widget build(BuildContext context) {
@@ -48,7 +49,7 @@ class DeviceTypeTileWidget extends StatelessWidget {
             DeviceNameWidget(name: product.name),
             const SizedBox(height: 4),
             CounterWidget(
-              isCreate: false,
+              isCreate: isCreate,
               initialCount: selectedProduct.count,
               onCountChanged: (newCount) {
                 context.read<AddDeviceTypeBloc>().add(

lib/pages/spaces_management/add_device_type/widgets/scrollable_grid_view_widget.dart

@@ -10,13 +10,14 @@ class ScrollableGridViewWidget extends StatelessWidget {
   final List<ProductModel>? products;
   final int crossAxisCount;
   final List<SelectedProduct>? initialProductCounts;
+  final bool isCreate;
 
-  const ScrollableGridViewWidget({
+  const ScrollableGridViewWidget(
-    super.key,
+      {super.key,
       required this.products,
       required this.crossAxisCount,
       this.initialProductCounts,
-  });
+      required this.isCreate});
 
   @override
   Widget build(BuildContext context) {
@@ -46,6 +47,7 @@ class ScrollableGridViewWidget extends StatelessWidget {
 
               return DeviceTypeTileWidget(
                 product: product,
+                isCreate: isCreate,
                 productCounts: initialProductCount != null
                     ? [...productCounts, initialProductCount]
                     : productCounts,

lib/pages/spaces_management/all_spaces/bloc/space_management_bloc.dart

@@ -5,12 +5,15 @@ import 'package:syncrow_web/pages/spaces_management/all_spaces/model/product_mod
 import 'package:syncrow_web/pages/spaces_management/all_spaces/model/space_model.dart';
 import 'package:syncrow_web/pages/spaces_management/all_spaces/bloc/space_management_event.dart';
 import 'package:syncrow_web/pages/spaces_management/all_spaces/bloc/space_management_state.dart';
+import 'package:syncrow_web/pages/spaces_management/all_spaces/model/subspace_model.dart';
 import 'package:syncrow_web/pages/spaces_management/all_spaces/model/tag.dart';
 import 'package:syncrow_web/pages/spaces_management/space_model/models/space_template_model.dart';
 import 'package:syncrow_web/pages/spaces_management/space_model/models/tag_body_model.dart';
+import 'package:syncrow_web/pages/spaces_management/space_model/models/tag_update_model.dart';
 import 'package:syncrow_web/services/product_api.dart';
 import 'package:syncrow_web/services/space_mana_api.dart';
 import 'package:syncrow_web/services/space_model_mang_api.dart';
+import 'package:syncrow_web/utils/constants/action_enum.dart';
 
 class SpaceManagementBloc
     extends Bloc<SpaceManagementEvent, SpaceManagementState> {
@@ -341,7 +344,7 @@ class SpaceManagementBloc
             products: _cachedProducts ?? [],
             selectedCommunity: selectedCommunity,
             selectedSpace: selectedSpace,
-            spaceModels: spaceModels ?? []));
+            spaceModels: spaceModels));
       }
     } catch (e) {
       emit(SpaceManagementError('Error updating state: $e'));
@@ -428,6 +431,76 @@ class SpaceManagementBloc
     for (var space in orderedSpaces) {
       try {
         if (space.uuid != null && space.uuid!.isNotEmpty) {
+          List<TagModelUpdate> tagUpdates = [];
+
+          final prevSpace = await _api.getSpace(communityUuid, space.uuid!);
+          final List<UpdateSubspaceTemplateModel> subspaceUpdates = [];
+          final List<SubspaceModel>? prevSubspaces = prevSpace?.subspaces;
+          final List<SubspaceModel>? newSubspaces = space.subspaces;
+
+          tagUpdates = processTagUpdates(prevSpace?.tags, space.tags);
+
+          if (prevSubspaces != null || newSubspaces != null) {
+            if (prevSubspaces != null && newSubspaces != null) {
+              for (var prevSubspace in prevSubspaces) {
+                final existsInNew = newSubspaces
+                    .any((subspace) => subspace.uuid == prevSubspace.uuid);
+                if (!existsInNew) {
+                  subspaceUpdates.add(UpdateSubspaceTemplateModel(
+                      action: Action.delete, uuid: prevSubspace.uuid));
+                }
+              }
+            } else if (prevSubspaces != null && newSubspaces == null) {
+              for (var prevSubspace in prevSubspaces) {
+                subspaceUpdates.add(UpdateSubspaceTemplateModel(
+                    action: Action.delete, uuid: prevSubspace.uuid));
+              }
+            }
+
+            if (newSubspaces != null) {
+              for (var newSubspace in newSubspaces) {
+                // Tag without UUID
+                if ((newSubspace.uuid == null || newSubspace.uuid!.isEmpty)) {
+                  final List<TagModelUpdate> tagUpdates = [];
+
+                  if (newSubspace.tags != null) {
+                    for (var tag in newSubspace.tags!) {
+                      tagUpdates.add(TagModelUpdate(
+                          action: Action.add,
+                          uuid: tag.uuid == '' ? null : tag.uuid,
+                          tag: tag.tag,
+                          productUuid: tag.product?.uuid));
+                    }
+                  }
+                  subspaceUpdates.add(UpdateSubspaceTemplateModel(
+                      action: Action.add,
+                      subspaceName: newSubspace.subspaceName,
+                      tags: tagUpdates));
+                }
+              }
+            }
+
+            if (prevSubspaces != null && newSubspaces != null) {
+              final newSubspaceMap = {
+                for (var subspace in newSubspaces) subspace.uuid: subspace
+              };
+
+              for (var prevSubspace in prevSubspaces) {
+                final newSubspace = newSubspaceMap[prevSubspace.uuid];
+
+                if (newSubspace != null) {
+                  final List<TagModelUpdate> tagSubspaceUpdates =
+                      processTagUpdates(prevSubspace.tags, newSubspace.tags);
+                  subspaceUpdates.add(UpdateSubspaceTemplateModel(
+                      action: Action.update,
+                      uuid: newSubspace.uuid,
+                      subspaceName: newSubspace.subspaceName,
+                      tags: tagSubspaceUpdates));
+                }
+              }
+            }
+          }
+
           final response = await _api.updateSpace(
             communityId: communityUuid,
             spaceId: space.uuid!,
@@ -436,6 +509,8 @@ class SpaceManagementBloc
             isPrivate: space.isPrivate,
             position: space.position,
             icon: space.icon,
+            subspaces: subspaceUpdates,
+            tags: tagUpdates,
             direction: space.incomingConnection?.direction,
           );
         } else {
@@ -535,4 +610,79 @@ class SpaceManagementBloc
       emit(SpaceManagementError('Error loading communities and spaces: $e'));
     }
   }
+
+  List<TagModelUpdate> processTagUpdates(
+    List<Tag>? prevTags,
+    List<Tag>? newTags,
+  ) {
+    final List<TagModelUpdate> tagUpdates = [];
+    final processedTags = <String?>{};
+
+    if (prevTags == null && newTags != null) {
+      for (var newTag in newTags) {
+        tagUpdates.add(TagModelUpdate(
+          action: Action.add,
+          tag: newTag.tag,
+          uuid: newTag.uuid,
+          productUuid: newTag.product?.uuid,
+        ));
+      }
+      return tagUpdates;
+    }
+
+    if (newTags != null || prevTags != null) {
+      // Case 1: Tags deleted
+      if (prevTags != null && newTags != null) {
+        for (var prevTag in prevTags) {
+          final existsInNew =
+              newTags.any((newTag) => newTag.uuid == prevTag.uuid);
+          if (!existsInNew) {
+            tagUpdates
+                .add(TagModelUpdate(action: Action.delete, uuid: prevTag.uuid));
+          }
+        }
+      } else if (prevTags != null && newTags == null) {
+        for (var prevTag in prevTags) {
+          tagUpdates
+              .add(TagModelUpdate(action: Action.delete, uuid: prevTag.uuid));
+        }
+      }
+
+      // Case 2: Tags added
+      if (newTags != null) {
+        final prevTagUuids = prevTags?.map((t) => t.uuid).toSet() ?? {};
+
+        for (var newTag in newTags) {
+          // Tag without UUID
+          if ((newTag.uuid == null || !prevTagUuids.contains(newTag.uuid)) &&
+              !processedTags.contains(newTag.tag)) {
+            tagUpdates.add(TagModelUpdate(
+                action: Action.add,
+                tag: newTag.tag,
+                uuid: newTag.uuid == '' ? null : newTag.uuid,
+                productUuid: newTag.product?.uuid));
+            processedTags.add(newTag.tag);
+          }
+        }
+      }
+
+      // Case 3: Tags updated
+      if (prevTags != null && newTags != null) {
+        final newTagMap = {for (var tag in newTags) tag.uuid: tag};
+
+        for (var prevTag in prevTags) {
+          final newTag = newTagMap[prevTag.uuid];
+          if (newTag != null) {
+            tagUpdates.add(TagModelUpdate(
+              action: Action.update,
+              uuid: newTag.uuid,
+              tag: newTag.tag,
+            ));
+          } else {}
+        }
+      }
+    }
+
+    return tagUpdates;
+  }
 }

lib/pages/spaces_management/all_spaces/model/space_model.dart

@@ -66,7 +66,6 @@ class SpaceModel {
     final instance = SpaceModel(
       internalId: internalId,
       uuid: json['uuid'] ?? '',
-      spaceTuyaUuid: json['spaceTuyaUuid'],
       name: json['spaceName'],
       isPrivate: json['isPrivate'] ?? false,
       invitationCode: json['invitationCode'],

lib/pages/spaces_management/all_spaces/widgets/community_structure_widget.dart

@@ -195,6 +195,7 @@ class _CommunityStructureAreaState extends State<CommunityStructureArea> {
                                     screenSize,
                                     position:
                                         spaces[index].position + newPosition,
+                                    
                                     parentIndex: index,
                                     direction: direction,
                                   );
@@ -294,6 +295,7 @@ class _CommunityStructureAreaState extends State<CommunityStructureArea> {
         return CreateSpaceDialog(
           products: widget.products,
           spaceModels: widget.spaceModels,
+          allTags: _getAllTagValues(spaces),
           parentSpace: parentIndex != null ? spaces[parentIndex] : null,
           onCreateSpace: (String name,
               String icon,
@@ -350,7 +352,10 @@ class _CommunityStructureAreaState extends State<CommunityStructureArea> {
             name: widget.selectedSpace!.name,
             icon: widget.selectedSpace!.icon,
             editSpace: widget.selectedSpace,
+            tags: widget.selectedSpace?.tags,
+            subspaces: widget.selectedSpace?.subspaces,
             isEdit: true,
+          allTags: _getAllTagValues(spaces),
             onCreateSpace: (String name,
                 String icon,
                 List<SelectedProduct> selectedProducts,
@@ -742,4 +747,14 @@ class _CommunityStructureAreaState extends State<CommunityStructureArea> {
       duplicateRecursive(space, space.position, duplicatedParent);
     }
   }
+
+    List<String> _getAllTagValues(List<SpaceModel> spaces) {
+    final List<String> allTags = [];
+    for (final space in spaces) {
+      if (space.tags != null) {
+        allTags.addAll(space.listAllTagValues());
+      }
+    }
+    return allTags;
+  }
 }

lib/pages/spaces_management/all_spaces/widgets/dialogs/create_space_dialog.dart

@@ -39,6 +39,7 @@ class CreateSpaceDialog extends StatefulWidget {
   final List<SpaceTemplateModel>? spaceModels;
   final List<SubspaceModel>? subspaces;
   final List<Tag>? tags;
+  final List<String>? allTags;
 
   const CreateSpaceDialog(
       {super.key,
@@ -49,6 +50,7 @@ class CreateSpaceDialog extends StatefulWidget {
       this.icon,
       this.isEdit = false,
       this.editSpace,
+      this.allTags,
       this.selectedProducts = const [],
       this.spaceModels,
       this.subspaces,
@@ -79,6 +81,8 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
         widget.selectedProducts.isNotEmpty ? widget.selectedProducts : [];
     isOkButtonEnabled =
         enteredName.isNotEmpty || nameController.text.isNotEmpty;
+    tags = widget.tags ?? [];
+    subspaces = widget.subspaces ?? [];
   }
 
   @override
@@ -171,14 +175,13 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
                             }
                           });
                         },
-                        style: const TextStyle(color: Colors.black),
+                        style: Theme.of(context).textTheme.bodyMedium,
                         decoration: InputDecoration(
                           hintText: 'Please enter the name',
-                          hintStyle: const TextStyle(
+                          hintStyle: Theme.of(context)
-                            fontSize: 14,
+                              .textTheme
-                            color: ColorsManager.lightGrayColor,
+                              .bodyMedium!
-                            fontWeight: FontWeight.w400,
+                              .copyWith(color: ColorsManager.lightGrayColor),
-                          ),
                           filled: true,
                           fillColor: ColorsManager.boxColor,
                           enabledBorder: OutlineInputBorder(
@@ -237,7 +240,7 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
                             ),
                           )
                         : Container(
-                            width: screenWidth * 0.35,
+                            width: screenWidth * 0.25,
                             padding: const EdgeInsets.symmetric(
                                 vertical: 10.0, horizontal: 16.0),
                             decoration: BoxDecoration(
@@ -251,7 +254,10 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
                                 Chip(
                                     label: Text(
                                       selectedSpaceModel?.modelName ?? '',
-                                      style: const TextStyle(
+                                      style: Theme.of(context)
+                                          .textTheme
+                                          .bodyMedium!
+                                          .copyWith(
                                               color: ColorsManager.spaceColor),
                                     ),
                                     backgroundColor: ColorsManager.whiteColors,
@@ -285,25 +291,25 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
                             ),
                           ),
                     const SizedBox(height: 25),
-                    const Row(
+                    Row(
                       children: [
-                        Expanded(
+                        const Expanded(
                           child: Divider(
                             color: ColorsManager.neutralGray,
                             thickness: 1.0,
                           ),
                         ),
                         Padding(
-                          padding: EdgeInsets.symmetric(horizontal: 6.0),
+                          padding: const EdgeInsets.symmetric(horizontal: 6.0),
                           child: Text(
                             'OR',
-                            style: TextStyle(
+                            style: Theme.of(context)
-                              color: Colors.black,
+                                .textTheme
-                              fontWeight: FontWeight.bold,
+                                .bodyMedium
+                                ?.copyWith(fontWeight: FontWeight.bold),
                           ),
                         ),
-                        ),
+                        const Expanded(
-                        Expanded(
                           child: Divider(
                             color: ColorsManager.neutralGray,
                             thickness: 1.0,
@@ -312,7 +318,7 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
                       ],
                     ),
                     const SizedBox(height: 25),
-                    subspaces == null
+                    subspaces == null || subspaces!.isEmpty
                         ? TextButton(
                             style: TextButton.styleFrom(
                               padding: EdgeInsets.zero,
@@ -344,21 +350,29 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
                                 runSpacing: 8.0,
                                 children: [
                                   if (subspaces != null)
-                                    ...subspaces!.map((subspace) =>
+                                    ...subspaces!.map((subspace) {
+                                      return Column(
+                                        crossAxisAlignment:
+                                            CrossAxisAlignment.start,
+                                        children: [
                                           SubspaceNameDisplayWidget(
-                                          validateName: (updatedName) {
-                                            return !subspaces!.any((s) =>
-                                                s != subspace &&
-                                                s.subspaceName == updatedName);
-                                          },
                                             text: subspace.subspaceName,
+                                            validateName: (updatedName) {
+                                              return subspaces!.any((s) =>
+                                                  s != subspace &&
+                                                  s.subspaceName ==
+                                                      updatedName);
+                                            },
                                             onNameChanged: (updatedName) {
                                               setState(() {
                                                 subspace.subspaceName =
                                                     updatedName;
                                               });
                                             },
-                                        )),
+                                          ),
+                                        ],
+                                      );
+                                    }),
                                   EditChip(
                                     onTap: () async {
                                       _showSubSpaceDialog(context, enteredName,
@@ -408,9 +422,12 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
                                           ),
                                           label: Text(
                                             'x${entry.value}', // Show count
-                                            style: const TextStyle(
+                                            style: Theme.of(context)
-                                              color: ColorsManager.spaceColor,
+                                                .textTheme
-                                            ),
+                                                .bodySmall
+                                                ?.copyWith(
+                                                    color: ColorsManager
+                                                        .spaceColor),
                                           ),
                                           backgroundColor:
                                               ColorsManager.whiteColors,
@@ -425,14 +442,29 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
                                       ),
 
                                   EditChip(onTap: () async {
-                                    _showTagCreateDialog(
+                                    final result = await showDialog(
-                                      context,
+                                      context: context,
-                                      enteredName,
+                                      builder: (context) => AssignTagDialog(
-                                      widget.isEdit,
+                                        products: widget.products,
-                                      widget.products,
+                                        subspaces: widget.subspaces,
-                                      subspaces,
+                                        addedProducts: TagHelper
+                                            .createInitialSelectedProductsForTags(
+                                                tags ?? [], subspaces),
+                                        title: 'Edit Device',
+                                        initialTags:
+                                            TagHelper.generateInitialForTags(
+                                                spaceTags: tags,
+                                                subspaces: subspaces),
+                                        spaceName: widget.name ?? '',
+                                        onSave:
+                                            (updatedTags, updatedSubspaces) {
+                                          setState(() {
+                                            tags = updatedTags;
+                                            subspaces = updatedSubspaces;
+                                          });
+                                        },
+                                      ),
                                     );
-                                    // Edit action
                                   })
                                 ],
                               ),
@@ -547,6 +579,7 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
               setState(() {
                 selectedSpaceModel = selectedModel;
                 subspaces = null;
+                tags = null;
               });
             }
           },
@@ -597,8 +630,26 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
                 spaceName: name,
                 products: products,
                 subspaces: subspaces,
-                allTags: [],
+                allTags: widget.allTags,
-                onSave: (selectedSpaceTags, selectedSubspaces) {},
+                onSave: (selectedSpaceTags, selectedSubspaces) {
+                  setState(() {
+                    tags = selectedSpaceTags;
+                    selectedSpaceModel = null;
+
+                    if (selectedSubspaces != null) {
+                      if (subspaces != null) {
+                        for (final subspace in subspaces!) {
+                          for (final selectedSubspace in selectedSubspaces) {
+                            if (subspace.subspaceName ==
+                                selectedSubspace.subspaceName) {
+                              subspace.tags = selectedSubspace.tags;
+                            }
+                          }
+                        }
+                      }
+                    }
+                  });
+                },
               );
             },
           )
@@ -610,7 +661,8 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
                 products: products,
                 subspaces: subspaces,
                 spaceTags: tags,
-                allTags: [],
+                isCreate: true,
+                allTags: widget.allTags,
                 initialSelectedProducts:
                     TagHelper.createInitialSelectedProductsForTags(
                         tags, subspaces),

lib/pages/spaces_management/all_spaces/widgets/space_widget.dart

@@ -20,8 +20,7 @@ class SpaceWidget extends StatelessWidget {
       top: position.dy,
       child: GestureDetector(
         onTap: onTap,
-        child: 
+        child: Container(
-        Container(
           padding: const EdgeInsets.all(8.0),
           decoration: BoxDecoration(
             color: ColorsManager.whiteColors,
@@ -39,11 +38,10 @@ class SpaceWidget extends StatelessWidget {
             children: [
               const Icon(Icons.location_on, color: ColorsManager.spaceColor),
               const SizedBox(width: 8),
-              Text(name, style: const TextStyle(fontSize: 16)),
+              Text(name, style: Theme.of(context).textTheme.bodyMedium),
             ],
           ),
         ),
-    
       ),
     );
   }

lib/pages/spaces_management/assign_tag/views/assign_tag_dialog.dart

@@ -12,6 +12,7 @@ import 'package:syncrow_web/pages/spaces_management/all_spaces/model/tag.dart';
 import 'package:syncrow_web/pages/spaces_management/assign_tag/bloc/assign_tag_bloc.dart';
 import 'package:syncrow_web/pages/spaces_management/assign_tag/bloc/assign_tag_event.dart';
 import 'package:syncrow_web/pages/spaces_management/assign_tag/bloc/assign_tag_state.dart';
+import 'package:syncrow_web/pages/spaces_management/helper/tag_helper.dart';
 import 'package:syncrow_web/utils/color_manager.dart';
 
 class AssignTagDialog extends StatelessWidget {
@@ -40,8 +41,11 @@ class AssignTagDialog extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    final List<String> locations =
+    final List<String> locations = (subspaces ?? [])
-        (subspaces ?? []).map((subspace) => subspace.subspaceName).toList();
+        .map((subspace) => subspace.subspaceName)
+        .toList()
+      ..add('Main Space');
+
     return BlocProvider(
       create: (_) => AssignTagBloc()
         ..add(InitializeTags(
@@ -93,21 +97,22 @@ class AssignTagDialog extends StatelessWidget {
                         ],
                         rows: state.tags.isEmpty
                             ? [
-                                const DataRow(cells: [
+                                DataRow(cells: [
                                   DataCell(
                                     Center(
-                                      child: Text(
+                                      child: Text('No Data Available',
-                                        'No Data Available',
+                                          style: Theme.of(context)
-                                        style: TextStyle(
+                                              .textTheme
-                                          fontSize: 14,
+                                              .bodyMedium
-                                          color: ColorsManager.lightGrayColor,
+                                              ?.copyWith(
+                                                color: ColorsManager
+                                                    .lightGrayColor,
+                                              )),
                                     ),
                                   ),
-                                    ),
+                                  const DataCell(SizedBox()),
-                                  ),
+                                  const DataCell(SizedBox()),
-                                  DataCell(SizedBox()),
+                                  const DataCell(SizedBox()),
-                                  DataCell(SizedBox()),
-                                  DataCell(SizedBox()),
                                 ])
                               ]
                             : List.generate(state.tags.length, (index) {
@@ -209,10 +214,11 @@ class AssignTagDialog extends StatelessWidget {
                       ),
                     ),
                     if (state.errorMessage != null)
-                      Text(
+                      Text(state.errorMessage!,
-                        state.errorMessage!,
+                          style: Theme.of(context)
-                        style: const TextStyle(color: ColorsManager.warningRed),
+                              .textTheme
-                      ),
+                              .bodySmall
+                              ?.copyWith(color: ColorsManager.warningRed)),
                   ],
                 ),
               ),
@@ -235,16 +241,18 @@ class AssignTagDialog extends StatelessWidget {
 
                             Navigator.of(context).pop();
 
-                            await showDialog<bool>(
+                            await showDialog(
-                              barrierDismissible: false,
                               context: context,
-                              builder: (dialogContext) => AddDeviceTypeWidget(
+                              builder: (context) => AddDeviceTypeWidget(
                                 products: products,
                                 subspaces: processedSubspaces,
-                                initialSelectedProducts: addedProducts,
+                                initialSelectedProducts: TagHelper
-                                allTags: allTags,
+                                    .createInitialSelectedProductsForTags(
+                                        processedTags, processedSubspaces),
                                 spaceName: spaceName,
                                 spaceTags: processedTags,
+                                isCreate: false,
+                                onSave: onSave,
                               ),
                             );
                           },
@@ -261,7 +269,6 @@ class AssignTagDialog extends StatelessWidget {
                         foregroundColor: ColorsManager.whiteColors,
                         onPressed: state.isSaveEnabled
                             ? () async {
-                                Navigator.of(context).pop();
                                 final updatedTags = List<Tag>.from(state.tags);
                                 final result =
                                     processTags(updatedTags, subspaces);
@@ -270,8 +277,8 @@ class AssignTagDialog extends StatelessWidget {
                                     result['updatedTags'] as List<Tag>;
                                 final processedSubspaces =
                                     result['subspaces'] as List<SubspaceModel>;
-
+                                onSave?.call(processedTags, processedSubspaces);
-                                onSave!(processedTags, processedSubspaces);
+                                Navigator.of(context).pop();
                               }
                             : null,
                         child: const Text('Save'),
@@ -307,6 +314,14 @@ class AssignTagDialog extends StatelessWidget {
     final modifiedTags = List<Tag>.from(updatedTags);
     final modifiedSubspaces = List<SubspaceModel>.from(subspaces ?? []);
 
+    if (subspaces != null) {
+      for (var subspace in subspaces) {
+        subspace.tags?.removeWhere(
+          (tag) => !modifiedTags
+              .any((updatedTag) => updatedTag.internalId == tag.internalId),
+        );
+      }
+    }
     for (var tag in modifiedTags.toList()) {
       if (modifiedSubspaces.isEmpty) continue;
 

lib/pages/spaces_management/assign_tag_models/views/assign_tag_models_dialog.dart

@@ -112,22 +112,22 @@ class AssignTagModelsDialog extends StatelessWidget {
                             ],
                             rows: state.tags.isEmpty
                                 ? [
-                                    const DataRow(cells: [
+                                    DataRow(cells: [
                                       DataCell(
                                         Center(
-                                          child: Text(
+                                          child: Text('No Devices Available',
-                                            'No Data Available',
+                                              style: Theme.of(context)
-                                            style: TextStyle(
+                                                  .textTheme
-                                              fontSize: 14,
+                                                  .bodyMedium
-                                              color:
+                                                  ?.copyWith(
-                                                  ColorsManager.lightGrayColor,
+                                                    color: ColorsManager
+                                                        .lightGrayColor,
+                                                  )),
                                         ),
                                       ),
-                                        ),
+                                      const DataCell(SizedBox()),
-                                      ),
+                                      const DataCell(SizedBox()),
-                                      DataCell(SizedBox()),
+                                      const DataCell(SizedBox()),
-                                      DataCell(SizedBox()),
-                                      DataCell(SizedBox()),
                                     ])
                                   ]
                                 : List.generate(state.tags.length, (index) {
@@ -233,11 +233,11 @@ class AssignTagModelsDialog extends StatelessWidget {
                           ),
                         ),
                         if (state.errorMessage != null)
-                          Text(
+                          Text(state.errorMessage!,
-                            state.errorMessage!,
+                              style: Theme.of(context)
-                            style: const TextStyle(
+                                  .textTheme
-                                color: ColorsManager.warningRed),
+                                  .bodySmall
-                          ),
+                                  ?.copyWith(color: ColorsManager.warningRed)),
                       ],
                     ),
                   ),
@@ -268,7 +268,7 @@ class AssignTagModelsDialog extends StatelessWidget {
                                     builder: (dialogContext) =>
                                         AddDeviceTypeModelWidget(
                                             products: products,
-                                            subspaces: subspaces,
+                                            subspaces: processedSubspaces,
                                             isCreate: false,
                                             initialSelectedProducts: TagHelper
                                                 .createInitialSelectedProducts(

lib/pages/spaces_management/create_community/view/create_community_dialog.dart

@@ -77,9 +77,7 @@ class CreateCommunityDialog extends StatelessWidget {
                                 .read<CommunityDialogBloc>()
                                 .add(ValidateCommunityNameEvent(value));
                           },
-                          style: const TextStyle(
+                          style: Theme.of(context).textTheme.bodyMedium,
-                            color: ColorsManager.blackColor,
-                          ),
                           decoration: InputDecoration(
                             hintText: 'Please enter the community name',
                             filled: true,

lib/pages/spaces_management/create_subspace/bloc/subspace_bloc.dart

@@ -82,7 +82,6 @@ class SubSpaceBloc extends Bloc<SubSpaceEvent, SubSpaceState> {
       ));
     });
 
-
     on<UpdateSubSpace>((event, emit) {
       final updatedSubSpaces = state.subSpaces.map((subSpace) {
         if (subSpace.uuid == event.updatedSubSpace.uuid) {

lib/pages/spaces_management/create_subspace/views/create_subspace_model_dialog.dart

@@ -102,12 +102,13 @@ class CreateSubSpaceDialog extends StatelessWidget {
                                           duplicateIndices.indexOf(index) != 0;
 
                                   return Chip(
-                                    label: Text(
+                                    label: Text(subSpace.subspaceName,
-                                      subSpace.subspaceName,
+                                        style: Theme.of(context)
-                                      style: const TextStyle(
+                                            .textTheme
-                                        color: ColorsManager.spaceColor,
+                                            .bodyMedium
-                                      ),
+                                            ?.copyWith(
-                                    ),
+                                                color:
+                                                    ColorsManager.spaceColor)),
                                     backgroundColor: ColorsManager.whiteColors,
                                     shape: RoundedRectangleBorder(
                                       borderRadius: BorderRadius.circular(10),
@@ -149,9 +150,12 @@ class CreateSubSpaceDialog extends StatelessWidget {
                                         hintText: state.subSpaces.isEmpty
                                             ? 'Please enter the name'
                                             : null,
-                                    hintStyle: const TextStyle(
+                                        hintStyle: Theme.of(context)
-                                        color: ColorsManager.lightGrayColor),
+                                            .textTheme
-                                  ),
+                                            .bodySmall
+                                            ?.copyWith(
+                                                color: ColorsManager
+                                                    .lightGrayColor)),
                                     onSubmitted: (value) {
                                       if (value.trim().isNotEmpty) {
                                         context.read<SubSpaceBloc>().add(
@@ -161,9 +165,8 @@ class CreateSubSpaceDialog extends StatelessWidget {
                                         textController.clear();
                                       }
                                     },
-                                  style: const TextStyle(
+                                    style:
-                                      color: ColorsManager.blackColor),
+                                        Theme.of(context).textTheme.bodyMedium),
-                                ),
                               ),
                             ],
                           ),
@@ -171,13 +174,13 @@ class CreateSubSpaceDialog extends StatelessWidget {
                         if (state.errorMessage.isNotEmpty)
                           Padding(
                             padding: const EdgeInsets.only(top: 8.0),
-                            child: Text(
+                            child: Text(state.errorMessage,
-                              state.errorMessage,
+                                style: Theme.of(context)
-                              style: const TextStyle(
+                                    .textTheme
+                                    .bodySmall
+                                    ?.copyWith(
                                       color: ColorsManager.warningRed,
-                                fontSize: 12,
+                                    )),
-                              ),
-                            ),
                           ),
                         const SizedBox(height: 16),
                         Row(
@@ -193,7 +196,9 @@ class CreateSubSpaceDialog extends StatelessWidget {
                             const SizedBox(width: 10),
                             Expanded(
                               child: DefaultButton(
-                                onPressed: () async {
+                                onPressed: (state.errorMessage.isNotEmpty)
+                                    ? null
+                                    : () async {
                                         final subSpaces = context
                                             .read<SubSpaceBloc>()
                                             .state
@@ -203,7 +208,9 @@ class CreateSubSpaceDialog extends StatelessWidget {
                                       },
                                 backgroundColor: ColorsManager.secondaryColor,
                                 borderRadius: 10,
-                                foregroundColor: ColorsManager.whiteColors,
+                                foregroundColor: state.errorMessage.isNotEmpty
+                                    ? ColorsManager.whiteColorsWithOpacity
+                                    : ColorsManager.whiteColors,
                                 child: const Text('OK'),
                               ),
                             ),

lib/pages/spaces_management/create_subspace_model/views/create_subspace_model_dialog.dart

@@ -94,12 +94,13 @@ class CreateSubSpaceModelDialog extends StatelessWidget {
                                           duplicateIndices.indexOf(index) != 0;
 
                                   return Chip(
-                                    label: Text(
+                                    label: Text(subSpace.subspaceName,
-                                      subSpace.subspaceName,
+                                        style: Theme.of(context)
-                                      style: const TextStyle(
+                                            .textTheme
+                                            .bodySmall
+                                            ?.copyWith(
                                               color: ColorsManager.spaceColor,
-                                      ),
+                                            )),
-                                    ),
                                     backgroundColor: ColorsManager.whiteColors,
                                     shape: RoundedRectangleBorder(
                                       borderRadius: BorderRadius.circular(10),
@@ -141,9 +142,12 @@ class CreateSubSpaceModelDialog extends StatelessWidget {
                                         hintText: state.subSpaces.isEmpty
                                             ? 'Please enter the name'
                                             : null,
-                                    hintStyle: const TextStyle(
+                                        hintStyle: Theme.of(context)
-                                        color: ColorsManager.lightGrayColor),
+                                            .textTheme
-                                  ),
+                                            .bodySmall!
+                                            .copyWith(
+                                                color: ColorsManager
+                                                    .lightGrayColor)),
                                     onSubmitted: (value) {
                                       if (value.trim().isNotEmpty) {
                                         context.read<SubSpaceModelBloc>().add(
@@ -154,9 +158,11 @@ class CreateSubSpaceModelDialog extends StatelessWidget {
                                         textController.clear();
                                       }
                                     },
-                                  style: const TextStyle(
+                                    style: Theme.of(context)
-                                      color: ColorsManager.blackColor),
+                                        .textTheme
-                                ),
+                                        .bodyMedium
+                                        ?.copyWith(
+                                            color: ColorsManager.blackColor)),
                               ),
                             ],
                           ),
@@ -164,13 +170,13 @@ class CreateSubSpaceModelDialog extends StatelessWidget {
                         if (state.errorMessage.isNotEmpty)
                           Padding(
                             padding: const EdgeInsets.only(bottom: 16.0),
-                            child: Text(
+                            child: Text(state.errorMessage,
-                              state.errorMessage,
+                                style: Theme.of(context)
-                              style: const TextStyle(
+                                    .textTheme
+                                    .bodySmall
+                                    ?.copyWith(
                                       color: ColorsManager.red,
-                                fontSize: 12,
+                                    )),
-                              ),
-                            ),
                           ),
                         const SizedBox(height: 16),
                         Row(

lib/pages/spaces_management/space_model/view/space_model_page.dart

@@ -97,7 +97,10 @@ class SpaceModelPage extends StatelessWidget {
           return Center(
             child: Text(
               'Error: ${state.message}',
-              style: const TextStyle(color: ColorsManager.warningRed),
+              style: Theme.of(context)
+                  .textTheme
+                  .bodySmall
+                  ?.copyWith(color: ColorsManager.warningRed),
             ),
           );
         }
@@ -109,14 +112,14 @@ class SpaceModelPage extends StatelessWidget {
   double _calculateChildAspectRatio(BuildContext context) {
     double screenWidth = MediaQuery.of(context).size.width;
     if (screenWidth > 1600) {
-      return 2;
+      return 1.5; // Decrease to make cards taller
     }
     if (screenWidth > 1200) {
-      return 3;
+      return 2.0;
     } else if (screenWidth > 800) {
-      return 3.5;
+      return 2.5;
     } else {
-      return 4.0;
+      return 3.0;
     }
   }
 

lib/pages/spaces_management/space_model/widgets/dialog/create_space_model_dialog.dart

@@ -167,7 +167,8 @@ class CreateSpaceModelDialog extends StatelessWidget {
                               onPressed: ((state.errorMessage != null &&
                                           state.errorMessage != '') ||
                                       !isNameValid)
-                                  ? () {
+                                  ? null
+                                  : () {
                                       final updatedSpaceTemplate =
                                           updatedSpaceModel.copyWith(
                                         modelName:
@@ -240,8 +241,7 @@ class CreateSpaceModelDialog extends StatelessWidget {
                                           }
                                         }
                                       }
-                                    }
+                                    },
-                                  : null,
                               backgroundColor: ColorsManager.secondaryColor,
                               borderRadius: 10,
                               foregroundColor: ((state.errorMessage != null &&

lib/pages/spaces_management/space_model/widgets/dynamic_room_widget.dart

@@ -29,7 +29,7 @@ class DynamicRoomWidget extends StatelessWidget {
       final TextPainter textPainter = TextPainter(
         text: TextSpan(
           text: subspace.subspaceName,
-          style: const TextStyle(fontSize: 16),
+          style: Theme.of(context).textTheme.bodyMedium
         ),
         textDirection: TextDirection.ltr,
       )..layout();

lib/pages/spaces_management/space_model/widgets/space_model_card_widget.dart

@@ -31,6 +31,9 @@ class SpaceModelCardWidget extends StatelessWidget {
       }
     }
 
+    return LayoutBuilder(
+      builder: (context, constraints) {
+        bool showOnlyName = constraints.maxWidth < 250;
        return Container(
           padding: const EdgeInsets.all(16.0),
           decoration: BoxDecoration(
@@ -57,6 +60,7 @@ class SpaceModelCardWidget extends StatelessWidget {
                 maxLines: 1,
                 overflow: TextOverflow.ellipsis,
               ),
+              if (!showOnlyName) ...[
                 const SizedBox(height: 10),
                 Expanded(
                   child: Row(
@@ -80,7 +84,8 @@ class SpaceModelCardWidget extends StatelessWidget {
                           ),
                         ),
                       ),
-                if (productTagCount.isNotEmpty && model.subspaceModels != null)
+                      if (productTagCount.isNotEmpty &&
+                          model.subspaceModels != null)
                         Container(
                           width: 1.0,
                           color: ColorsManager.softGray,
@@ -105,8 +110,11 @@ class SpaceModelCardWidget extends StatelessWidget {
                     ],
                   ),
                 ),
+              ]
             ],
           ),
         );
+      },
+    );
   }
 }

lib/pages/spaces_management/space_model/widgets/subspace_model_create_widget.dart

@@ -105,6 +105,7 @@ class _SubspaceModelCreateState extends State<SubspaceModelCreate> {
           isEdit: true,
           dialogTitle: dialogTitle,
           existingSubSpaces: _subspaces,
+          
           onUpdate: (subspaceModels) {
             setState(() {
               _subspaces = subspaceModels;

lib/services/space_mana_api.dart

@@ -4,7 +4,9 @@ import 'package:syncrow_web/pages/spaces_management/all_spaces/model/create_subs
 import 'package:syncrow_web/pages/spaces_management/all_spaces/model/space_model.dart';
 import 'package:syncrow_web/pages/spaces_management/all_spaces/model/space_response_model.dart';
 import 'package:syncrow_web/pages/spaces_management/space_model/models/create_space_template_body_model.dart';
+import 'package:syncrow_web/pages/spaces_management/space_model/models/space_template_model.dart';
 import 'package:syncrow_web/pages/spaces_management/space_model/models/tag_body_model.dart';
+import 'package:syncrow_web/pages/spaces_management/space_model/models/tag_update_model.dart';
 import 'package:syncrow_web/services/api/http_service.dart';
 import 'package:syncrow_web/utils/constants/api_const.dart';
 import 'package:syncrow_web/utils/constants/temp_const.dart';
@@ -154,7 +156,7 @@ class CommunitySpaceManagementApi {
             .replaceAll('{spaceId}', spaceId)
             .replaceAll('{projectId}', TempConst.projectId),
         expectedResponseModel: (json) {
-          return SpaceModel.fromJson(json);
+          return SpaceModel.fromJson(json['data']);
         },
       );
       return response;
@@ -210,7 +212,7 @@ class CommunitySpaceManagementApi {
     }
   }
 
-  Future<SpaceModel?> updateSpace({
+  Future<bool> updateSpace({
     required String communityId,
     required spaceId,
     required String name,
@@ -219,6 +221,8 @@ class CommunitySpaceManagementApi {
     String? direction,
     bool isPrivate = false,
     required Offset position,
+    List<TagModelUpdate>? tags,
+    List<UpdateSubspaceTemplateModel>? subspaces,
   }) async {
     try {
       final body = {
@@ -228,6 +232,8 @@ class CommunitySpaceManagementApi {
         'y': position.dy,
         'direction': direction,
         'icon': icon,
+        'subspace': subspaces,
+        'tags': tags,
       };
       if (parentId != null) {
         body['parentUuid'] = parentId;
@@ -240,13 +246,13 @@ class CommunitySpaceManagementApi {
             .replaceAll('{projectId}', TempConst.projectId),
         body: body,
         expectedResponseModel: (json) {
-          return SpaceModel.fromJson(json['data']);
+          return json['success'] ?? false;
         },
       );
       return response;
     } catch (e) {
-      debugPrint('Error creating space: $e');
+      debugPrint('Error updating space: $e');
-      return null;
+      return false;
     }
   }