Fixed design issues

lib/pages/common/custom_table.dart

@@ -53,11 +53,26 @@ class _DynamicTableState extends State<DynamicTable> {
   @override
   void didUpdateWidget(DynamicTable oldWidget) {
     super.didUpdateWidget(oldWidget);
-    if (oldWidget.data.length != widget.data.length) {
+    if (!_compareListOfLists(oldWidget.data, widget.data)) {
       _initializeSelection();
     }
   }
 
+  bool _compareListOfLists(List<List<dynamic>> oldList, List<List<dynamic>> newList) {
+    // Check if the old and new lists are the same
+    if (oldList.length != newList.length) return false;
+
+    for (int i = 0; i < oldList.length; i++) {
+      if (oldList[i].length != newList[i].length) return false;
+
+      for (int j = 0; j < oldList[i].length; j++) {
+        if (oldList[i][j] != newList[i][j]) return false;
+      }
+    }
+
+    return true;
+  }
+
   void _initializeSelection() {
     _selectedRows = List<bool>.filled(widget.data.length, false);
     _selectAll = false;
@@ -90,13 +105,11 @@ class _DynamicTableState extends State<DynamicTable> {
           child: Column(
             children: [
               Container(
-                decoration: widget.headerDecoration ??
+                decoration: widget.headerDecoration ?? BoxDecoration(color: Colors.grey[200]),
-                    BoxDecoration(color: Colors.grey[200]),
                 child: Row(
                   children: [
                     if (widget.withCheckBox) _buildSelectAllCheckbox(),
-                    ...widget.headers
+                    ...widget.headers.map((header) => _buildTableHeaderCell(header)),
-                        .map((header) => _buildTableHeaderCell(header)),
                   ],
                 ),
               ),
@@ -123,8 +136,7 @@ class _DynamicTableState extends State<DynamicTable> {
                                     style: Theme.of(context)
                                         .textTheme
                                         .bodySmall!
-                                        .copyWith(
+                                        .copyWith(color: ColorsManager.grayColor),
-                                            color: ColorsManager.grayColor),
                                   )
                                 ],
                               ),
@@ -144,11 +156,9 @@ class _DynamicTableState extends State<DynamicTable> {
                             return Row(
                               children: [
                                 if (widget.withCheckBox)
-                                  _buildRowCheckbox(
+                                  _buildRowCheckbox(index, widget.size.height * 0.10),
-                                      index, widget.size.height * 0.10),
+                                ...row.map((cell) =>
-                                ...row.map((cell) => _buildTableCell(
+                                    _buildTableCell(cell.toString(), widget.size.height * 0.10)),
-                                    cell.toString(),
-                                    widget.size.height * 0.10)),
                               ],
                             );
                           },
@@ -173,9 +183,7 @@ class _DynamicTableState extends State<DynamicTable> {
       ),
       child: Checkbox(
         value: _selectAll,
-        onChanged: widget.withSelectAll && widget.data.isNotEmpty
+        onChanged: widget.withSelectAll && widget.data.isNotEmpty ? _toggleSelectAll : null,
-            ? _toggleSelectAll
-            : null,
       ),
     );
   }

lib/pages/common/text_field/custom_text_field.dart

@@ -2,20 +2,21 @@ import 'package:flutter/material.dart';
 import 'package:syncrow_web/utils/extension/build_context_x.dart';
 
 class StatefulTextField extends StatefulWidget {
-  const StatefulTextField({
+  const StatefulTextField(
-    super.key,
+      {super.key,
-    required this.title,
+      required this.title,
-    this.hintText = 'Please enter',
+      this.hintText = 'Please enter',
-    required this.width,
+      required this.width,
-    this.elevation = 0,
+      this.elevation = 0,
-    required this.controller,
+      required this.controller,
-  });
+      this.onSubmitted});
 
   final String title;
   final String hintText;
   final double width;
   final double elevation;
   final TextEditingController controller;
+  final Function? onSubmitted;
 
   @override
   State<StatefulTextField> createState() => _StatefulTextFieldState();
@@ -25,30 +26,31 @@ class _StatefulTextFieldState extends State<StatefulTextField> {
   @override
   Widget build(BuildContext context) {
     return CustomTextField(
-      title: widget.title,
+        title: widget.title,
-      controller: widget.controller,
+        controller: widget.controller,
-      hintText: widget.hintText,
+        hintText: widget.hintText,
-      width: widget.width,
+        width: widget.width,
-      elevation: widget.elevation,
+        elevation: widget.elevation,
-    );
+        onSubmittedFun: widget.onSubmitted);
   }
 }
 
 class CustomTextField extends StatelessWidget {
-  const CustomTextField({
+  const CustomTextField(
-    super.key,
+      {super.key,
-    required this.title,
+      required this.title,
-    required this.controller,
+      required this.controller,
-    this.hintText = 'Please enter',
+      this.hintText = 'Please enter',
-    required this.width,
+      required this.width,
-    this.elevation = 0,
+      this.elevation = 0,
-  });
+      this.onSubmittedFun});
 
   final String title;
   final TextEditingController controller;
   final String hintText;
   final double width;
   final double elevation;
+  final Function? onSubmittedFun;
 
   @override
   Widget build(BuildContext context) {
@@ -81,10 +83,12 @@ class CustomTextField extends StatelessWidget {
               decoration: InputDecoration(
                 hintText: hintText,
                 hintStyle: const TextStyle(fontSize: 12),
-                contentPadding:
+                contentPadding: const EdgeInsets.symmetric(horizontal: 12, vertical: 10),
-                    const EdgeInsets.symmetric(horizontal: 12, vertical: 10),
                 border: InputBorder.none,
               ),
+              onFieldSubmitted: (_) {
+                onSubmittedFun!();
+              },
             ),
           ),
         ),

lib/pages/device_managment/all_devices/widgets/device_managment_body.dart

@@ -37,8 +37,8 @@ class DeviceManagementBody extends StatelessWidget with HelperResponsiveLayout {
           offlineCount = state.offlineCount;
           lowBatteryCount = state.lowBatteryCount;
           isControlButtonEnabled = state.isControlButtonEnabled;
-          selectedDevices = state.selectedDevice ??
+          selectedDevices =
-              context.read<DeviceManagementBloc>().selectedDevices;
+              state.selectedDevice ?? context.read<DeviceManagementBloc>().selectedDevices;
         } else if (state is DeviceManagementFiltered) {
           devicesToShow = state.filteredDevices;
           selectedIndex = state.selectedIndex;
@@ -46,8 +46,8 @@ class DeviceManagementBody extends StatelessWidget with HelperResponsiveLayout {
           offlineCount = state.offlineCount;
           lowBatteryCount = state.lowBatteryCount;
           isControlButtonEnabled = state.isControlButtonEnabled;
-          selectedDevices = state.selectedDevice ??
+          selectedDevices =
-              context.read<DeviceManagementBloc>().selectedDevices;
+              state.selectedDevice ?? context.read<DeviceManagementBloc>().selectedDevices;
         } else if (state is DeviceManagementInitial) {
           devicesToShow = [];
           selectedIndex = 0;
@@ -61,15 +61,13 @@ class DeviceManagementBody extends StatelessWidget with HelperResponsiveLayout {
           'Low Battery ($lowBatteryCount)',
         ];
 
-        final buttonLabel =
+        final buttonLabel = (selectedDevices.length > 1) ? 'Batch Control' : 'Control';
-            (selectedDevices.length > 1) ? 'Batch Control' : 'Control';
 
         return Column(
           children: [
             Container(
-              padding: isLargeScreenSize(context)
+              padding:
-                  ? const EdgeInsets.all(30)
+                  isLargeScreenSize(context) ? const EdgeInsets.all(30) : const EdgeInsets.all(15),
-                  : const EdgeInsets.all(15),
               child: Column(
                 crossAxisAlignment: CrossAxisAlignment.start,
                 children: [
@@ -78,9 +76,7 @@ class DeviceManagementBody extends StatelessWidget with HelperResponsiveLayout {
                     tabs: tabs,
                     selectedIndex: selectedIndex,
                     onTabChanged: (index) {
-                      context
+                      context.read<DeviceManagementBloc>().add(SelectedFilterChanged(index));
-                          .read<DeviceManagementBloc>()
-                          .add(SelectedFilterChanged(index));
                     },
                   ),
                   const SizedBox(height: 20),
@@ -102,14 +98,12 @@ class DeviceManagementBody extends StatelessWidget with HelperResponsiveLayout {
                                     ),
                                   );
                                 } else if (selectedDevices.length > 1) {
-                                  final productTypes = selectedDevices
+                                  final productTypes =
-                                      .map((device) => device.productType)
+                                      selectedDevices.map((device) => device.productType).toSet();
-                                      .toSet();
                                   if (productTypes.length == 1) {
                                     showDialog(
                                       context: context,
-                                      builder: (context) =>
+                                      builder: (context) => DeviceBatchControlDialog(
-                                          DeviceBatchControlDialog(
                                         devices: selectedDevices,
                                       ),
                                     );
@@ -123,9 +117,7 @@ class DeviceManagementBody extends StatelessWidget with HelperResponsiveLayout {
                           textAlign: TextAlign.center,
                           style: TextStyle(
                             fontSize: 12,
-                            color: isControlButtonEnabled
+                            color: isControlButtonEnabled ? Colors.white : Colors.grey,
-                                ? Colors.white
-                                : Colors.grey,
                           ),
                         ),
                       ),
@@ -144,9 +136,7 @@ class DeviceManagementBody extends StatelessWidget with HelperResponsiveLayout {
                   cellDecoration: containerDecoration,
                   onRowSelected: (index, isSelected, row) {
                     final selectedDevice = devicesToShow[index];
-                    context
+                    context.read<DeviceManagementBloc>().add(SelectDevice(selectedDevice));
-                        .read<DeviceManagementBloc>()
-                        .add(SelectDevice(selectedDevice));
                   },
                   withCheckBox: true,
                   size: context.screenSize,
@@ -169,20 +159,16 @@ class DeviceManagementBody extends StatelessWidget with HelperResponsiveLayout {
                       device.uuid ?? '',
                       device.unit?.name ?? '',
                       device.room?.name ?? '',
-                      device.batteryLevel != null
+                      device.batteryLevel != null ? '${device.batteryLevel}%' : '-',
-                          ? '${device.batteryLevel}%'
+                      formatDateTime(
-                          : '-',
+                          DateTime.fromMillisecondsSinceEpoch((device.createTime ?? 0) * 1000)),
-                      formatDateTime(DateTime.fromMillisecondsSinceEpoch(
-                          (device.createTime ?? 0) * 1000)),
                       device.online == true ? 'Online' : 'Offline',
-                      formatDateTime(DateTime.fromMillisecondsSinceEpoch(
+                      formatDateTime(
-                          (device.updateTime ?? 0) * 1000)),
+                          DateTime.fromMillisecondsSinceEpoch((device.updateTime ?? 0) * 1000)),
                     ];
                   }).toList(),
                   onSelectionChanged: (selectedRows) {
-                    context
+                    context.read<DeviceManagementBloc>().add(UpdateSelection(selectedRows));
-                        .read<DeviceManagementBloc>()
-                        .add(UpdateSelection(selectedRows));
                   },
                   initialSelectedIds: context
                       .read<DeviceManagementBloc>()

lib/pages/device_managment/all_devices/widgets/device_search_filters.dart

@@ -12,8 +12,7 @@ class DeviceSearchFilters extends StatefulWidget {
   State<DeviceSearchFilters> createState() => _DeviceSearchFiltersState();
 }
 
-class _DeviceSearchFiltersState extends State<DeviceSearchFilters>
+class _DeviceSearchFiltersState extends State<DeviceSearchFilters> with HelperResponsiveLayout {
-    with HelperResponsiveLayout {
   final TextEditingController communityController = TextEditingController();
   final TextEditingController unitNameController = TextEditingController();
   final TextEditingController productNameController = TextEditingController();
@@ -35,8 +34,7 @@ class _DeviceSearchFiltersState extends State<DeviceSearchFilters>
               const SizedBox(width: 20),
               _buildSearchField("Unit Name", unitNameController, 200),
               const SizedBox(width: 20),
-              _buildSearchField(
+              _buildSearchField("Device Name / Product Name", productNameController, 300),
-                  "Device Name / Product Name", productNameController, 300),
               const SizedBox(width: 20),
               _buildSearchResetButtons(),
             ],
@@ -45,22 +43,34 @@ class _DeviceSearchFiltersState extends State<DeviceSearchFilters>
             spacing: 20,
             runSpacing: 10,
             children: [
-              _buildSearchField("Community", communityController, 200),
+              _buildSearchField(
+                "Community",
+                communityController,
+                200,
+              ),
               _buildSearchField("Unit Name", unitNameController, 200),
               _buildSearchField(
-                  "Device Name / Product Name", productNameController, 300),
+                "Device Name / Product Name",
+                productNameController,
+                300,
+              ),
               _buildSearchResetButtons(),
             ],
           );
   }
 
-  Widget _buildSearchField(
+  Widget _buildSearchField(String title, TextEditingController controller, double width) {
-      String title, TextEditingController controller, double width) {
     return StatefulTextField(
       title: title,
       width: width,
       elevation: 2,
       controller: controller,
+      onSubmitted: () {
+        context.read<DeviceManagementBloc>().add(SearchDevices(
+              productName: productNameController.text,
+              unitName: unitNameController.text,
+            ));
+      },
     );
   }
 

lib/pages/device_managment/shared/batch_control/firmware_update.dart

@@ -7,8 +7,7 @@ import 'package:syncrow_web/utils/constants/assets.dart';
 import 'package:syncrow_web/utils/extension/build_context_x.dart';
 
 class FirmwareUpdateWidget extends StatefulWidget {
-  const FirmwareUpdateWidget(
+  const FirmwareUpdateWidget({super.key, required this.deviceId, required this.version});
-      {super.key, required this.deviceId, required this.version});
 
   final String deviceId;
   final int version;
@@ -34,20 +33,23 @@ class _FirmwareUpdateWidgetState extends State<FirmwareUpdateWidget> {
               crossAxisAlignment: CrossAxisAlignment.start,
               mainAxisAlignment: MainAxisAlignment.spaceBetween,
               children: [
-                Text(
+                Column(
-                  'Firmware Update',
+                  children: [
-                  style: context.textTheme.titleMedium!.copyWith(
+                    Text(
-                    fontWeight: FontWeight.bold,
+                      'Firmware Update',
-                    color: ColorsManager.blackColor,
+                      style: context.textTheme.titleMedium!.copyWith(
-                  ),
+                        fontWeight: FontWeight.bold,
+                        color: ColorsManager.blackColor,
+                      ),
+                    ),
+                    Text(
+                      'Are you sure?',
+                      style: context.textTheme.bodySmall!.copyWith(
+                        color: ColorsManager.grayColor,
+                      ),
+                    ),
+                  ],
                 ),
-                Text(
-                  'Are you sure?',
-                  style: context.textTheme.bodySmall!.copyWith(
-                    color: ColorsManager.grayColor,
-                  ),
-                ),
-                const SizedBox(height: 16),
                 Row(
                   children: [
                     Flexible(

lib/pages/device_managment/shared/device_controls_container.dart

@@ -11,7 +11,14 @@ class DeviceControlsContainer extends StatelessWidget {
       decoration: BoxDecoration(
         borderRadius: BorderRadius.circular(20),
         color: ColorsManager.greyColor.withOpacity(0.2),
-        border: Border.all(color: ColorsManager.boxDivider),
+        // border: Border.all(color: ColorsManager.boxDivider),
+        // boxShadow: <BoxShadow>[
+        //   BoxShadow(
+        //       color: ColorsManager.blackColor.withOpacity(0.05),
+        //       blurRadius: 6.0,
+        //       offset: const Offset(0, 5),
+        //       spreadRadius: 0)
+        // ],
       ),
       padding: const EdgeInsets.all(12),
       child: child,

lib/utils/color_manager.dart

@@ -5,8 +5,7 @@ abstract class ColorsManager {
   static const Color switchOffColor = Color(0x7F8D99AE);
   static const Color primaryColor = Color(0xFF0030CB); //023DFE
   static const Color secondaryTextColor = Color(0xFF848484);
-  static Color primaryColorWithOpacity =
+  static Color primaryColorWithOpacity = const Color(0xFF023DFE).withOpacity(0.6);
-      const Color(0xFF023DFE).withOpacity(0.6);
   static const Color whiteColors = Colors.white;
   static const Color secondaryColor = Color(0xFF023DFE);
   static const Color onSecondaryColor = Color(0xFF023DFE);