From 55f39fadbcd2f2b2c7366bbbef3310a145b28670 Mon Sep 17 00:00:00 2001 From: WaseemAbbasi22 <50428976+WaseemAbbasi22@users.noreply.github.com> Date: Thu, 27 Aug 2026 14:27:09 +0300 Subject: [PATCH] improvements --- .../cleaner_module/pages/one_time_task_preview_screen.dart | 1 + .../cm_module/views/components/activities_list_view.dart | 1 + .../cm_module/views/components/service_request_detail_view.dart | 2 ++ lib/modules/demo_module/demo_activities_page.dart | 1 + lib/modules/demo_module/demo_detail_view_page.dart | 2 ++ lib/modules/incident_module/incident_detail_page.dart | 1 + .../equipment_internal_audit_detail_page.dart | 1 + lib/modules/loan_module/pages/intallation_details_view.dart | 2 +- lib/modules/loan_module/pages/loan_equipment_detail_page.dart | 1 + .../medical_gas_inspection_detail_page.dart | 1 + .../tm_module/device_transfer/device_transfer_details.dart | 1 + lib/modules/tm_module/gas_refill/gas_refill_details.dart | 1 + lib/modules/tm_module/tasks/task_request_detail_view.dart | 1 + lib/views/widgets/equipment/asset_details_view.dart | 1 + lib/views/widgets/images/files_list.dart | 2 +- 15 files changed, 17 insertions(+), 2 deletions(-) diff --git a/lib/modules/cleaner_module/pages/one_time_task_preview_screen.dart b/lib/modules/cleaner_module/pages/one_time_task_preview_screen.dart index 19ef67fb..57433425 100644 --- a/lib/modules/cleaner_module/pages/one_time_task_preview_screen.dart +++ b/lib/modules/cleaner_module/pages/one_time_task_preview_screen.dart @@ -300,6 +300,7 @@ class OneTimeTaskPreviewScreen extends StatelessWidget { InfoHeader16Widget( 'Attachments', ), + 4.height, FilesList( images: taskData.attachments!.map((attachment) => URLs.getFileUrl(attachment.name ?? '') ?? '').toList(), showAsListView: true, diff --git a/lib/modules/cm_module/views/components/activities_list_view.dart b/lib/modules/cm_module/views/components/activities_list_view.dart index b611c3a4..1a4821c6 100644 --- a/lib/modules/cm_module/views/components/activities_list_view.dart +++ b/lib/modules/cm_module/views/components/activities_list_view.dart @@ -175,6 +175,7 @@ class _ActivitiesListViewState extends State { 2.height, const Divider().defaultStyle(context), InfoHeader16Widget("Attachments".addTranslation), + 4.height, FilesList( showAsListView: true, types: activity.activitySparePart!.acitiySparePartAttachments! diff --git a/lib/modules/cm_module/views/components/service_request_detail_view.dart b/lib/modules/cm_module/views/components/service_request_detail_view.dart index 4a081b6e..e6055eec 100644 --- a/lib/modules/cm_module/views/components/service_request_detail_view.dart +++ b/lib/modules/cm_module/views/components/service_request_detail_view.dart @@ -266,6 +266,7 @@ class _ServiceRequestDetailViewState extends State { if (_attachments.isNotEmpty) ...[ const Divider().defaultStyle(context), InfoHeader16Widget("Attachments".addTranslation), + 4.height, FilesList( showAsListView: true, images: _attachments.map((toElement) => URLs.getFileUrl(toElement.name!) ?? '').toList(), @@ -299,6 +300,7 @@ class _ServiceRequestDetailViewState extends State { if (_attachments.isNotEmpty) ...[ const Divider().defaultStyle(context), InfoHeader16Widget("Attachments".addTranslation), + 4.height, FilesList( images: _attachments.map((toElement) => URLs.getFileUrl(toElement.name!)!).toList(), showAsListView: true, diff --git a/lib/modules/demo_module/demo_activities_page.dart b/lib/modules/demo_module/demo_activities_page.dart index 7fa31795..0288afc4 100644 --- a/lib/modules/demo_module/demo_activities_page.dart +++ b/lib/modules/demo_module/demo_activities_page.dart @@ -56,6 +56,7 @@ class DemoActivitiesPage extends StatelessWidget { if (allAttachments.isNotEmpty) ...[ const Divider().defaultStyle(context), InfoHeader16Widget("Attachments".addTranslation), + 4.height, FilesList( showAsListView: true, types: allAttachments diff --git a/lib/modules/demo_module/demo_detail_view_page.dart b/lib/modules/demo_module/demo_detail_view_page.dart index 3278a6fb..5aa0939a 100644 --- a/lib/modules/demo_module/demo_detail_view_page.dart +++ b/lib/modules/demo_module/demo_detail_view_page.dart @@ -126,6 +126,7 @@ class _DemoDetailViewPageState extends State { if (allAttachments.isNotEmpty) ...[ const Divider().defaultStyle(context), InfoHeader16Widget("Attachments"), + 4.height, FilesList( showAsListView: true, images: allAttachments.map((e) => URLs.getFileUrl(e.name ?? '') ?? '').toList(), @@ -134,6 +135,7 @@ class _DemoDetailViewPageState extends State { if (suppAttachments.isNotEmpty) ...[ const Divider().defaultStyle(context), InfoHeader16Widget("Supplier Attachments"), + 4.height, FilesList( showAsListView: true, images: suppAttachments.whereType().map((e) => URLs.getFileUrl(e.name ?? '') ?? '').toList(), diff --git a/lib/modules/incident_module/incident_detail_page.dart b/lib/modules/incident_module/incident_detail_page.dart index d54f4b4d..d22f72d7 100644 --- a/lib/modules/incident_module/incident_detail_page.dart +++ b/lib/modules/incident_module/incident_detail_page.dart @@ -71,6 +71,7 @@ class IncidentDetailPage extends StatelessWidget { if (allAttachments.isNotEmpty) ...[ const Divider().defaultStyle(context), InfoHeader16Widget("Attachments".addTranslation), + 4.height, FilesList( showAsListView: true, types: allAttachments diff --git a/lib/modules/internal_audit_module/pages/equipment_internal_audit/equipment_internal_audit_detail_page.dart b/lib/modules/internal_audit_module/pages/equipment_internal_audit/equipment_internal_audit_detail_page.dart index 7276a1fa..239a366a 100644 --- a/lib/modules/internal_audit_module/pages/equipment_internal_audit/equipment_internal_audit_detail_page.dart +++ b/lib/modules/internal_audit_module/pages/equipment_internal_audit/equipment_internal_audit_detail_page.dart @@ -127,6 +127,7 @@ class _EquipmentInternalAuditDetailPageState extends State URLs.getFileUrl(e.name ?? '') ?? '').toList() ?? []), ], ], diff --git a/lib/modules/loan_module/pages/loan_equipment_detail_page.dart b/lib/modules/loan_module/pages/loan_equipment_detail_page.dart index 49800d56..b19b20ff 100644 --- a/lib/modules/loan_module/pages/loan_equipment_detail_page.dart +++ b/lib/modules/loan_module/pages/loan_equipment_detail_page.dart @@ -97,6 +97,7 @@ class _LoanEquipmentDetailPageState extends State { if (allAttachments.isNotEmpty) ...[ const Divider().defaultStyle(context), InfoHeader16Widget("Attachments".addTranslation), + 4.height, FilesList( showAsListView: true, types: allAttachments diff --git a/lib/modules/medical_gas_inspection/medical_gas_inspection_detail_page.dart b/lib/modules/medical_gas_inspection/medical_gas_inspection_detail_page.dart index 29eb131f..3840418c 100644 --- a/lib/modules/medical_gas_inspection/medical_gas_inspection_detail_page.dart +++ b/lib/modules/medical_gas_inspection/medical_gas_inspection_detail_page.dart @@ -104,6 +104,7 @@ class _MedicalGasInspectionDetailPageState extends State { if (_attachments.isNotEmpty) ...[ const Divider().defaultStyle(context), InfoHeader16Widget("Attachments".addTranslation), + 4.height, FilesList( showAsListView: true, types: _model?.assetTransferAttachments!=null? diff --git a/lib/modules/tm_module/gas_refill/gas_refill_details.dart b/lib/modules/tm_module/gas_refill/gas_refill_details.dart index cfd42605..77b5b70a 100644 --- a/lib/modules/tm_module/gas_refill/gas_refill_details.dart +++ b/lib/modules/tm_module/gas_refill/gas_refill_details.dart @@ -238,6 +238,7 @@ class _GasRefillDetailsPageState extends State { if (_attachments.isNotEmpty) ...[ const Divider().defaultStyle(context), InfoHeader16Widget("Attachments".addTranslation), + 4.height, FilesList( showAsListView: true, types: _model.gasRefillAttachments?.whereType().where((e) => e.documentType != null && e.documentType!.name != null).map((toElement) => toElement.documentType!.name!).toList() ?? [], diff --git a/lib/modules/tm_module/tasks/task_request_detail_view.dart b/lib/modules/tm_module/tasks/task_request_detail_view.dart index 13ad69f8..b60740fb 100644 --- a/lib/modules/tm_module/tasks/task_request_detail_view.dart +++ b/lib/modules/tm_module/tasks/task_request_detail_view.dart @@ -160,6 +160,7 @@ class _TaskRequestDetailsViewState extends State { if (taskProvider.taskRequestModel?.taskJobAttachments?.isNotEmpty ?? false) ...[ const Divider().defaultStyle(context), InfoHeader16Widget("Attachments".addTranslation), + 4.height, FilesList( showAsListView: true, types: taskProvider.taskRequestModel!.taskJobAttachments!.whereType().where((e) => e.name != null).map((toElement) => toElement.documentType?.name ?? '').toList(), diff --git a/lib/views/widgets/equipment/asset_details_view.dart b/lib/views/widgets/equipment/asset_details_view.dart index 5ccf24c3..5db28370 100644 --- a/lib/views/widgets/equipment/asset_details_view.dart +++ b/lib/views/widgets/equipment/asset_details_view.dart @@ -318,6 +318,7 @@ class _AssetDetailsViewState extends State with SingleTickerPr child: const Divider().defaultStyle(context), ), InfoHeader16Widget("Attachments".addTranslation), + 4.height, FadeTransition( opacity: attachmentAnimation, child: SlideTransition( diff --git a/lib/views/widgets/images/files_list.dart b/lib/views/widgets/images/files_list.dart index 265b5e82..67d71ce6 100644 --- a/lib/views/widgets/images/files_list.dart +++ b/lib/views/widgets/images/files_list.dart @@ -115,7 +115,7 @@ class FilesList extends StatelessWidget { icon: "download".toSvgAsset(height: 24, width: 24, color: AppColor.icon2Color(context)), padding: EdgeInsets.zero, visualDensity: VisualDensity.compact, - tooltip: "Download", + // tooltip: "Download", onPressed: () async { try { if (!await launchUrl(Uri.parse(images[itemIndex]), mode: LaunchMode.externalApplication)) {