Skip to content

Commit 07ddcb1

Browse files
committed
fix(headlines-feed): wire reset filters button to trigger "All" state refresh
- Update HeadlinesFilterPage to dispatch AllFilterSelected event when reset button is pressed - Add AdThemeStyle.fromTheme(theme) to the event to preserve current theme - Close the filter page after resetting filters
1 parent 264f571 commit 07ddcb1

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

lib/headlines-feed/view/headlines_filter_page.dart

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -266,9 +266,15 @@ class _HeadlinesFilterViewState extends State<_HeadlinesFilterView> {
266266
icon: const Icon(Icons.refresh),
267267
tooltip: l10n.headlinesFeedFilterResetButton,
268268
onPressed: () {
269-
context.read<HeadlinesFilterBloc>().add(
270-
const FilterSelectionsCleared(),
269+
// Dispatch event to clear filters in the feed bloc
270+
// and trigger a refresh to the "All" state.
271+
context.read<HeadlinesFeedBloc>().add(
272+
AllFilterSelected(
273+
adThemeStyle: AdThemeStyle.fromTheme(theme),
274+
),
271275
);
276+
// Close the filter page.
277+
context.pop();
272278
},
273279
),
274280
// Manage Saved Filters Button

0 commit comments

Comments
 (0)