Refactor CommunityStructureCanvas to simplify DragTarget logic by replacing SizedBox with SizedBox.shrink() for better performance and readability.

This commit is contained in:
Faris Armoush
2025-07-22 09:32:57 +03:00
parent 40251b846b
commit 9d60f913eb

View File

@ -425,18 +425,14 @@ class _CommunityStructureCanvasState extends State<CommunityStructureCanvas>
height: _cardHeight, height: _cardHeight,
child: DragTarget<SpaceReorderDataModel>( child: DragTarget<SpaceReorderDataModel>(
builder: (context, candidateData, rejectedData) { builder: (context, candidateData, rejectedData) {
if (_draggedData == null) { if (_draggedData == null) return const SizedBox.shrink();
return const SizedBox();
}
final isTargetForDragged = (_draggedData?.parent?.uuid == parent?.uuid && final isTargetForDragged = (_draggedData?.parent?.uuid == parent?.uuid &&
_draggedData?.community == null) || _draggedData?.community == null) ||
(_draggedData?.community?.uuid == community?.uuid && (_draggedData?.community?.uuid == community?.uuid &&
_draggedData?.parent == null); _draggedData?.parent == null);
if (!isTargetForDragged) { if (!isTargetForDragged) return const SizedBox.shrink();
return const SizedBox();
}
return Container( return Container(
width: 40, width: 40,