Remove unused parameters from CreateSubSpaceDialog constructor

lib/common/widgets/empty_search_result_widget.dart

@@ -1,26 +0,0 @@
-import 'package:flutter/material.dart';
-import 'package:syncrow_web/utils/color_manager.dart';
-import 'package:syncrow_web/utils/extension/build_context_x.dart';
-
-class EmptySearchResultWidget extends StatelessWidget {
-  const EmptySearchResultWidget({
-    this.message = 'No results found',
-    super.key,
-  });
-
-  final String message;
-
-  @override
-  Widget build(BuildContext context) {
-    return Center(
-      child: Text(
-        message,
-        textAlign: TextAlign.center,
-        style: context.textTheme.bodySmall?.copyWith(
-          color: ColorsManager.lightGreyColor,
-          fontWeight: FontWeight.w400,
-        ),
-      ),
-    );
-  }
-}

lib/common/widgets/sidebar_communities_list.dart

@@ -1,53 +0,0 @@
-import 'package:flutter/material.dart';
-import 'package:syncrow_web/pages/spaces_management/all_spaces/model/community_model.dart';
-import 'package:syncrow_web/utils/extension/build_context_x.dart';
-
-class SidebarCommunitiesList extends StatelessWidget {
-  const SidebarCommunitiesList({
-    required this.communities,
-    required this.itemBuilder,
-    required this.scrollController,
-    required this.onScrollToEnd,
-    super.key,
-  });
-
-  final List<CommunityModel> communities;
-  final Widget Function(BuildContext context, int index) itemBuilder;
-  final ScrollController scrollController;
-  final void Function() onScrollToEnd;
-
-  bool _onNotification(ScrollEndNotification notification) {
-    final hasReachedEnd = notification.metrics.extentAfter == 0;
-    if (hasReachedEnd) {
-      onScrollToEnd.call();
-      return true;
-    }
-
-    return false;
-  }
-
-  @override
-  Widget build(BuildContext context) {
-    return SingleChildScrollView(
-      scrollDirection: Axis.horizontal,
-      child: SizedBox(
-        width: context.screenWidth * 0.5,
-        child: Scrollbar(
-          scrollbarOrientation: ScrollbarOrientation.left,
-          thumbVisibility: true,
-          controller: scrollController,
-          child: NotificationListener<ScrollEndNotification>(
-            onNotification: _onNotification,
-            child: ListView.builder(
-              shrinkWrap: true,
-              padding: const EdgeInsetsDirectional.only(start: 16),
-              itemCount: communities.length,
-              controller: scrollController,
-              itemBuilder: itemBuilder,
-            ),
-          ),
-        ),
-      ),
-    );
-  }
-}

lib/pages/routines/view/routines_view.dart

@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
-import 'package:syncrow_web/pages/routines/bloc/create_routine_bloc/create_routine_bloc.dart';
 import 'package:syncrow_web/pages/routines/bloc/create_routine_bloc/create_routine_event.dart';
+import 'package:syncrow_web/pages/routines/bloc/create_routine_bloc/create_routine_bloc.dart';
 import 'package:syncrow_web/pages/routines/bloc/routine_bloc/routine_bloc.dart';
 import 'package:syncrow_web/pages/routines/create_new_routines/create_new_routines.dart';
 import 'package:syncrow_web/pages/routines/view/create_new_routine_view.dart';
@@ -9,7 +9,6 @@ import 'package:syncrow_web/pages/routines/widgets/main_routine_view/fetch_routi
 import 'package:syncrow_web/pages/routines/widgets/main_routine_view/routine_view_card.dart';
 import 'package:syncrow_web/pages/space_tree/view/space_tree_view.dart';
 import 'package:syncrow_web/utils/color_manager.dart';
-import 'package:syncrow_web/utils/extension/build_context_x.dart';
 
 class RoutinesView extends StatefulWidget {
   const RoutinesView({super.key});
@@ -28,10 +27,9 @@ class _RoutinesViewState extends State<RoutinesView> {
     if (result == null) return;
     final communityId = result['community'];
     final spaceId = result['space'];
-    final bloc = BlocProvider.of<CreateRoutineBloc>(context);
+    final _bloc = BlocProvider.of<CreateRoutineBloc>(context);
     final routineBloc = context.read<RoutineBloc>();
-    bloc.add(
+    _bloc.add(SaveCommunityIdAndSpaceIdEvent(communityID: communityId, spaceID: spaceId));
-        SaveCommunityIdAndSpaceIdEvent(communityID: communityId, spaceID: spaceId));
     await Future.delayed(const Duration(seconds: 1));
     routineBloc.add(const CreateNewRoutineViewEvent(createRoutineView: true));
   }
@@ -56,15 +54,13 @@ class _RoutinesViewState extends State<RoutinesView> {
             ),
             Expanded(
               flex: 4,
-              child: SizedBox(
+              child: ListView(
-                height: context.screenHeight,
+                children: [
-                width: context.screenWidth,
+                  Container(
-                child: SingleChildScrollView(
+                    padding: const EdgeInsets.all(16),
-                  padding: const EdgeInsetsDirectional.all(16),
+                    height: MediaQuery.sizeOf(context).height,
                     child: Column(
                       crossAxisAlignment: CrossAxisAlignment.start,
-                    mainAxisAlignment: MainAxisAlignment.start,
-                    spacing: 16,
                       children: [
                         Text(
                           "Create New Routines",
@@ -73,6 +69,7 @@ class _RoutinesViewState extends State<RoutinesView> {
                                 fontWeight: FontWeight.bold,
                               ),
                         ),
+                        const SizedBox(height: 10),
                         RoutineViewCard(
                           isLoading: false,
                           onChanged: (v) {},
@@ -87,10 +84,13 @@ class _RoutinesViewState extends State<RoutinesView> {
                           icon: Icons.add,
                           textString: '',
                         ),
-                      const FetchRoutineScenesAutomation(),
+                        const SizedBox(height: 15),
+                        const Expanded(child: FetchRoutineScenesAutomation()),
                       ],
                     ),
                   ),
+                  const SizedBox(height: 50),
+                ],
               ),
             )
           ],

lib/pages/routines/widgets/main_routine_view/fetch_routine_scenes_automation.dart

@@ -8,121 +8,67 @@ import 'package:syncrow_web/utils/constants/assets.dart';
 import 'package:syncrow_web/utils/extension/build_context_x.dart';
 import 'package:syncrow_web/utils/helpers/responsice_layout_helper/responsive_layout_helper.dart';
 
-class FetchRoutineScenesAutomation extends StatelessWidget
+class FetchRoutineScenesAutomation extends StatefulWidget {
-    with HelperResponsiveLayout {
   const FetchRoutineScenesAutomation({super.key});
 
+  @override
+  State<FetchRoutineScenesAutomation> createState() =>
+      _FetchRoutineScenesState();
+}
+
+class _FetchRoutineScenesState extends State<FetchRoutineScenesAutomation>
+    with HelperResponsiveLayout {
+  @override
+  void initState() {
+    super.initState();
+  }
+
   @override
   Widget build(BuildContext context) {
     return BlocBuilder<RoutineBloc, RoutineState>(
       builder: (context, state) {
-        if (state.isLoading) return const Center(child: CircularProgressIndicator());
+        return state.isLoading
-
+            ? const Center(
-        return SingleChildScrollView(
+                child: CircularProgressIndicator(),
+              )
+            : SingleChildScrollView(
                 child: Padding(
                   padding: const EdgeInsets.symmetric(vertical: 16.0),
                   child: Column(
                     crossAxisAlignment: CrossAxisAlignment.start,
                     mainAxisSize: MainAxisSize.min,
                     children: [
-                _buildListTitle(context, "Scenes (Tab to Run)"),
+                      Text(
-                const SizedBox(height: 10),
+                        "Scenes (Tab to Run)",
-                Visibility(
+                        style: Theme.of(context).textTheme.titleLarge?.copyWith(
-                  visible: state.scenes.isNotEmpty,
+                              color: ColorsManager.grayColor,
-                  replacement: _buildEmptyState(context, "No scenes found"),
+                              fontWeight: FontWeight.bold,
-                  child: SizedBox(
-                    height: 200,
-                    child: _buildScenes(state),
                             ),
                       ),
                       const SizedBox(height: 10),
-                _buildListTitle(context, "Automations"),
+                      if (state.scenes.isEmpty)
-                const SizedBox(height: 3),
+                        Text(
-                Visibility(
+                          "No scenes found",
-                  visible: state.automations.isNotEmpty,
+                          style: context.textTheme.bodyMedium?.copyWith(
-                  replacement: _buildEmptyState(context, "No automations found"),
+                            color: ColorsManager.grayColor,
-                  child: SizedBox(
+                          ),
+                        ),
+                      if (state.scenes.isNotEmpty)
+                        SizedBox(
                           height: 200, 
-                    child: _buildAutomations(state),
+                          child: ListView.builder(
-                  ),
+                              shrinkWrap: true,
-                )
-              ],
-            ),
-          ),
-        );
-      },
-    );
-  }
-
-  Widget _buildAutomations(RoutineState state) {
-    return ListView.builder(
-      scrollDirection: Axis.horizontal,
-      itemCount: state.automations.length,
-      itemBuilder: (context, index) {
-        final isLoading = state.automations.contains(state.automations[index].id);
-
-        return Column(
-          children: [
-            Padding(
-              padding: EdgeInsets.only(
-                right: isSmallScreenSize(context) ? 4.0 : 8.0,
-              ),
-              child: RoutineViewCard(
-                isLoading: isLoading,
-                onChanged: (v) {
-                  context.read<RoutineBloc>().add(
-                        UpdateAutomationStatus(
-                          automationId: state.automations[index].id,
-                          automationStatusUpdate: AutomationStatusUpdate(
-                            spaceUuid: state.automations[index].spaceId,
-                            isEnable: v,
-                          ),
-                          communityId: state.automations[index].communityId,
-                        ),
-                      );
-                },
-                status: state.automations[index].status,
-                communityId: '',
-                spaceId: state.automations[index].spaceId,
-                sceneId: '',
-                automationId: state.automations[index].id,
-                cardType: 'automations',
-                spaceName: state.automations[index].spaceName,
-                onTap: () {
-                  BlocProvider.of<RoutineBloc>(context).add(
-                    const CreateNewRoutineViewEvent(
-                      createRoutineView: true,
-                    ),
-                  );
-                  context.read<RoutineBloc>().add(
-                        GetAutomationDetails(
-                          automationId: state.automations[index].id,
-                          isAutomation: true,
-                          isUpdate: true,
-                        ),
-                      );
-                },
-                textString: state.automations[index].name,
-                icon: state.automations[index].icon ?? Assets.automation,
-              ),
-            ),
-          ],
-        );
-      },
-    );
-  }
-
-  Widget _buildScenes(RoutineState state) {
-    return ListView.builder(
                               scrollDirection: Axis.horizontal,
                               itemCount: state.scenes.length,
                               itemBuilder: (context, index) {
                                 final scene = state.scenes[index];
-          final isLoading = state.loadingSceneId == scene.id;
+                                final isLoading =
+                                    state.loadingSceneId == scene.id;
 
                                 return Padding(
                                   padding: EdgeInsets.only(
-              right: isSmallScreenSize(context) ? 4.0 : 8.0,
+                                    right:
+                                        isSmallScreenSize(context) ? 4.0 : 8.0,
                                   ),
                                   child: Column(
                                     children: [
@@ -132,58 +78,141 @@ class FetchRoutineScenesAutomation extends StatelessWidget
                                           context.read<RoutineBloc>().add(
                                               SceneTrigger(
                                                   sceneId: scene.id,
-                            name: scene.name,
+                                                  name: scene.name));
-                          ),
-                        );
                                         },
                                         status: state.scenes[index].status,
-                  communityId: state.scenes[index].communityId,
+                                        communityId:
+                                            state.scenes[index].communityId ??
+                                                '',
                                         spaceId: state.scenes[index].spaceId,
-                  sceneId: state.scenes[index].sceneTuyaId!,
+                                        sceneId:
+                                            state.scenes[index].sceneTuyaId!,
                                         automationId: state.scenes[index].id,
                                         cardType: 'scenes',
-                  spaceName: state.scenes[index].spaceName,
+                                        spaceName:
+                                            state.scenes[index].spaceName,
                                         onTap: () {
-                    BlocProvider.of<RoutineBloc>(context).add(
+                                          BlocProvider.of<RoutineBloc>(context)
+                                              .add(
                                             const CreateNewRoutineViewEvent(
-                        createRoutineView: true,
+                                                createRoutineView: true),
-                      ),
                                           );
                                           context.read<RoutineBloc>().add(
                                                 GetSceneDetails(
-                            sceneId: state.scenes[index].id,
+                                                  sceneId:
+                                                      state.scenes[index].id,
                                                   isTabToRun: true,
                                                   isUpdate: true,
                                                 ),
                                               );
                                         },
                                         textString: state.scenes[index].name,
-                  icon: state.scenes[index].icon ?? Assets.logoHorizontal,
+                                        icon: state.scenes[index].icon ??
+                                            Assets.logoHorizontal,
                                         isFromScenes: true,
-                  iconInBytes: state.scenes[index].iconInBytes,
+                                        iconInBytes:
+                                            state.scenes[index].iconInBytes,
                                       ),
                                     ],
                                   ),
                                 );
-        });
+                              }),
-  }
+                        ),
-
+                      const SizedBox(height: 10),
-  Widget _buildListTitle(BuildContext context, String title) {
+                      Text(
-    return Text(
+                        "Automations",
-      title,
                         style: Theme.of(context).textTheme.titleLarge?.copyWith(
                               color: ColorsManager.grayColor,
                               fontWeight: FontWeight.bold,
                             ),
-    );
+                      ),
-  }
+                      const SizedBox(height: 3),
-
+                      if (state.automations.isEmpty)
-  Widget _buildEmptyState(BuildContext context, String title) {
+                        Text(
-    return Text(
+                          "No automations found",
-      title,
                           style: context.textTheme.bodyMedium?.copyWith(
                             color: ColorsManager.grayColor,
                           ),
+                        ),
+                      if (state.automations.isNotEmpty)
+                        SizedBox(
+                          height: 200, 
+
+                          child: ListView.builder(
+                              shrinkWrap: true,
+                              scrollDirection: Axis.horizontal,
+                              itemCount: state.automations.length,
+                              itemBuilder: (context, index) {
+                                final isLoading = state.automations!
+                                    .contains(state.automations[index].id);
+
+                                return Column(
+                                  children: [
+                                    Padding(
+                                      padding: EdgeInsets.only(
+                                        right: isSmallScreenSize(context)
+                                            ? 4.0
+                                            : 8.0,
+                                      ),
+                                      child: RoutineViewCard(
+                                        isLoading: isLoading,
+                                        onChanged: (v) {
+                                          context.read<RoutineBloc>().add(
+                                                UpdateAutomationStatus(
+                                                  automationId: state
+                                                      .automations[index].id,
+                                                  automationStatusUpdate:
+                                                      AutomationStatusUpdate(
+                                                          spaceUuid: state
+                                                              .automations[
+                                                                  index]
+                                                              .spaceId,
+                                                          isEnable: v),
+                                                  communityId: state
+                                                      .automations[index]
+                                                      .communityId,
+                                                ),
+                                              );
+                                        },
+                                        status: state.automations[index].status,
+                                        communityId: '',
+                                        spaceId:
+                                            state.automations[index].spaceId,
+                                        sceneId: '',
+                                        automationId:
+                                            state.automations[index].id,
+                                        cardType: 'automations',
+                                        spaceName:
+                                            state.automations[index].spaceName,
+                                        onTap: () {
+                                          BlocProvider.of<RoutineBloc>(context)
+                                              .add(
+                                            const CreateNewRoutineViewEvent(
+                                                createRoutineView: true),
+                                          );
+                                          context.read<RoutineBloc>().add(
+                                                GetAutomationDetails(
+                                                    automationId: state
+                                                        .automations[index].id,
+                                                    isAutomation: true,
+                                                    isUpdate: true),
+                                              );
+                                        },
+                                        textString:
+                                            state.automations[index].name,
+                                        icon: state.automations[index].icon ??
+                                            Assets.automation,
+                                      ),
+                                    ),
+                                  ],
+                                );
+                              }),
+                        ),
+                    ],
+                  ),
+                ),
+              );
+      },
     );
   }
 }

lib/pages/routines/widgets/main_routine_view/routine_view_card.dart

@@ -1,5 +1,4 @@
 import 'dart:async';
-
 import 'package:flutter/cupertino.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
@@ -67,6 +66,7 @@ class _RoutineViewCardState extends State<RoutineViewCard> {
 
   @override
   Widget build(BuildContext context) {
+    // Use widget.<mixinMethod> instead of just <mixinMethod>
     final double cardWidth = widget.isSmallScreenSize(context)
         ? 120
         : widget.isMediumScreenSize(context)
@@ -127,23 +127,22 @@ class _RoutineViewCardState extends State<RoutineViewCard> {
                             )
                           else
                             CupertinoSwitch(
-                              activeTrackColor: ColorsManager.primaryColor,
+                              activeColor: ColorsManager.primaryColor,
                               value: widget.status == 'enable',
                               onChanged: widget.onChanged,
                             )
                         ],
                       )
                     : const SizedBox(),
-                Column(
+                InkWell(
+                  onTap: widget.onTap,
+                  child: Column(
                     children: [
                       Center(
-                      child: InkWell(
-                        customBorder: const CircleBorder(),
-                        onTap: widget.onTap,
                         child: Container(
                           decoration: BoxDecoration(
                             color: ColorsManager.graysColor,
-                            shape: BoxShape.circle,
+                            borderRadius: BorderRadius.circular(120),
                             border: Border.all(
                               color: ColorsManager.greyColor,
                               width: 2.0,
@@ -159,8 +158,7 @@ class _RoutineViewCardState extends State<RoutineViewCard> {
                                       height: iconSize,
                                       width: iconSize,
                                       fit: BoxFit.contain,
-                                      errorBuilder: (context, error, stackTrace) =>
+                                      errorBuilder: (context, error, stackTrace) => Image.asset(
-                                          Image.asset(
                                         Assets.logo,
                                         height: iconSize,
                                         width: iconSize,
@@ -173,8 +171,7 @@ class _RoutineViewCardState extends State<RoutineViewCard> {
                                       width: iconSize,
                                       fit: BoxFit.contain,
                                     )
-                              : (widget.icon is String &&
+                              : (widget.icon is String && widget.icon.endsWith('.svg'))
-                                      widget.icon.endsWith('.svg'))
                                   ? SvgPicture.asset(
                                       height: iconSize,
                                       width: iconSize,
@@ -184,10 +181,7 @@ class _RoutineViewCardState extends State<RoutineViewCard> {
                                   : Icon(
                                       widget.icon,
                                       color: ColorsManager.dialogBlueTitle,
-                                      size: widget.isSmallScreenSize(context)
+                                      size: widget.isSmallScreenSize(context) ? 30 : 40,
-                                          ? 30
-                                          : 40,
-                                    ),
                                     ),
                         ),
                       ),
@@ -222,8 +216,7 @@ class _RoutineViewCardState extends State<RoutineViewCard> {
                                     maxLines: 1,
                                     style: context.textTheme.bodySmall?.copyWith(
                                       color: ColorsManager.blackColor,
-                                    fontSize:
+                                      fontSize: widget.isSmallScreenSize(context) ? 10 : 12,
-                                        widget.isSmallScreenSize(context) ? 10 : 12,
                                     ),
                                   ),
                                 ],
@@ -233,6 +226,7 @@ class _RoutineViewCardState extends State<RoutineViewCard> {
                       ),
                     ],
                   ),
+                ),
               ],
             ),
           ),

lib/pages/space_tree/view/custom_expansion.dart

@@ -8,20 +8,19 @@ class CustomExpansionTileSpaceTree extends StatelessWidget {
   final bool isSelected;
   final bool isSoldCheck;
   final bool isExpanded;
-  final void Function()? onExpansionChanged;
+  final Function? onExpansionChanged;
-  final void Function()? onItemSelected;
+  final Function? onItemSelected;
 
-  const CustomExpansionTileSpaceTree({
+  const CustomExpansionTileSpaceTree(
-    required this.isSelected,
+      {super.key,
-    required this.title,
       this.spaceId,
+      required this.title,
       this.children,
+      this.isExpanded = false,
       this.onExpansionChanged,
       this.onItemSelected,
-    this.isExpanded = false,
+      required this.isSelected,
-    this.isSoldCheck = false,
+      this.isSoldCheck = false});
-    super.key,
-  });
 
   @override
   Widget build(BuildContext context) {
@@ -31,30 +30,50 @@ class CustomExpansionTileSpaceTree extends StatelessWidget {
           children: [
             Checkbox(
               value: isSoldCheck ? null : isSelected,
-              onChanged: (value) => onItemSelected?.call(),
+              onChanged: (bool? value) {
+                if (onItemSelected != null) {
+                  onItemSelected!();
+                }
+              },
               tristate: true,
-              side: WidgetStateBorderSide.resolveWith(
+              side: WidgetStateBorderSide.resolveWith((states) {
-                (states) => const BorderSide(color: ColorsManager.grayBorder),
+                return const BorderSide(color: ColorsManager.grayBorder);
-              ),
+              }),
               fillColor: WidgetStateProperty.resolveWith((states) {
                 if (states.contains(WidgetState.selected)) {
                   return ColorsManager.blue1;
-                }
+                } else {
-
                   return ColorsManager.checkBoxFillColor;
+                }
               }),
               checkColor: ColorsManager.whiteColors,
             ),
-            _buildExpansionIcon(),
+            if (children != null && children!.isNotEmpty)
+              InkWell(
+                onTap: () {
+                  if (onExpansionChanged != null) {
+                    onExpansionChanged!();
+                  }
+                },
+                child: Icon(
+                  isExpanded ? Icons.keyboard_arrow_down : Icons.keyboard_arrow_right,
+                  color: ColorsManager.lightGrayColor,
+                  size: 16.0,
+                ),
+              ),
             Expanded(
               child: GestureDetector(
-                onTap: onItemSelected,
+                onTap: () {
+                  if (onItemSelected != null) {
+                    onItemSelected!();
+                  }
+                },
                 child: Text(
                   _capitalizeFirstLetter(title),
                   style: Theme.of(context).textTheme.bodySmall!.copyWith(
                         color: isSelected
-                            ? ColorsManager.blackColor
+                            ? ColorsManager.blackColor // Change color to black when selected
-                            : ColorsManager.lightGrayColor,
+                            : ColorsManager.lightGrayColor, // Gray when not selected
                         fontWeight: FontWeight.w400,
                       ),
                 ),
@@ -73,20 +92,6 @@ class CustomExpansionTileSpaceTree extends StatelessWidget {
     );
   }
 
-  Widget _buildExpansionIcon() {
-    return Visibility(
-      visible: children != null && children!.isNotEmpty,
-      child: InkWell(
-        onTap: onExpansionChanged,
-        child: Icon(
-          isExpanded ? Icons.keyboard_arrow_down : Icons.keyboard_arrow_right,
-          color: ColorsManager.lightGrayColor,
-          size: 16.0,
-        ),
-      ),
-    );
-  }
-
   String _capitalizeFirstLetter(String text) {
     if (text.isEmpty) return text;
     return text[0].toUpperCase() + text.substring(1);

lib/pages/space_tree/view/space_tree_view.dart

@@ -2,7 +2,6 @@ import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:flutter_svg/svg.dart';
 import 'package:syncrow_web/common/widgets/search_bar.dart';
-import 'package:syncrow_web/common/widgets/sidebar_communities_list.dart';
 import 'package:syncrow_web/pages/space_tree/bloc/space_tree_bloc.dart';
 import 'package:syncrow_web/pages/space_tree/bloc/space_tree_event.dart';
 import 'package:syncrow_web/pages/space_tree/bloc/space_tree_state.dart';
@@ -24,13 +23,7 @@ class SpaceTreeView extends StatefulWidget {
 }
 
 class _SpaceTreeViewState extends State<SpaceTreeView> {
-  late final ScrollController _scrollController;
+  final ScrollController _scrollController = ScrollController();
-
-  @override
-  void initState() {
-    _scrollController = ScrollController();
-    super.initState();
-  }
 
   @override
   void dispose() {
@@ -41,27 +34,23 @@ class _SpaceTreeViewState extends State<SpaceTreeView> {
   @override
   Widget build(BuildContext context) {
     return BlocBuilder<SpaceTreeBloc, SpaceTreeState>(builder: (context, state) {
-      final communities = state.searchQuery.isNotEmpty
+      List<CommunityModel> list =
-          ? state.filteredCommunity
+          state.searchQuery.isNotEmpty ? state.filteredCommunity : state.communityList;
-          : state.communityList;
       return Container(
         height: MediaQuery.sizeOf(context).height,
         decoration: widget.isSide == true
-            ? subSectionContainerDecoration.copyWith(
+            ? subSectionContainerDecoration.copyWith(color: ColorsManager.whiteColors)
-                color: ColorsManager.whiteColors)
             : const BoxDecoration(color: ColorsManager.whiteColors),
         child: state is SpaceTreeLoadingState
             ? const Center(child: CircularProgressIndicator())
             : Column(
                 children: [
-                  if (widget.isSide == true)
+                  widget.isSide == true
-                    Container(
+                      ? Container(
                           decoration: const BoxDecoration(
                             color: ColorsManager.circleRolesBackground,
                             borderRadius: BorderRadius.only(
-                          topRight: Radius.circular(20),
+                                topRight: Radius.circular(20), topLeft: Radius.circular(20)),
-                          topLeft: Radius.circular(20),
-                        ),
                           ),
                           child: Padding(
                             padding: const EdgeInsets.all(8.0),
@@ -70,23 +59,15 @@ class _SpaceTreeViewState extends State<SpaceTreeView> {
                                 Expanded(
                                   child: Container(
                                     decoration: BoxDecoration(
-                                  borderRadius: const BorderRadius.all(
+                                        borderRadius: const BorderRadius.all(Radius.circular(20)),
-                                    Radius.circular(20),
+                                        border: Border.all(color: ColorsManager.grayBorder)),
-                                  ),
-                                  border: Border.all(
-                                    color: ColorsManager.grayBorder,
-                                  ),
-                                ),
                                     child: TextFormField(
-                                  style: context.textTheme.bodyMedium?.copyWith(
+                                      style: context.textTheme.bodyMedium
-                                    color: ColorsManager.blackColor,
+                                          ?.copyWith(color: ColorsManager.blackColor),
-                                  ),
+                                      onChanged: (value) {
-                                  onChanged: (value) =>
+                                        context.read<SpaceTreeBloc>().add(SearchQueryEvent(value));
-                                      context.read<SpaceTreeBloc>().add(
+                                      },
-                                            SearchQueryEvent(value),
+                                      decoration: textBoxDecoration(radios: 20)!.copyWith(
-                                          ),
-                                  decoration:
-                                      textBoxDecoration(radios: 20)?.copyWith(
                                         fillColor: Colors.white,
                                         suffixIcon: Padding(
                                           padding: const EdgeInsets.only(right: 16),
@@ -96,12 +77,10 @@ class _SpaceTreeViewState extends State<SpaceTreeView> {
                                             height: 24,
                                           ),
                                         ),
-                                    hintStyle:
+                                        hintStyle: context.textTheme.bodyMedium?.copyWith(
-                                        context.textTheme.bodyMedium?.copyWith(
                                             fontWeight: FontWeight.w400,
                                             fontSize: 12,
-                                      color: ColorsManager.textGray,
+                                            color: ColorsManager.textGray),
-                                    ),
                                       ),
                                     ),
                                   ),
@@ -110,92 +89,170 @@ class _SpaceTreeViewState extends State<SpaceTreeView> {
                             ),
                           ),
                         )
-                  else
+                      : CustomSearchBar(
-                    CustomSearchBar(
+                          onSearchChanged: (query) {
-                      onSearchChanged: (query) => context.read<SpaceTreeBloc>().add(
+                            context.read<SpaceTreeBloc>().add(SearchQueryEvent(query));
-                            SearchQueryEvent(query),
+                          },
-                          ),
                         ),
                   const SizedBox(height: 16),
                   Expanded(
                     child: state.isSearching
                         ? const Center(child: CircularProgressIndicator())
-                        : SidebarCommunitiesList(
+                        : ListView(
-                            onScrollToEnd: () => context.read<SpaceTreeBloc>().add(
+                            shrinkWrap: true,
-                                  PaginationEvent(
+                            scrollDirection: Axis.horizontal,
-                                    state.paginationModel,
+                            children: [
-                                    state.communityList,
+                              Container(
+                                width: MediaQuery.sizeOf(context).width * 0.5,
+                                padding: const EdgeInsets.all(8.0),
+                                child: list.isEmpty
+                                    ? Center(
+                                        child: Text(
+                                          'No results found',
+                                          style: Theme.of(context).textTheme.bodySmall!.copyWith(
+                                                color: ColorsManager.lightGrayColor,
+                                                fontWeight: FontWeight.w400,
                                               ),
                                         ),
-                            scrollController: _scrollController,
+                                      )
-                            communities: communities,
+                                    : Scrollbar(
+                                        scrollbarOrientation: ScrollbarOrientation.left,
+                                        thumbVisibility: true,
+                                        controller: _scrollController,
+                                        child: NotificationListener(
+                                          onNotification: (notification) {
+                                            if (notification is ScrollEndNotification &&
+                                                notification.metrics.extentAfter == 0) {
+                                              // If the user has reached the end of the list Load more data
+                                              context.read<SpaceTreeBloc>().add(PaginationEvent(
+                                                  state.paginationModel, state.communityList));
+                                            }
+                                            return false;
+                                          },
+                                          child: Padding(
+                                            padding: const EdgeInsets.only(left: 16),
+                                            child: ListView.builder(
+                                                shrinkWrap: true,
+                                                itemCount: list.length,
+                                                controller: _scrollController,
                                                 itemBuilder: (context, index) {
                                                   return CustomExpansionTileSpaceTree(
-                                title: communities[index].name,
+                                                    title: list[index].name,
                                                     isSelected: state.selectedCommunities
-                                    .contains(communities[index].uuid),
+                                                        .contains(list[index].uuid),
                                                     isSoldCheck: state.selectedCommunities
-                                    .contains(communities[index].uuid),
+                                                        .contains(list[index].uuid),
-                                onExpansionChanged: () =>
+                                                    onExpansionChanged: () {
                                                       context.read<SpaceTreeBloc>().add(
-                                          OnCommunityExpanded(
+                                                          OnCommunityExpanded(list[index].uuid));
-                                            communities[index].uuid,
+                                                    },
-                                          ),
+                                                    isExpanded: state.expandedCommunities
-                                        ),
+                                                        .contains(list[index].uuid),
-                                isExpanded: state.expandedCommunities.contains(
-                                  communities[index].uuid,
-                                ),
                                                     onItemSelected: () {
                                                       context.read<SpaceTreeBloc>().add(
-                                        OnCommunitySelected(
+                                                          OnCommunitySelected(list[index].uuid,
-                                          communities[index].uuid,
+                                                              list[index].spaces));
-                                          communities[index].spaces,
-                                        ),
-                                      );
                                                       widget.onSelect();
                                                     },
-                                children: communities[index].spaces.map(
+                                                    children: list[index].spaces.map((space) {
-                                  (space) {
                                                       return CustomExpansionTileSpaceTree(
                                                         title: space.name,
-                                      isExpanded:
+                                                        isExpanded: state.expandedSpaces
-                                          state.expandedSpaces.contains(space.uuid),
+                                                            .contains(space.uuid),
                                                         onItemSelected: () {
                                                           context.read<SpaceTreeBloc>().add(
                                                               OnSpaceSelected(
-                                                communities[index],
+                                                                  list[index],
                                                                   space.uuid ?? '',
-                                                space.children,
+                                                                  space.children));
-                                              ),
-                                            );
                                                           widget.onSelect();
                                                         },
-                                      onExpansionChanged: () =>
+                                                        onExpansionChanged: () {
                                                           context.read<SpaceTreeBloc>().add(
-                                                OnSpaceExpanded(
+                                                              OnSpaceExpanded(list[index].uuid,
-                                                  communities[index].uuid,
+                                                                  space.uuid ?? ''));
-                                                  space.uuid ?? '',
+                                                        },
-                                                ),
-                                              ),
                                                         isSelected: state.selectedSpaces
                                                                 .contains(space.uuid) ||
                                                             state.soldCheck.contains(space.uuid),
                                                         isSoldCheck:
                                                             state.soldCheck.contains(space.uuid),
                                                         children: _buildNestedSpaces(
-                                        context,
+                                                            context, state, space, list[index]),
-                                        state,
+                                                      );
-                                        space,
+                                                    }).toList(),
-                                        communities[index],
+                                                  );
+                                                }),
                                           ),
-                                    );
+                                        ),
-                                  },
+                                      ),
-                                ).toList(),
+                              ),
-                              );
+                            ],
-                            },
                           ),
                   ),
                   if (state.paginationIsLoading) const CircularProgressIndicator(),
+                  // Expanded(
+                  //   child: Padding(
+                  //     padding: const EdgeInsets.all(8.0),
+                  //     child: list.isEmpty
+                  //         ? Center(
+                  //             child: Text(
+                  //               'No results found',
+                  //               style: Theme.of(context).textTheme.bodySmall!.copyWith(
+                  //                     color: ColorsManager.lightGrayColor, // Gray when not selected
+                  //                     fontWeight: FontWeight.w400,
+                  //                   ),
+                  //             ),
+                  //           )
+                  //         : ListView(
+                  //             shrinkWrap: true,
+                  //             children: list
+                  //                 .map(
+                  //                   (community) => CustomExpansionTileSpaceTree(
+                  //                     title: community.name,
+                  //                     isSelected:
+                  //                         state.selectedCommunities.contains(community.uuid),
+                  //                     isSoldCheck:
+                  //                         state.selectedCommunities.contains(community.uuid),
+                  //                     onExpansionChanged: () {
+                  //                       context
+                  //                           .read<SpaceTreeBloc>()
+                  //                           .add(OnCommunityExpanded(community.uuid));
+                  //                     },
+                  //                     isExpanded:
+                  //                         state.expandedCommunities.contains(community.uuid),
+                  //                     onItemSelected: () {
+                  //                       context.read<SpaceTreeBloc>().add(
+                  //                           OnCommunitySelected(community.uuid, community.spaces));
+
+                  //                       onSelect();
+                  //                     },
+                  //                     children: community.spaces.map((space) {
+                  //                       return CustomExpansionTileSpaceTree(
+                  //                         title: space.name,
+                  //                         isExpanded: state.expandedSpaces.contains(space.uuid),
+                  //                         onItemSelected: () {
+                  //                           context.read<SpaceTreeBloc>().add(OnSpaceSelected(
+                  //                               community.uuid, space.uuid ?? '', space.children));
+                  //                           onSelect();
+                  //                         },
+                  //                         onExpansionChanged: () {
+                  //                           context.read<SpaceTreeBloc>().add(
+                  //                               OnSpaceExpanded(community.uuid, space.uuid ?? ''));
+                  //                         },
+                  //                         isSelected: state.selectedSpaces.contains(space.uuid) ||
+                  //                             state.soldCheck.contains(space.uuid),
+                  //                         isSoldCheck: state.soldCheck.contains(space.uuid),
+                  //                         children: _buildNestedSpaces(
+                  //                             context, state, space, community.uuid),
+                  //                       );
+                  //                     }).toList(),
+                  //                   ),
+                  //                 )
+                  //                 .toList(),
+                  //           ),
+                  //   ),
+                  // ),
                 ],
               ),
       );
@@ -203,28 +260,22 @@ class _SpaceTreeViewState extends State<SpaceTreeView> {
   }
 
   List<Widget> _buildNestedSpaces(
-    BuildContext context,
+      BuildContext context, SpaceTreeState state, SpaceModel space, CommunityModel community) {
-    SpaceTreeState state,
-    SpaceModel space,
-    CommunityModel community,
-  ) {
     return space.children.map((child) {
       return CustomExpansionTileSpaceTree(
-        isSelected: state.selectedSpaces.contains(child.uuid) ||
+        isSelected:
-            state.soldCheck.contains(child.uuid),
+            state.selectedSpaces.contains(child.uuid) || state.soldCheck.contains(child.uuid),
         isSoldCheck: state.soldCheck.contains(child.uuid),
         title: child.name,
         isExpanded: state.expandedSpaces.contains(child.uuid),
         onItemSelected: () {
-          context.read<SpaceTreeBloc>().add(
+          context
-                OnSpaceSelected(community, child.uuid ?? '', child.children),
+              .read<SpaceTreeBloc>()
-              );
+              .add(OnSpaceSelected(community, child.uuid ?? '', child.children));
           widget.onSelect();
         },
         onExpansionChanged: () {
-          context.read<SpaceTreeBloc>().add(
+          context.read<SpaceTreeBloc>().add(OnSpaceExpanded(community.uuid, child.uuid ?? ''));
-                OnSpaceExpanded(community.uuid, child.uuid ?? ''),
-              );
         },
         children: _buildNestedSpaces(context, state, child, community),
       );

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

@@ -592,8 +592,6 @@ class CreateSpaceDialogState extends State<CreateSpaceDialog> {
         return CreateSubSpaceDialog(
             spaceName: name,
             dialogTitle: isEdit ? 'Edit Sub-space' : 'Create Sub-space',
-            spaceTags: spaceTags,
-            isEdit: isEdit,
             products: products,
             existingSubSpaces: existingSubSpaces,
             onSave: (slectedSubspaces) {

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

@@ -1,8 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
-import 'package:syncrow_web/common/widgets/empty_search_result_widget.dart';
 import 'package:syncrow_web/common/widgets/search_bar.dart';
-import 'package:syncrow_web/common/widgets/sidebar_communities_list.dart';
 import 'package:syncrow_web/pages/spaces_management/all_spaces/bloc/space_management_bloc.dart';
 import 'package:syncrow_web/pages/spaces_management/all_spaces/bloc/space_management_event.dart';
 import 'package:syncrow_web/pages/spaces_management/all_spaces/model/community_model.dart';
@@ -32,8 +30,6 @@ class SidebarWidget extends StatefulWidget {
 }
 
 class _SidebarWidgetState extends State<SidebarWidget> {
-  late final ScrollController _scrollController;
-
   String _searchQuery = '';
   String? _selectedSpaceUuid;
   String? _selectedId;
@@ -41,16 +37,9 @@ class _SidebarWidgetState extends State<SidebarWidget> {
   @override
   void initState() {
     _selectedId = widget.selectedSpaceUuid;
-    _scrollController = ScrollController();
     super.initState();
   }
 
-  @override
-  void dispose() {
-    _scrollController.dispose();
-    super.dispose();
-  }
-
   @override
   void didUpdateWidget(covariant SidebarWidget oldWidget) {
     if (widget.selectedSpaceUuid != oldWidget.selectedSpaceUuid) {
@@ -97,14 +86,12 @@ class _SidebarWidgetState extends State<SidebarWidget> {
     return isSpaceSelected || anySubSpaceIsSelected;
   }
 
-  static const _width = 300.0;
-
   @override
   Widget build(BuildContext context) {
     final filteredCommunities = _filteredCommunities();
 
     return Container(
-      width: _width,
+      width: 300,
       decoration: subSectionContainerDecoration,
       child: Column(
         mainAxisSize: MainAxisSize.min,
@@ -116,18 +103,10 @@ class _SidebarWidgetState extends State<SidebarWidget> {
           ),
           const SizedBox(height: 16),
           Expanded(
-            child: Visibility(
+            child: ListView(
-              visible: filteredCommunities.isNotEmpty,
+              children: filteredCommunities
-              replacement: const EmptySearchResultWidget(),
+                  .map((community) => _buildCommunityTile(context, community))
-              child: SidebarCommunitiesList(
+                  .toList(),
-                scrollController: _scrollController,
-                onScrollToEnd: () {},
-                communities: filteredCommunities,
-                itemBuilder: (context, index) => _buildCommunityTile(
-                  context,
-                  filteredCommunities[index],
-                ),
-              ),
             ),
           ),
         ],
@@ -155,12 +134,11 @@ class _SidebarWidgetState extends State<SidebarWidget> {
       },
       onExpansionChanged: (title, expanded) {},
       children: community.spaces
-          .where(
+          .where((space) {
-            (space) => {
+            final isDeleted = space.status != SpaceStatus.deleted;
-              SpaceStatus.deleted,
+            final isParentDeleted = space.status != SpaceStatus.parentDeleted;
-              SpaceStatus.parentDeleted,
+            return (isDeleted || isParentDeleted);
-            }.contains(space.status),
+          })
-          )
           .map((space) => _buildSpaceTile(space: space, community: community))
           .toList(),
     );

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

@@ -4,169 +4,155 @@ import 'package:syncrow_web/pages/common/buttons/cancel_button.dart';
 import 'package:syncrow_web/pages/common/buttons/default_button.dart';
 import 'package:syncrow_web/pages/spaces_management/all_spaces/model/product_model.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/create_subspace/bloc/subspace_bloc.dart';
 import 'package:syncrow_web/pages/spaces_management/create_subspace/bloc/subspace_event.dart';
 import 'package:syncrow_web/pages/spaces_management/create_subspace/bloc/subspace_state.dart';
+import 'package:syncrow_web/pages/spaces_management/create_subspace_model/widgets/subspace_chip.dart';
+import 'package:syncrow_web/pages/spaces_management/space_model/models/subspace_template_model.dart';
 import 'package:syncrow_web/utils/color_manager.dart';
+import 'package:syncrow_web/utils/extension/build_context_x.dart';
 
-class CreateSubSpaceDialog extends StatelessWidget {
+class CreateSubSpaceDialog extends StatefulWidget {
-  final bool isEdit;
   final String dialogTitle;
   final List<SubspaceModel>? existingSubSpaces;
   final String? spaceName;
-  final List<Tag>? spaceTags;
   final List<ProductModel>? products;
-  final Function(List<SubspaceModel>?)? onSave;
+  final void Function(List<SubspaceModel>?)? onSave;
 
-  const CreateSubSpaceDialog(
+  const CreateSubSpaceDialog({
-      {Key? key,
-      required this.isEdit,
     required this.dialogTitle,
-      this.existingSubSpaces,
     required this.spaceName,
-      required this.spaceTags,
     required this.products,
-      required this.onSave})
+    required this.onSave,
-      : super(key: key);
+    this.existingSubSpaces,
+    super.key,
+  });
+
+  @override
+  State<CreateSubSpaceDialog> createState() => _CreateSubSpaceDialogState();
+}
+
+class _CreateSubSpaceDialogState extends State<CreateSubSpaceDialog> {
+  late final TextEditingController _subspaceNameController;
+
+  @override
+  void initState() {
+    _subspaceNameController = TextEditingController();
+    super.initState();
+  }
+
+  @override
+  void dispose() {
+    _subspaceNameController.dispose();
+    super.dispose();
+  }
 
   @override
   Widget build(BuildContext context) {
-    final screenWidth = MediaQuery.of(context).size.width;
+    return BlocProvider(
-    final textController = TextEditingController();
-
-    return Dialog(
-      shape: RoundedRectangleBorder(
-        borderRadius: BorderRadius.circular(20),
-      ),
-      child: BlocProvider(
       create: (_) {
         final bloc = SubSpaceBloc();
-          if (existingSubSpaces != null) {
+        if (widget.existingSubSpaces != null) {
-            for (var subSpace in existingSubSpaces!) {
+          for (final subSpace in widget.existingSubSpaces ?? []) {
             bloc.add(AddSubSpace(subSpace));
           }
         }
         return bloc;
       },
+      child: Dialog(
+        shape: RoundedRectangleBorder(
+          borderRadius: BorderRadius.circular(20),
+        ),
         child: BlocBuilder<SubSpaceBloc, SubSpaceState>(
           builder: (context, state) {
             return Container(
+              width: context.screenWidth * 0.35,
               color: ColorsManager.whiteColors,
-                child: SizedBox(
+              padding: const EdgeInsets.all(16),
-                  width: screenWidth * 0.35,
-                  child: Padding(
-                    padding: const EdgeInsets.all(16.0),
               child: Column(
                 crossAxisAlignment: CrossAxisAlignment.start,
                 mainAxisSize: MainAxisSize.min,
                 children: [
                   Text(
-                          dialogTitle,
+                    widget.dialogTitle,
-                          style: Theme.of(context)
+                    style: context.textTheme.headlineLarge?.copyWith(
-                              .textTheme
+                      color: ColorsManager.blackColor,
-                              .headlineLarge
+                    ),
-                              ?.copyWith(color: ColorsManager.blackColor),
                   ),
                   const SizedBox(height: 16),
                   Container(
-                          width: screenWidth * 0.35,
+                    width: context.screenWidth * 0.35,
                     padding: const EdgeInsets.symmetric(
-                              vertical: 10.0, horizontal: 16.0),
+                      vertical: 10,
+                      horizontal: 16,
+                    ),
                     decoration: BoxDecoration(
                       color: ColorsManager.boxColor,
                       borderRadius: BorderRadius.circular(10),
                     ),
                     child: Wrap(
-                            spacing: 8.0,
+                      spacing: 8,
-                            runSpacing: 8.0,
+                      runSpacing: 8,
+                      alignment: WrapAlignment.start,
+                      crossAxisAlignment: WrapCrossAlignment.center,
                       children: [
                         ...state.subSpaces.asMap().entries.map(
                           (entry) {
                             final index = entry.key;
                             final subSpace = entry.value;
 
-                                  final lowerName =
+                            final lowerName = subSpace.subspaceName.toLowerCase();
-                                      subSpace.subspaceName.toLowerCase();
 
                             final duplicateIndices = state.subSpaces
                                 .asMap()
                                 .entries
                                 .where((e) =>
-                                          e.value.subspaceName.toLowerCase() ==
+                                    e.value.subspaceName.toLowerCase() == lowerName)
-                                          lowerName)
                                 .map((e) => e.key)
                                 .toList();
-                                  final isDuplicate =
+                            final isDuplicate = duplicateIndices.length > 1 &&
-                                      duplicateIndices.length > 1 &&
                                 duplicateIndices.indexOf(index) != 0;
-
+                            return SubspaceChip(
-                                  return Chip(
+                              subSpace: SubspaceTemplateModel(
-                                    label: Text(subSpace.subspaceName,
+                                subspaceName: entry.value.subspaceName,
-                                        style: Theme.of(context)
+                                disabled: entry.value.disabled,
-                                            .textTheme
-                                            .bodyMedium
-                                            ?.copyWith(
-                                                color:
-                                                    ColorsManager.spaceColor)),
-                                    backgroundColor: ColorsManager.whiteColors,
-                                    shape: RoundedRectangleBorder(
-                                      borderRadius: BorderRadius.circular(10),
-                                      side: BorderSide(
-                                        color: isDuplicate
-                                            ? ColorsManager.red
-                                            : ColorsManager.transparentColor,
-                                        width: 0,
                               ),
+                              isDuplicate: isDuplicate,
+                              onDeleted: () => context.read<SubSpaceBloc>().add(
+                                    RemoveSubSpace(subSpace),
                                   ),
-                                    deleteIcon: Container(
-                                      width: 24,
-                                      height: 24,
-                                      decoration: BoxDecoration(
-                                        shape: BoxShape.circle,
-                                        border: Border.all(
-                                          color: ColorsManager.lightGrayColor,
-                                          width: 1.5,
-                                        ),
-                                      ),
-                                      child: const Icon(
-                                        Icons.close,
-                                        size: 16,
-                                        color: ColorsManager.lightGrayColor,
-                                      ),
-                                    ),
-                                    onDeleted: () => context
-                                        .read<SubSpaceBloc>()
-                                        .add(RemoveSubSpace(subSpace)),
                             );
                           },
                         ),
                         SizedBox(
                           width: 200,
                           child: TextField(
-                                    controller: textController,
+                            controller: _subspaceNameController,
                             decoration: InputDecoration(
                               border: InputBorder.none,
                               hintText: state.subSpaces.isEmpty
                                   ? 'Please enter the name'
                                   : null,
-                                        hintStyle: Theme.of(context)
+                              hintStyle: context.textTheme.bodySmall?.copyWith(
-                                            .textTheme
+                                color: ColorsManager.lightGrayColor,
-                                            .bodySmall
+                              ),
-                                            ?.copyWith(
+                            ),
-                                                color: ColorsManager
-                                                    .lightGrayColor)),
                             onSubmitted: (value) {
-                                      if (value.trim().isNotEmpty) {
+                              final trimmedValue = value.trim();
+                              if (trimmedValue.isNotEmpty) {
                                 context.read<SubSpaceBloc>().add(
-                                            AddSubSpace(SubspaceModel(
+                                      AddSubSpace(
-                                                subspaceName: value.trim(),
+                                        SubspaceModel(
-                                                disabled: false)));
+                                          subspaceName: trimmedValue,
-                                        textController.clear();
+                                          disabled: false,
+                                        ),
+                                      ),
+                                    );
+                                _subspaceNameController.clear();
                               }
                             },
-                                    style:
+                            style: context.textTheme.bodyMedium,
-                                        Theme.of(context).textTheme.bodyMedium),
+                          ),
                         ),
                       ],
                     ),
@@ -174,13 +160,12 @@ class CreateSubSpaceDialog extends StatelessWidget {
                   if (state.errorMessage.isNotEmpty)
                     Padding(
                       padding: const EdgeInsets.only(top: 8.0),
-                            child: Text(state.errorMessage,
+                      child: Text(
-                                style: Theme.of(context)
+                        state.errorMessage,
-                                    .textTheme
+                        style: context.textTheme.bodySmall?.copyWith(
-                                    .bodySmall
-                                    ?.copyWith(
                           color: ColorsManager.warningRed,
-                                    )),
+                        ),
+                      ),
                     ),
                   const SizedBox(height: 16),
                   Row(
@@ -196,16 +181,14 @@ class CreateSubSpaceDialog extends StatelessWidget {
                       const SizedBox(width: 10),
                       Expanded(
                         child: DefaultButton(
-                                onPressed: (state.errorMessage.isNotEmpty)
+                          onPressed: state.errorMessage.isEmpty
-                                    ? null
+                              ? () {
-                                    : () async {
+                                  final subSpacesBloc = context.read<SubSpaceBloc>();
-                                        final subSpaces = context
+                                  final subSpaces = subSpacesBloc.state.subSpaces;
-                                            .read<SubSpaceBloc>()
+                                  widget.onSave?.call(subSpaces);
-                                            .state
-                                            .subSpaces;
-                                        onSave!(subSpaces);
                                   Navigator.of(context).pop();
-                                      },
+                                }
+                              : null,
                           backgroundColor: ColorsManager.secondaryColor,
                           borderRadius: 10,
                           foregroundColor: state.errorMessage.isNotEmpty
@@ -218,8 +201,7 @@ class CreateSubSpaceDialog extends StatelessWidget {
                   ),
                 ],
               ),
-                  ),
+            );
-                ));
           },
         ),
       ),

lib/pages/spaces_management/create_subspace_model/widgets/create_subspace_model_chips_box.dart

@@ -1,4 +1,7 @@
 import 'package:flutter/material.dart';
+import 'package:flutter_bloc/flutter_bloc.dart';
+import 'package:syncrow_web/pages/spaces_management/create_subspace_model/bloc/subspace_model_bloc.dart';
+import 'package:syncrow_web/pages/spaces_management/create_subspace_model/bloc/subspace_model_event.dart';
 import 'package:syncrow_web/pages/spaces_management/create_subspace_model/widgets/subspace_chip.dart';
 import 'package:syncrow_web/pages/spaces_management/create_subspace_model/widgets/subspaces_textfield.dart';
 import 'package:syncrow_web/pages/spaces_management/space_model/models/subspace_template_model.dart';
@@ -51,6 +54,9 @@ class CreateSubspaceModelChipsBox extends StatelessWidget {
               return SubspaceChip(
                 subSpace: subSpace,
                 isDuplicate: isDuplicate,
+                onDeleted: () => context.read<SubSpaceModelBloc>().add(
+                      RemoveSubSpaceModel(subSpace),
+                    ),
               );
             },
           ),

lib/pages/spaces_management/create_subspace_model/widgets/subspace_chip.dart

@@ -1,7 +1,4 @@
 import 'package:flutter/material.dart';
-import 'package:flutter_bloc/flutter_bloc.dart';
-import 'package:syncrow_web/pages/spaces_management/create_subspace_model/bloc/subspace_model_bloc.dart';
-import 'package:syncrow_web/pages/spaces_management/create_subspace_model/bloc/subspace_model_event.dart';
 import 'package:syncrow_web/pages/spaces_management/space_model/models/subspace_template_model.dart';
 import 'package:syncrow_web/utils/color_manager.dart';
 import 'package:syncrow_web/utils/extension/build_context_x.dart';
@@ -11,10 +8,12 @@ class SubspaceChip extends StatelessWidget {
     required this.subSpace,
     required this.isDuplicate,
     super.key,
+    required this.onDeleted,
   });
 
   final SubspaceTemplateModel subSpace;
   final bool isDuplicate;
+  final void Function() onDeleted;
 
   @override
   Widget build(BuildContext context) {
@@ -50,9 +49,7 @@ class SubspaceChip extends StatelessWidget {
           ),
         ),
       ),
-      onDeleted: () => context.read<SubSpaceModelBloc>().add(
+      onDeleted: onDeleted,
-            RemoveSubSpaceModel(subSpace),
-          ),
     );
   }
 }

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

@@ -145,11 +145,13 @@ class CreateSpaceModelDialog extends StatelessWidget {
                     ),
                     const SizedBox(height: 10),
                     TagChipDisplay(
+                      context,
+                      screenWidth: screenWidth,
                       spaceModel: updatedSpaceModel,
                       products: products,
                       subspaces: subspaces,
                       allTags: allTags,
-                      spaceName: spaceNameController.text,
+                      spaceNameController: spaceNameController,
                       pageContext: pageContext,
                       otherSpaceModels: otherSpaceModels,
                       allSpaceModels: allSpaceModels,

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

@@ -10,77 +10,90 @@ import 'package:syncrow_web/pages/spaces_management/space_model/models/subspace_
 import 'package:syncrow_web/pages/spaces_management/space_model/widgets/button_content_widget.dart';
 import 'package:syncrow_web/pages/spaces_management/tag_model/views/add_device_type_model_widget.dart';
 import 'package:syncrow_web/utils/color_manager.dart';
-import 'package:syncrow_web/utils/constants/assets.dart';
-import 'package:syncrow_web/utils/extension/build_context_x.dart';
 
 class TagChipDisplay extends StatelessWidget {
-  const TagChipDisplay({
+  final double screenWidth;
-    required this.spaceModel,
-    required this.products,
-    required this.subspaces,
-    required this.allTags,
-    required this.spaceName,
-    required this.projectTags,
-    this.pageContext,
-    this.otherSpaceModels,
-    this.allSpaceModels,
-    super.key,
-  });
-
   final SpaceTemplateModel? spaceModel;
   final List<ProductModel>? products;
   final List<SubspaceTemplateModel>? subspaces;
   final List<String>? allTags;
-  final String spaceName;
+  final TextEditingController spaceNameController;
   final BuildContext? pageContext;
   final List<String>? otherSpaceModels;
   final List<SpaceTemplateModel>? allSpaceModels;
   final List<Tag> projectTags;
 
-  Map<ProductModel, int> get _groupedTags {
+  const TagChipDisplay(BuildContext context,
-    final spaceTags = spaceModel?.tags ?? <Tag>[];
+      {Key? key,
-
+      required this.screenWidth,
-    final subspaces = spaceModel?.subspaceModels ?? [];
+      required this.spaceModel,
-    final subspaceTags = subspaces.expand((e) => e.tags ?? <Tag>[]).toList();
+      required this.products,
-
+      required this.subspaces,
-    return TagHelper.groupTags([...spaceTags, ...subspaceTags]);
+      required this.allTags,
-  }
+      required this.spaceNameController,
+      this.pageContext,
+      this.otherSpaceModels,
+      this.allSpaceModels,
+      required this.projectTags})
+      : super(key: key);
 
   @override
   Widget build(BuildContext context) {
-    final hasTags = spaceModel?.tags?.isNotEmpty ?? false;
+    return (spaceModel?.tags?.isNotEmpty == true ||
-    final hasSubspaceTags =
+            spaceModel?.subspaceModels?.any((subspace) => subspace.tags?.isNotEmpty == true) ==
-        spaceModel?.subspaceModels?.any((e) => e.tags?.isNotEmpty ?? false) ?? false;
+                true)
-
+        ? SizedBox(
-    if (hasTags || hasSubspaceTags) {
+            width: screenWidth * 0.25,
-      return Container(
+            child: Container(
-        width: context.screenWidth * 0.25,
+              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,
+                  width: 3.0, // Border width
                 ),
               ),
               child: Wrap(
-          spacing: 8,
+                spacing: 8.0,
-          runSpacing: 8,
+                runSpacing: 8.0,
                 children: [
-            ..._groupedTags.entries.map((entry) => _buildChip(context, entry)),
+                  // Combine tags from spaceModel and subspaces
-            _buildEditChip(context),
+                  ...TagHelper.groupTags([
-          ],
+                    ...?spaceModel?.tags,
+                    ...?spaceModel?.subspaceModels?.expand((subspace) => subspace.tags ?? [])
+                  ]).entries.map(
+                        (entry) => Chip(
+                          avatar: SizedBox(
+                            width: 24,
+                            height: 24,
+                            child: SvgPicture.asset(
+                              entry.key.icon ?? 'assets/icons/gateway.svg',
+                              fit: BoxFit.contain,
                             ),
-      );
+                          ),
-    }
+                          label: Text(
+                            'x${entry.value}', // Show count
+                            style: Theme.of(context)
+                                .textTheme
+                                .bodySmall!
+                                .copyWith(color: ColorsManager.spaceColor),
+                          ),
+                          backgroundColor: ColorsManager.whiteColors,
+                          shape: RoundedRectangleBorder(
+                            borderRadius: BorderRadius.circular(16),
+                            side: const BorderSide(
+                              color: ColorsManager.spaceColor,
+                            ),
+                          ),
+                        ),
+                      ),
+                  EditChip(onTap: () async {
+                    // Use the Navigator's context for showDialog
+                    Navigator.of(context).pop();
 
-    return _buildAddDevicesButton(context);
+                    await showDialog<bool>(
-  }
+                        barrierDismissible: false,
-
-  Widget _buildEditChip(BuildContext context) {
-    return EditChip(
-      onTap: () => showDialog<void>(
                         context: context,
                         builder: (context) => AssignTagModelsDialog(
                               products: products,
@@ -91,64 +104,38 @@ class TagChipDisplay extends StatelessWidget {
                               spaceModel: spaceModel,
                               otherSpaceModels: otherSpaceModels,
                               initialTags: TagHelper.generateInitialTags(
-            subspaces: subspaces,
+                                  subspaces: subspaces, spaceTagModels: spaceModel?.tags ?? []),
-            spaceTagModels: spaceModel?.tags ?? [],
-          ),
                               title: 'Edit Device',
                               addedProducts: TagHelper.createInitialSelectedProducts(
-            spaceModel?.tags ?? [],
+                                  spaceModel?.tags ?? [], subspaces),
-            subspaces,
-          ),
                               spaceName: spaceModel?.modelName ?? '',
                               projectTags: projectTags,
+                            ));
+                  })
+                ],
               ),
             ),
-    );
+          )
-  }
+        : TextButton(
+            onPressed: () async {
+              Navigator.of(context).pop();
 
-  Widget _buildChip(
+              await showDialog<bool>(
-    BuildContext context,
+                barrierDismissible: false,
-    MapEntry<ProductModel, int> entry,
-  ) {
-    return Chip(
-      backgroundColor: ColorsManager.whiteColors,
-      shape: RoundedRectangleBorder(
-        borderRadius: BorderRadius.circular(16),
-        side: const BorderSide(
-          color: ColorsManager.spaceColor,
-        ),
-      ),
-      avatar: SvgPicture.asset(
-        entry.key.icon ?? Assets.gateway,
-        fit: BoxFit.contain,
-        height: 24,
-        width: 24,
-      ),
-      label: Text(
-        '${entry.value}',
-        style: context.textTheme.bodySmall!.copyWith(
-          color: ColorsManager.spaceColor,
-        ),
-      ),
-    );
-  }
-
-  Widget _buildAddDevicesButton(BuildContext context) {
-    return TextButton(
-      onPressed: () => showDialog<void>(
                 context: context,
                 builder: (context) => AddDeviceTypeModelWidget(
                   products: products,
                   subspaces: subspaces,
                   allTags: allTags,
-          spaceName: spaceName,
+                  spaceName: spaceNameController.text,
                   pageContext: pageContext,
                   isCreate: true,
                   spaceModel: spaceModel,
                   otherSpaceModels: otherSpaceModels,
                   projectTags: projectTags,
                 ),
-      ),
+              );
+            },
             style: TextButton.styleFrom(
               padding: EdgeInsets.zero,
             ),