fix: Remove unnecessary background color setting in report pages

This commit is contained in:
Achintha Isuru
2026-03-03 23:44:17 -05:00
parent 0726f1dfbb
commit 8e83e6dcbf
7 changed files with 9 additions and 22 deletions

View File

@@ -1,14 +1,13 @@
// ignore_for_file: always_specify_types, depend_on_referenced_packages, dead_code, dead_null_aware_expression, unused_local_variable, unused_import, sort_constructors_first, prefer_final_fields, prefer_const_constructors, deprecated_member_use, implicit_call_tearoffs import 'package:client_reports/src/presentation/blocs/coverage/coverage_bloc.dart';
import 'package:client_reports/src/presentation/blocs/coverage/coverage_bloc.dart';
import 'package:client_reports/src/presentation/blocs/coverage/coverage_event.dart'; import 'package:client_reports/src/presentation/blocs/coverage/coverage_event.dart';
import 'package:client_reports/src/presentation/blocs/coverage/coverage_state.dart'; import 'package:client_reports/src/presentation/blocs/coverage/coverage_state.dart';
import 'package:krow_domain/krow_domain.dart';
import 'package:core_localization/core_localization.dart'; import 'package:core_localization/core_localization.dart';
import 'package:design_system/design_system.dart'; import 'package:design_system/design_system.dart';
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:flutter_bloc/flutter_bloc.dart'; import 'package:flutter_bloc/flutter_bloc.dart';
import 'package:flutter_modular/flutter_modular.dart'; import 'package:flutter_modular/flutter_modular.dart';
import 'package:intl/intl.dart'; import 'package:intl/intl.dart';
import 'package:krow_domain/krow_domain.dart';
class CoverageReportPage extends StatefulWidget { class CoverageReportPage extends StatefulWidget {
const CoverageReportPage({super.key}); const CoverageReportPage({super.key});
@@ -27,7 +26,6 @@ class _CoverageReportPageState extends State<CoverageReportPage> {
create: (BuildContext context) => Modular.get<CoverageBloc>() create: (BuildContext context) => Modular.get<CoverageBloc>()
..add(LoadCoverageReport(startDate: _startDate, endDate: _endDate)), ..add(LoadCoverageReport(startDate: _startDate, endDate: _endDate)),
child: Scaffold( child: Scaffold(
backgroundColor: UiColors.bgMenu,
body: BlocBuilder<CoverageBloc, CoverageState>( body: BlocBuilder<CoverageBloc, CoverageState>(
builder: (BuildContext context, CoverageState state) { builder: (BuildContext context, CoverageState state) {
if (state is CoverageLoading) { if (state is CoverageLoading) {

View File

@@ -1,5 +1,4 @@
// ignore_for_file: always_specify_types, depend_on_referenced_packages, dead_code, dead_null_aware_expression, unused_local_variable, unused_import, sort_constructors_first, prefer_final_fields, prefer_const_constructors, deprecated_member_use, implicit_call_tearoffs, implementation_imports import 'package:client_reports/src/presentation/blocs/daily_ops/daily_ops_bloc.dart';
import 'package:client_reports/src/presentation/blocs/daily_ops/daily_ops_bloc.dart';
import 'package:client_reports/src/presentation/blocs/daily_ops/daily_ops_event.dart'; import 'package:client_reports/src/presentation/blocs/daily_ops/daily_ops_event.dart';
import 'package:client_reports/src/presentation/blocs/daily_ops/daily_ops_state.dart'; import 'package:client_reports/src/presentation/blocs/daily_ops/daily_ops_state.dart';
import 'package:core_localization/core_localization.dart'; import 'package:core_localization/core_localization.dart';
@@ -8,7 +7,7 @@ import 'package:flutter/material.dart';
import 'package:flutter_bloc/flutter_bloc.dart'; import 'package:flutter_bloc/flutter_bloc.dart';
import 'package:flutter_modular/flutter_modular.dart'; import 'package:flutter_modular/flutter_modular.dart';
import 'package:intl/intl.dart'; import 'package:intl/intl.dart';
import 'package:krow_domain/src/entities/reports/daily_ops_report.dart'; import 'package:krow_domain/krow_domain.dart';
class DailyOpsReportPage extends StatefulWidget { class DailyOpsReportPage extends StatefulWidget {
const DailyOpsReportPage({super.key}); const DailyOpsReportPage({super.key});
@@ -54,7 +53,6 @@ class _DailyOpsReportPageState extends State<DailyOpsReportPage> {
create: (BuildContext context) => Modular.get<DailyOpsBloc>() create: (BuildContext context) => Modular.get<DailyOpsBloc>()
..add(LoadDailyOpsReport(date: _selectedDate)), ..add(LoadDailyOpsReport(date: _selectedDate)),
child: Scaffold( child: Scaffold(
backgroundColor: UiColors.bgMenu,
body: BlocBuilder<DailyOpsBloc, DailyOpsState>( body: BlocBuilder<DailyOpsBloc, DailyOpsState>(
builder: (BuildContext context, DailyOpsState state) { builder: (BuildContext context, DailyOpsState state) {
if (state is DailyOpsLoading) { if (state is DailyOpsLoading) {

View File

@@ -28,7 +28,6 @@ class _ForecastReportPageState extends State<ForecastReportPage> {
create: (BuildContext context) => Modular.get<ForecastBloc>() create: (BuildContext context) => Modular.get<ForecastBloc>()
..add(LoadForecastReport(startDate: _startDate, endDate: _endDate)), ..add(LoadForecastReport(startDate: _startDate, endDate: _endDate)),
child: Scaffold( child: Scaffold(
backgroundColor: UiColors.bgMenu,
body: BlocBuilder<ForecastBloc, ForecastState>( body: BlocBuilder<ForecastBloc, ForecastState>(
builder: (BuildContext context, ForecastState state) { builder: (BuildContext context, ForecastState state) {
if (state is ForecastLoading) { if (state is ForecastLoading) {

View File

@@ -27,7 +27,6 @@ class _NoShowReportPageState extends State<NoShowReportPage> {
create: (BuildContext context) => Modular.get<NoShowBloc>() create: (BuildContext context) => Modular.get<NoShowBloc>()
..add(LoadNoShowReport(startDate: _startDate, endDate: _endDate)), ..add(LoadNoShowReport(startDate: _startDate, endDate: _endDate)),
child: Scaffold( child: Scaffold(
backgroundColor: UiColors.bgMenu,
body: BlocBuilder<NoShowBloc, NoShowState>( body: BlocBuilder<NoShowBloc, NoShowState>(
builder: (BuildContext context, NoShowState state) { builder: (BuildContext context, NoShowState state) {
if (state is NoShowLoading) { if (state is NoShowLoading) {

View File

@@ -1,5 +1,4 @@
// ignore_for_file: always_specify_types, depend_on_referenced_packages, dead_code, dead_null_aware_expression, unused_local_variable, unused_import, sort_constructors_first, prefer_final_fields, prefer_const_constructors, deprecated_member_use, implicit_call_tearoffs, implementation_imports import 'package:client_reports/src/presentation/blocs/performance/performance_bloc.dart';
import 'package:client_reports/src/presentation/blocs/performance/performance_bloc.dart';
import 'package:client_reports/src/presentation/blocs/performance/performance_event.dart'; import 'package:client_reports/src/presentation/blocs/performance/performance_event.dart';
import 'package:client_reports/src/presentation/blocs/performance/performance_state.dart'; import 'package:client_reports/src/presentation/blocs/performance/performance_state.dart';
import 'package:core_localization/core_localization.dart'; import 'package:core_localization/core_localization.dart';
@@ -7,7 +6,7 @@ import 'package:design_system/design_system.dart';
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:flutter_bloc/flutter_bloc.dart'; import 'package:flutter_bloc/flutter_bloc.dart';
import 'package:flutter_modular/flutter_modular.dart'; import 'package:flutter_modular/flutter_modular.dart';
import 'package:krow_domain/src/entities/reports/performance_report.dart'; import 'package:krow_domain/krow_domain.dart';
class PerformanceReportPage extends StatefulWidget { class PerformanceReportPage extends StatefulWidget {
const PerformanceReportPage({super.key}); const PerformanceReportPage({super.key});
@@ -26,7 +25,6 @@ class _PerformanceReportPageState extends State<PerformanceReportPage> {
create: (BuildContext context) => Modular.get<PerformanceBloc>() create: (BuildContext context) => Modular.get<PerformanceBloc>()
..add(LoadPerformanceReport(startDate: _startDate, endDate: _endDate)), ..add(LoadPerformanceReport(startDate: _startDate, endDate: _endDate)),
child: Scaffold( child: Scaffold(
backgroundColor: UiColors.bgMenu,
body: BlocBuilder<PerformanceBloc, PerformanceState>( body: BlocBuilder<PerformanceBloc, PerformanceState>(
builder: (BuildContext context, PerformanceState state) { builder: (BuildContext context, PerformanceState state) {
if (state is PerformanceLoading) { if (state is PerformanceLoading) {

View File

@@ -1,7 +1,5 @@
// ignore_for_file: always_specify_types, depend_on_referenced_packages, dead_code, dead_null_aware_expression, unused_local_variable, unused_import, sort_constructors_first, prefer_final_fields, prefer_const_constructors, deprecated_member_use, implicit_call_tearoffs, implementation_imports import 'package:client_reports/src/presentation/blocs/summary/reports_summary_bloc.dart';
import 'package:client_reports/src/presentation/blocs/summary/reports_summary_bloc.dart';
import 'package:client_reports/src/presentation/blocs/summary/reports_summary_event.dart'; import 'package:client_reports/src/presentation/blocs/summary/reports_summary_event.dart';
import 'package:design_system/design_system.dart';
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:flutter_bloc/flutter_bloc.dart'; import 'package:flutter_bloc/flutter_bloc.dart';
import 'package:flutter_modular/flutter_modular.dart'; import 'package:flutter_modular/flutter_modular.dart';
@@ -80,10 +78,9 @@ class _ReportsPageState extends State<ReportsPage>
@override @override
Widget build(BuildContext context) { Widget build(BuildContext context) {
return BlocProvider.value( return BlocProvider<ReportsSummaryBloc>.value(
value: _summaryBloc, value: _summaryBloc,
child: Scaffold( child: Scaffold(
backgroundColor: UiColors.bgMenu,
body: SingleChildScrollView( body: SingleChildScrollView(
child: Column( child: Column(
children: <Widget>[ children: <Widget>[

View File

@@ -1,5 +1,4 @@
// ignore_for_file: always_specify_types, depend_on_referenced_packages, dead_code, dead_null_aware_expression, unused_local_variable, unused_import, sort_constructors_first, prefer_final_fields, prefer_const_constructors, deprecated_member_use, implicit_call_tearoffs import 'package:client_reports/src/presentation/blocs/spend/spend_bloc.dart';
import 'package:client_reports/src/presentation/blocs/spend/spend_bloc.dart';
import 'package:client_reports/src/presentation/blocs/spend/spend_event.dart'; import 'package:client_reports/src/presentation/blocs/spend/spend_event.dart';
import 'package:client_reports/src/presentation/blocs/spend/spend_state.dart'; import 'package:client_reports/src/presentation/blocs/spend/spend_state.dart';
import 'package:core_localization/core_localization.dart'; import 'package:core_localization/core_localization.dart';
@@ -39,7 +38,6 @@ class _SpendReportPageState extends State<SpendReportPage> {
create: (BuildContext context) => Modular.get<SpendBloc>() create: (BuildContext context) => Modular.get<SpendBloc>()
..add(LoadSpendReport(startDate: _startDate, endDate: _endDate)), ..add(LoadSpendReport(startDate: _startDate, endDate: _endDate)),
child: Scaffold( child: Scaffold(
backgroundColor: UiColors.bgMenu,
body: BlocBuilder<SpendBloc, SpendState>( body: BlocBuilder<SpendBloc, SpendState>(
builder: (BuildContext context, SpendState state) { builder: (BuildContext context, SpendState state) {
if (state is SpendLoading) { if (state is SpendLoading) {