Skip to content

Commit 96ceb40

Browse files
committed
style: misc
1 parent 87f3fb9 commit 96ceb40

File tree

3 files changed

+9
-14
lines changed

3 files changed

+9
-14
lines changed

lib/headline-details/view/headline_details_page.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,8 @@ class _HeadlineDetailsView extends StatelessWidget {
292292
// Use country.flagUrl for the avatar
293293
chips.add(
294294
Chip(
295-
avatar: CircleAvatar( // Use CircleAvatar for better image display
295+
avatar: CircleAvatar(
296+
// Use CircleAvatar for better image display
296297
radius: chipAvatarSize / 2, // Adjust radius as needed
297298
backgroundColor: Colors.transparent, // Avoid background color clash
298299
backgroundImage: NetworkImage(headline.eventCountry!.flagUrl),

lib/headlines-feed/bloc/headlines_feed_event.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ final class HeadlinesFeedFilterChanged extends HeadlinesFeedEvent {
3737
/// {@macro headlines_feed_filter_changed}
3838
const HeadlinesFeedFilterChanged({
3939
this.categories, // Changed from category
40-
this.sources, // Changed from source
40+
this.sources, // Changed from source
4141
this.eventCountries, // Changed from eventCountry
4242
});
4343

lib/headlines-feed/view/headlines_feed_page.dart

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
import 'package:flutter/material.dart';
22
import 'package:flutter_bloc/flutter_bloc.dart';
33
import 'package:go_router/go_router.dart';
4+
import 'package:ht_categories_client/ht_categories_client.dart'; // Import Category
5+
import 'package:ht_countries_client/ht_countries_client.dart'; // Import Country
46
import 'package:ht_headlines_repository/ht_headlines_repository.dart';
57
import 'package:ht_main/headlines-feed/bloc/headlines_feed_bloc.dart';
68
import 'package:ht_main/headlines-feed/widgets/headline_item_widget.dart';
79
import 'package:ht_main/l10n/l10n.dart';
810
import 'package:ht_main/router/routes.dart';
911
import 'package:ht_main/shared/constants/constants.dart';
10-
import 'package:ht_categories_client/ht_categories_client.dart'; // Import Category
11-
import 'package:ht_countries_client/ht_countries_client.dart'; // Import Country
1212
import 'package:ht_main/shared/widgets/failure_state_widget.dart';
1313
import 'package:ht_main/shared/widgets/loading_state_widget.dart';
1414
import 'package:ht_sources_client/ht_sources_client.dart'; // Import Source
@@ -260,7 +260,7 @@ class _HeadlinesFilterBottomSheetState
260260
style: Theme.of(context).textTheme.titleMedium,
261261
),
262262
const SizedBox(height: AppSpacing.md),
263-
// TODO(cline): Implement multi-select UI for categories
263+
// TODO(fulleni): Implement multi-select UI for categories
264264
// Fetch available categories from a repository
265265
// Use Wrap + FilterChip to display options
266266
// Update selectedCategories list in setState when chips are toggled
@@ -283,7 +283,7 @@ class _HeadlinesFilterBottomSheetState
283283
style: Theme.of(context).textTheme.titleMedium,
284284
),
285285
const SizedBox(height: AppSpacing.md),
286-
// TODO(cline): Implement multi-select UI for sources
286+
// TODO(fulleni): Implement multi-select UI for sources
287287
// Fetch available sources from a repository
288288
// Use Wrap + FilterChip to display options
289289
// Update selectedSources list in setState when chips are toggled
@@ -306,7 +306,7 @@ class _HeadlinesFilterBottomSheetState
306306
style: Theme.of(context).textTheme.titleMedium,
307307
),
308308
const SizedBox(height: AppSpacing.md),
309-
// TODO(cline): Implement multi-select UI for event countries
309+
// TODO(fulleni): Implement multi-select UI for event countries
310310
// Fetch available countries from a repository
311311
// Use Wrap + FilterChip to display options
312312
// Update selectedEventCountries list in setState when chips are toggled
@@ -355,13 +355,7 @@ class _HeadlinesFilterBottomSheetState
355355
selectedEventCountries.clear();
356356
});
357357
// Dispatch event with null lists to clear filters in BLoC
358-
widget.bloc.add(
359-
const HeadlinesFeedFilterChanged(
360-
categories: null,
361-
sources: null,
362-
eventCountries: null,
363-
),
364-
);
358+
widget.bloc.add(const HeadlinesFeedFilterChanged());
365359
Navigator.pop(context);
366360
},
367361
child: Text(l10n.headlinesFeedFilterResetButton),

0 commit comments

Comments
 (0)