From f670ae78aa87e7022f6b10a9910770c8faa24f25 Mon Sep 17 00:00:00 2001 From: fkarmoush Date: Wed, 26 Mar 2025 12:39:51 +0300 Subject: [PATCH] Refactor device search filters layout for improved code organization and to remove any code duplication. --- .../widgets/device_search_filters.dart | 61 ++++++++----------- 1 file changed, 26 insertions(+), 35 deletions(-) diff --git a/lib/pages/device_managment/all_devices/widgets/device_search_filters.dart b/lib/pages/device_managment/all_devices/widgets/device_search_filters.dart index 659a291a..3910257f 100644 --- a/lib/pages/device_managment/all_devices/widgets/device_search_filters.dart +++ b/lib/pages/device_managment/all_devices/widgets/device_search_filters.dart @@ -31,43 +31,34 @@ class _DeviceSearchFiltersState extends State super.dispose(); } + List get _widgets => [ + _buildSearchField( + "Space Name", + unitNameController, + 200, + ), + _buildSearchField( + "Device Name / Product Name", + productNameController, + 300, + ), + _buildSearchResetButtons(), + ]; + @override Widget build(BuildContext context) { - return isExtraLargeScreenSize(context) - ? Row( - children: [ - _buildSearchField( - "Space Name", - unitNameController, - 200, - ), - const SizedBox(width: 20), - _buildSearchField( - "Device Name / Product Name", - productNameController, - 300, - ), - const SizedBox(width: 20), - _buildSearchResetButtons(), - ], - ) - : Wrap( - spacing: 20, - runSpacing: 10, - children: [ - _buildSearchField( - "Space Name", - unitNameController, - 200, - ), - _buildSearchField( - "Device Name / Product Name", - productNameController, - 300, - ), - _buildSearchResetButtons(), - ], - ); + if (isExtraLargeScreenSize(context)) { + return Row( + spacing: 20, + children: _widgets, + ); + } + + return Wrap( + spacing: 20, + runSpacing: 10, + children: _widgets, + ); } Widget _buildSearchField(