ui/ux asset delivery improvements

lib/modules/asset_delivery_module/pages/delivery_inspection/cost_center_form_view.dart

@@ -18,6 +18,7 @@ import 'package:test_sa/new_views/common_widgets/app_lazy_loading.dart';
 import 'package:test_sa/new_views/common_widgets/app_text_form_field.dart';
 import 'package:test_sa/new_views/common_widgets/default_app_bar.dart';
 import 'package:test_sa/new_views/common_widgets/single_item_drop_down_menu.dart';
+import 'package:test_sa/views/widgets/item_views/info_text_widget.dart';
 
 class CostCenterFormView extends StatefulWidget {
   final List<CostCenterModel>? costCenterList;
@@ -113,12 +114,19 @@ class _CostCenterFormViewState extends State<CostCenterFormView> {
       ),
       body: Consumer<AssetDeliveryProvider>(builder: (context, provider, child) {
         return provider.isLoading
-            ?  CircularProgressIndicator(color: AppColor.loadingColor(context)).center
+            ? CircularProgressIndicator(color: AppColor.loadingColor(context)).center
             : Column(
                 children: [
-                  itemDetails(),
+                  ListView(
-                  selectionButton(),
+                    padding: const EdgeInsets.all(16),
-                  _selectedType == CostCenterType.costCenter ? _buildCostCenterList() : _buildSerialNumberList(),
+                    children: [
+                      itemDetails(),
+                      12.height,
+                      selectionButton(),
+                      12.height,
+                      _selectedType == CostCenterType.costCenter ? _buildCostCenterList() : _buildSerialNumberList(),
+                    ],
+                  ).expanded,
                   FooterActionButton.footerContainer(
                     context: context,
                     child: AppFilledButton(
@@ -147,10 +155,12 @@ class _CostCenterFormViewState extends State<CostCenterFormView> {
   }
 
   Widget _buildCostCenterList() {
-    return ListView.builder(
+    return ListView.separated(
       itemCount: formModel.costCenters!.length + 1,
       shrinkWrap: true,
-      padding: const EdgeInsets.only(left: 16, right: 16, bottom: 16, top: 8),
+      padding: EdgeInsets.zero,
+      physics: const NeverScrollableScrollPhysics(),
+      separatorBuilder: (context, index) => 12.height,
       itemBuilder: (context, index) {
         if (index == formModel.costCenters!.length) {
           return AppFilledButton(
@@ -184,10 +194,12 @@ class _CostCenterFormViewState extends State<CostCenterFormView> {
   }
 
   Widget _buildSerialNumberList() {
-    return ListView.builder(
+    return ListView.separated(
       itemCount: formModel.serialNos!.length + 1,
       shrinkWrap: true,
-      padding: const EdgeInsets.only(left: 16, right: 16, bottom: 16, top: 8),
+      padding: EdgeInsets.zero,
+      physics: const NeverScrollableScrollPhysics(),
+      separatorBuilder: (context, index) => 12.height,
       itemBuilder: (context, index) {
         if (index == formModel.serialNos!.length) {
           return AppFilledButton(
@@ -217,36 +229,33 @@ class _CostCenterFormViewState extends State<CostCenterFormView> {
   }
 
   Widget selectionButton() {
-    return Padding(
+    return Row(
-      padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 12),
+      children: [
-      child: Row(
+        Expanded(
-        children: [
+          child: _radioItem(
-          Expanded(
+            value: CostCenterType.costCenter,
-            child: _radioItem(
+            groupValue: _selectedType,
-              value: CostCenterType.costCenter,
+            title: 'Cost Center',
-              groupValue: _selectedType,
+            onTap: () {
-              title: 'Cost Center',
+              setState(() {
-              onTap: () {
+                _selectedType = CostCenterType.costCenter;
-                setState(() {
+              });
-                  _selectedType = CostCenterType.costCenter;
+            },
-                });
-              },
-            ),
           ),
-          Expanded(
+        ),
-            child: _radioItem(
+        Expanded(
-              value: CostCenterType.serialNumber,
+          child: _radioItem(
-              groupValue: _selectedType,
+            value: CostCenterType.serialNumber,
-              title: 'Serial Number',
+            groupValue: _selectedType,
-              onTap: () {
+            title: 'Serial Number',
-                setState(() {
+            onTap: () {
-                  _selectedType = CostCenterType.serialNumber;
+              setState(() {
-                });
+                _selectedType = CostCenterType.serialNumber;
-              },
+              });
-            ),
+            },
           ),
-        ],
+        ),
-      ),
+      ],
     );
   }
 
@@ -263,6 +272,13 @@ class _CostCenterFormViewState extends State<CostCenterFormView> {
         children: [
           Radio<CostCenterType>(
             value: value,
+            activeColor: AppColor.primary10,
+            fillColor: WidgetStateColor.resolveWith((states) {
+              if (states.contains(WidgetState.selected)) {
+                return AppColor.primary10; // Thumb color when selected
+              }
+              return AppColor.icon2Color(context); // Thumb color when unselected (grey)
+            }),
             groupValue: groupValue,
             onChanged: (_) => onTap(),
             materialTapTargetSize: MaterialTapTargetSize.shrinkWrap,
@@ -279,34 +295,12 @@ class _CostCenterFormViewState extends State<CostCenterFormView> {
     return Column(
       crossAxisAlignment: CrossAxisAlignment.start,
       mainAxisSize: MainAxisSize.min,
+      spacing: 4,
       children: [
-        Text(
+        InfoTextWidget(label: "Item No", value: "\n${widget.itemNo ?? '-'}"),
-          'Item No'.addTranslation,
+        InfoTextWidget(label: "Item Description", value: "\n${widget.itemDescription ?? '-'}"),
-          style: AppTextStyles.bodyText.copyWith(color: context.isDark ? AppColor.neutral30 : AppColor.black10),
-        ),
-        8.height,
-        Text(
-          widget.itemNo ?? '-',
-          style: AppTextStyles.bodyText2.copyWith(color: context.isDark ? AppColor.neutral30 : AppColor.neutral120),
-        ),
-        8.height,
-        Text(
-          'Item Description'.addTranslation,
-          style: AppTextStyles.bodyText.copyWith(color: context.isDark ? AppColor.neutral30 : AppColor.black10),
-        ),
-        8.height,
-        Text(
-          widget.itemDescription ?? '-',
-          style: AppTextStyles.bodyText2.copyWith(color: context.isDark ? AppColor.neutral30 : AppColor.neutral120),
-        ),
       ],
-    ).toShadowContainer(
+    ).toShadowContainer(context);
-      context,
-      borderRadius: 20,
-      padding: 12,
-      showShadow: false,
-      margin: const EdgeInsets.only(left: 16, right: 16, top: 16, bottom: 8),
-    );
   }
 }
 
@@ -402,7 +396,7 @@ class CostCenterItemCard extends StatelessWidget {
           ],
         )
       ],
-    ).toShadowContainer(context, paddingObject: const EdgeInsets.symmetric(horizontal: 12, vertical: 12)).paddingOnly(bottom: 12);
+    ).toShadowContainer(context);
   }
 }
 
@@ -447,11 +441,6 @@ class SerialNumberItemCard extends StatelessWidget {
           style: Theme.of(context).textTheme.titleMedium,
         ),
       ],
-    )
+    ).toShadowContainer(context);
-        .toShadowContainer(
-          context,
-          paddingObject: const EdgeInsets.symmetric(horizontal: 12, vertical: 12),
-        )
-        .paddingOnly(bottom: 12);
   }
 }