refactor: update color definitions and improve UI component structure; handle null safety in repository data fetching

This commit is contained in:
Achintha Isuru
2026-03-17 13:20:04 -04:00
parent a0d5a18e6f
commit e83b8fff1c
13 changed files with 49 additions and 43 deletions

View File

@@ -245,7 +245,7 @@ class UiColors {
static const Color buttonPrimaryStill = primary;
/// Primary button hover (#082EB2)
static const Color buttonPrimaryHover = Color(0xFF082EB2);
static const Color buttonPrimaryHover = Color.fromARGB(255, 8, 46, 178);
/// Primary button inactive (#F1F3F5)
static const Color buttonPrimaryInactive = secondary;

View File

@@ -368,7 +368,6 @@ class UiTypography {
fontWeight: FontWeight.w400,
fontSize: 12,
height: 1.5,
letterSpacing: -0.1,
color: UiColors.textPrimary,
);

View File

@@ -67,39 +67,45 @@ class UiNoticeBanner extends StatelessWidget {
color: backgroundColor ?? UiColors.primary.withValues(alpha: 0.08),
borderRadius: borderRadius ?? UiConstants.radiusLg,
),
child: Row(
crossAxisAlignment: CrossAxisAlignment.start,
child: Column(
children: <Widget>[
if (leading != null) ...<Widget>[
leading!,
const SizedBox(width: UiConstants.space3),
] else if (icon != null) ...<Widget>[
Icon(icon, color: iconColor ?? UiColors.primary, size: 24),
const SizedBox(width: UiConstants.space3),
],
Expanded(
child: Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: <Widget>[
Row(
crossAxisAlignment: CrossAxisAlignment.start,
children: <Widget>[
if (leading != null) ...<Widget>[
leading!,
const SizedBox(width: UiConstants.space3),
] else if (icon != null) ...<Widget>[
Icon(icon, color: iconColor ?? UiColors.primary, size: 24),
const SizedBox(width: UiConstants.space3),
Text(
title,
style: UiTypography.body2b.copyWith(color: titleColor),
),
if (description != null) ...<Widget>[
const SizedBox(height: 2),
Text(
description!,
style: UiTypography.body3r.copyWith(
color: descriptionColor,
),
style: UiTypography.body2b.copyWith(
color: titleColor ?? UiColors.primary,
),
],
if (action != null) ...<Widget>[
const SizedBox(height: UiConstants.space2),
action!,
],
),
],
),
],
),
Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: <Widget>[
if (description != null) ...<Widget>[
const SizedBox(height: UiConstants.space2),
Text(
description!,
style: UiTypography.body3r.copyWith(
//color: descriptionColor ?? UiColors.textSecondary,
color: descriptionColor ?? UiColors.primary,
),
),
],
if (action != null) ...<Widget>[
const SizedBox(height: UiConstants.space2),
action!,
],
],
),
],
),

View File

@@ -29,7 +29,7 @@ class CertificatesRepositoryImpl implements CertificatesRepository {
final ApiResponse response =
await _api.get(StaffEndpoints.certificates);
final List<dynamic> items =
response.data['certificates'] as List<dynamic>;
response.data['certificates'] as List<dynamic>? ?? <dynamic>[];
return items
.map((dynamic json) =>
StaffCertificate.fromJson(json as Map<String, dynamic>))

View File

@@ -28,7 +28,7 @@ class DocumentsRepositoryImpl implements DocumentsRepository {
Future<List<ProfileDocument>> getDocuments() async {
final ApiResponse response =
await _api.get(StaffEndpoints.documents);
final List<dynamic> items = response.data['documents'] as List<dynamic>;
final List<dynamic> items = response.data['documents'] as List<dynamic>? ?? <dynamic>[];
return items
.map((dynamic json) =>
ProfileDocument.fromJson(json as Map<String, dynamic>))

View File

@@ -19,7 +19,7 @@ class TaxFormsRepositoryImpl implements TaxFormsRepository {
Future<List<TaxForm>> getTaxForms() async {
final ApiResponse response =
await _api.get(StaffEndpoints.taxForms);
final List<dynamic> items = response.data['taxForms'] as List<dynamic>;
final List<dynamic> items = response.data['taxForms'] as List<dynamic>? ?? <dynamic>[];
return items
.map((dynamic json) =>
TaxForm.fromJson(json as Map<String, dynamic>))

View File

@@ -17,7 +17,7 @@ class BankAccountRepositoryImpl implements BankAccountRepository {
Future<List<BankAccount>> getAccounts() async {
final ApiResponse response =
await _api.get(StaffEndpoints.bankAccounts);
final List<dynamic> items = response.data['accounts'] as List<dynamic>;
final List<dynamic> items = response.data['accounts'] as List<dynamic>? ?? <dynamic>[];
return items
.map((dynamic json) =>
BankAccount.fromJson(json as Map<String, dynamic>))

View File

@@ -22,7 +22,7 @@ class TimeCardRepositoryImpl implements TimeCardRepository {
'month': month.month,
},
);
final List<dynamic> items = response.data['entries'] as List<dynamic>;
final List<dynamic> items = response.data['entries'] as List<dynamic>? ?? <dynamic>[];
return items
.map((dynamic json) =>
TimeCardEntry.fromJson(json as Map<String, dynamic>))

View File

@@ -28,7 +28,7 @@ class AttireRepositoryImpl implements AttireRepository {
@override
Future<List<AttireChecklist>> getAttireOptions() async {
final ApiResponse response = await _api.get(StaffEndpoints.attire);
final List<dynamic> items = response.data['items'] as List<dynamic>;
final List<dynamic> items = response.data['items'] as List<dynamic>? ?? <dynamic>[];
return items
.map((dynamic json) =>
AttireChecklist.fromJson(json as Map<String, dynamic>))

View File

@@ -18,7 +18,7 @@ class EmergencyContactRepositoryImpl
Future<List<EmergencyContact>> getContacts() async {
final ApiResponse response =
await _api.get(StaffEndpoints.emergencyContacts);
final List<dynamic> items = response.data['contacts'] as List<dynamic>;
final List<dynamic> items = response.data['contacts'] as List<dynamic>? ?? <dynamic>[];
return items
.map((dynamic json) =>
EmergencyContact.fromJson(json as Map<String, dynamic>))

View File

@@ -17,14 +17,14 @@ class ExperienceRepositoryImpl implements ExperienceRepositoryInterface {
Future<List<String>> getIndustries() async {
final ApiResponse response =
await _api.get(StaffEndpoints.industries);
final List<dynamic> items = response.data['industries'] as List<dynamic>;
final List<dynamic> items = response.data['industries'] as List<dynamic>? ?? <dynamic>[];
return items.map((dynamic e) => e.toString()).toList();
}
@override
Future<List<String>> getSkills() async {
final ApiResponse response = await _api.get(StaffEndpoints.skills);
final List<dynamic> items = response.data['skills'] as List<dynamic>;
final List<dynamic> items = response.data['skills'] as List<dynamic>? ?? <dynamic>[];
return items.map((dynamic e) => e.toString()).toList();
}

View File

@@ -9,10 +9,10 @@ import 'package:staff_faqs/src/domain/repositories/faqs_repository_interface.dar
/// Fetches FAQ data from the V2 REST backend via [ApiService].
class FaqsRepositoryImpl implements FaqsRepositoryInterface {
/// Creates a [FaqsRepositoryImpl] backed by the given [apiService].
FaqsRepositoryImpl({required ApiService apiService})
FaqsRepositoryImpl({required BaseApiService apiService})
: _apiService = apiService;
final ApiService _apiService;
final BaseApiService _apiService;
@override
Future<List<FaqCategory>> getFaqs() async {
@@ -40,7 +40,7 @@ class FaqsRepositoryImpl implements FaqsRepositoryInterface {
/// Parses the `items` array from a V2 API response into [FaqCategory] list.
List<FaqCategory> _parseCategories(ApiResponse response) {
final List<dynamic> items = response.data['items'] as List<dynamic>;
final List<dynamic> items = response.data['items'] as List<dynamic>? ?? <dynamic>[];
return items
.map(
(dynamic item) =>

View File

@@ -1,5 +1,6 @@
import 'package:flutter_modular/flutter_modular.dart';
import 'package:krow_core/core.dart';
import 'package:krow_domain/krow_domain.dart' show BaseApiService;
import 'package:staff_faqs/src/data/repositories_impl/faqs_repository_impl.dart';
import 'package:staff_faqs/src/domain/repositories/faqs_repository_interface.dart';
@@ -21,7 +22,7 @@ class FaqsModule extends Module {
// Repository
i.addLazySingleton<FaqsRepositoryInterface>(
() => FaqsRepositoryImpl(
apiService: i<ApiService>(),
apiService: i.get<BaseApiService>(),
),
);