From a4432656ab05cf39351240d1134cb033e65fbc5b Mon Sep 17 00:00:00 2001 From: Faris Armoush Date: Tue, 15 Apr 2025 15:25:19 +0300 Subject: [PATCH] refactor: extract EditChip into a private method for improved readability --- .../widgets/tag_chips_display_widget.dart | 55 ++++++++++--------- 1 file changed, 30 insertions(+), 25 deletions(-) diff --git a/lib/pages/spaces_management/space_model/widgets/tag_chips_display_widget.dart b/lib/pages/spaces_management/space_model/widgets/tag_chips_display_widget.dart index 200cd659..8cf52c4f 100644 --- a/lib/pages/spaces_management/space_model/widgets/tag_chips_display_widget.dart +++ b/lib/pages/spaces_management/space_model/widgets/tag_chips_display_widget.dart @@ -67,30 +67,7 @@ class TagChipDisplay extends StatelessWidget { runSpacing: 8, children: [ ..._groupedTags.entries.map((entry) => _buildChip(context, entry)), - EditChip( - onTap: () => showDialog( - context: context, - builder: (context) => AssignTagModelsDialog( - products: products, - allSpaceModels: allSpaceModels, - subspaces: subspaces, - pageContext: pageContext, - allTags: allTags, - spaceModel: spaceModel, - otherSpaceModels: otherSpaceModels, - initialTags: TagHelper.generateInitialTags( - subspaces: subspaces, - spaceTagModels: spaceModel?.tags ?? []), - title: 'Edit Device', - addedProducts: TagHelper.createInitialSelectedProducts( - spaceModel?.tags ?? [], - subspaces, - ), - spaceName: spaceModel?.modelName ?? '', - projectTags: projectTags, - ), - ), - ), + _buildEditChip(context), ], ), ), @@ -100,7 +77,35 @@ class TagChipDisplay extends StatelessWidget { } } - Chip _buildChip( + Widget _buildEditChip(BuildContext context) { + return EditChip( + onTap: () => showDialog( + context: context, + builder: (context) => AssignTagModelsDialog( + products: products, + allSpaceModels: allSpaceModels, + subspaces: subspaces, + pageContext: pageContext, + allTags: allTags, + spaceModel: spaceModel, + otherSpaceModels: otherSpaceModels, + initialTags: TagHelper.generateInitialTags( + subspaces: subspaces, + spaceTagModels: spaceModel?.tags ?? [], + ), + title: 'Edit Device', + addedProducts: TagHelper.createInitialSelectedProducts( + spaceModel?.tags ?? [], + subspaces, + ), + spaceName: spaceModel?.modelName ?? '', + projectTags: projectTags, + ), + ), + ); + } + + Widget _buildChip( BuildContext context, MapEntry entry, ) {