Merge pull request #210 from flutter-news-app-full-source-code/refactor/overhaul-headlines-saved-filter

Refactor/overhaul headlines saved filter

Author: fulleni

README.md

@@ -26,19 +26,20 @@ Explore the high-level domains below to see how.
 <details>
 <summary><strong>📰 Content Discovery & User Engagement</strong></summary>
 
-### 📱 Dynamic, High-Performance News Feed
-A beautiful, infinitely scrolling feed serves as the core of the user experience. It's not just a list; it's an intelligent content delivery system.
+### 📱 Dynamic & Personalized News Feed
+A beautiful, infinitely scrolling feed serves as the core of the user experience. It's not just a list; it's an intelligent, user-driven content delivery system.
 - **Instantaneous Content Switching:** An intelligent, session-based cache pre-fetches and holds data, eliminating loading spinners when users switch between their preferred content views.
 - **Personalized Viewing:** Users control their experience with settings for information density and image presentation, adapting the feed to their reading style.
 - **Smart In-Feed Prompts:** The feed dynamically injects context-aware items like calls-to-action and content suggestions, driven by configurable rules to avoid user fatigue.
 > **Your Advantage:** You get a world-class, production-quality feed system out of the box. Skip the complex UI, state management, and performance optimization work.
 
 ---
 
-### 🔎 Powerful & Intuitive Search
+### 🔎 Powerful Content Curation & Discovery
 Give users the tools to find exactly what they're looking for with a multi-faceted discovery system.
 - **Customizable Filter Bar:** A persistent, one-tap filter bar provides instant access to pre-defined and user-created content streams.
-- **Advanced Content Curation:** A dedicated UI allows users to construct and save highly specific news feeds by combining various content categories, publishers, and regions of interest.
+- **Advanced Content Curation:** A dedicated management UI allows users to construct, save, and manage highly specific news feeds. Users can save frequently used filters for on-demand use and pin them for one-tap access on the main feed.
+- **Proactive Notification Subscriptions:** When saving a filter, users can subscribe to receive push notifications—such as breaking news alerts or daily digests—for content that matches their specific criteria.
 - **Dedicated Discovery Hub:** Users can browse publishers by category in horizontally scrolling carousels, apply regional filters, and perform targeted searches.
 > **Your Advantage:** Deliver powerful content discovery tools that keep users engaged, increase session duration, and encourage return visits.
 

lib/account/view/account_page.dart

@@ -199,12 +199,6 @@ class AccountPage extends StatelessWidget {
           onTap: () => context.pushNamed(Routes.accountSavedHeadlinesName),
         ),
         const Divider(),
-        buildTile(
-          icon: Icons.filter_alt_outlined,
-          title: l10n.accountSavedFiltersTile,
-          onTap: () => context.pushNamed(Routes.accountSavedFiltersName),
-        ),
-        const Divider(),
       ],
     );
   }

lib/account/view/saved_filters_page.dart

@@ -75,10 +75,10 @@ class AppBloc extends Bloc<AppEvent, AppState> {
     on<AppPeriodicConfigFetchRequested>(_onAppPeriodicConfigFetchRequested);
     on<AppUserFeedDecoratorShown>(_onAppUserFeedDecoratorShown);
     on<AppUserContentPreferencesChanged>(_onAppUserContentPreferencesChanged);
-    on<SavedFilterAdded>(_onSavedFilterAdded);
-    on<SavedFilterUpdated>(_onSavedFilterUpdated);
-    on<SavedFilterDeleted>(_onSavedFilterDeleted);
-    on<SavedFiltersReordered>(_onSavedFiltersReordered);
+    on<SavedHeadlineFilterAdded>(_onSavedHeadlineFilterAdded);
+    on<SavedHeadlineFilterUpdated>(_onSavedHeadlineFilterUpdated);
+    on<SavedHeadlineFilterDeleted>(_onSavedHeadlineFilterDeleted);
+    on<SavedHeadlineFiltersReordered>(_onSavedHeadlineFiltersReordered);
     on<AppLogoutRequested>(_onLogoutRequested);
   }
 
@@ -439,124 +439,135 @@ class AppBloc extends Bloc<AppEvent, AppState> {
     }
   }
 
-  /// Handles adding a new saved filter to the user's content preferences.
-  Future<void> _onSavedFilterAdded(
-    SavedFilterAdded event,
+  /// Handles adding a new saved headline filter to the user's content
+  /// preferences.
+  Future<void> _onSavedHeadlineFilterAdded(
+    SavedHeadlineFilterAdded event,
     Emitter<AppState> emit,
   ) async {
     _logger.fine(
-      '[AppBloc] SavedFilterAdded event received for filter: "${event.filter.name}".',
+      '[AppBloc] SavedHeadlineFilterAdded event received for filter: '
+      '"${event.filter.name}".',
     );
     // This method modifies the preferences in memory and then delegates the
     // persistence and final state update to the AppUserContentPreferencesChanged event.
     if (state.userContentPreferences == null) {
       _logger.warning(
-        '[AppBloc] Skipping SavedFilterAdded: UserContentPreferences not loaded.',
+        '[AppBloc] Skipping SavedHeadlineFilterAdded: UserContentPreferences '
+        'not loaded.',
       );
       return;
     }
 
-    final updatedSavedFilters = List<SavedFilter>.from(
-      state.userContentPreferences!.savedFilters,
+    final updatedSavedFilters = List<SavedHeadlineFilter>.from(
+      state.userContentPreferences!.savedHeadlineFilters,
     )..add(event.filter);
 
     final updatedPreferences = state.userContentPreferences!.copyWith(
-      savedFilters: updatedSavedFilters,
+      savedHeadlineFilters: updatedSavedFilters,
     );
 
     add(AppUserContentPreferencesChanged(preferences: updatedPreferences));
   }
 
-  /// Handles updating an existing saved filter (e.g., renaming it).
-  Future<void> _onSavedFilterUpdated(
-    SavedFilterUpdated event,
+  /// Handles updating an existing saved headline filter (e.g., renaming it).
+  Future<void> _onSavedHeadlineFilterUpdated(
+    SavedHeadlineFilterUpdated event,
     Emitter<AppState> emit,
   ) async {
     _logger.fine(
-      '[AppBloc] SavedFilterUpdated event received for filter id: ${event.filter.id}.',
+      '[AppBloc] SavedHeadlineFilterUpdated event received for filter id: '
+      '${event.filter.id}.',
     );
     // This method modifies the preferences in memory and then delegates the
     // persistence and final state update to the AppUserContentPreferencesChanged event.
     if (state.userContentPreferences == null) {
       _logger.warning(
-        '[AppBloc] Skipping SavedFilterUpdated: UserContentPreferences not loaded.',
+        '[AppBloc] Skipping SavedHeadlineFilterUpdated: '
+        'UserContentPreferences not loaded.',
       );
       return;
     }
 
-    final originalFilters = state.userContentPreferences!.savedFilters;
+    final originalFilters = state.userContentPreferences!.savedHeadlineFilters;
     final index = originalFilters.indexWhere((f) => f.id == event.filter.id);
 
     if (index == -1) {
       _logger.warning(
-        '[AppBloc] Skipping SavedFilterUpdated: Filter with id ${event.filter.id} not found.',
+        '[AppBloc] Skipping SavedHeadlineFilterUpdated: Filter with id '
+        '${event.filter.id} not found.',
       );
       return;
     }
 
-    final updatedSavedFilters = List<SavedFilter>.from(originalFilters)
+    final updatedSavedFilters = List<SavedHeadlineFilter>.from(originalFilters)
       ..[index] = event.filter;
 
     final updatedPreferences = state.userContentPreferences!.copyWith(
-      savedFilters: updatedSavedFilters,
+      savedHeadlineFilters: updatedSavedFilters,
     );
 
     add(AppUserContentPreferencesChanged(preferences: updatedPreferences));
   }
 
-  /// Handles deleting a saved filter from the user's content preferences.
-  Future<void> _onSavedFilterDeleted(
-    SavedFilterDeleted event,
+  /// Handles deleting a saved headline filter from the user's content
+  /// preferences.
+  Future<void> _onSavedHeadlineFilterDeleted(
+    SavedHeadlineFilterDeleted event,
     Emitter<AppState> emit,
   ) async {
     _logger.fine(
-      '[AppBloc] SavedFilterDeleted event received for filter id: ${event.filterId}.',
+      '[AppBloc] SavedHeadlineFilterDeleted event received for filter id: '
+      '${event.filterId}.',
     );
     // This method modifies the preferences in memory and then delegates the
     // persistence and final state update to the AppUserContentPreferencesChanged event.
     if (state.userContentPreferences == null) {
       _logger.warning(
-        '[AppBloc] Skipping SavedFilterDeleted: UserContentPreferences not loaded.',
+        '[AppBloc] Skipping SavedHeadlineFilterDeleted: '
+        'UserContentPreferences not loaded.',
       );
       return;
     }
 
-    final updatedSavedFilters = List<SavedFilter>.from(
-      state.userContentPreferences!.savedFilters,
+    final updatedSavedFilters = List<SavedHeadlineFilter>.from(
+      state.userContentPreferences!.savedHeadlineFilters,
     )..removeWhere((f) => f.id == event.filterId);
 
     if (updatedSavedFilters.length ==
-        state.userContentPreferences!.savedFilters.length) {
+        state.userContentPreferences!.savedHeadlineFilters.length) {
       _logger.warning(
-        '[AppBloc] Skipping SavedFilterDeleted: Filter with id ${event.filterId} not found.',
+        '[AppBloc] Skipping SavedHeadlineFilterDeleted: Filter with id '
+        '${event.filterId} not found.',
       );
       return;
     }
 
     final updatedPreferences = state.userContentPreferences!.copyWith(
-      savedFilters: updatedSavedFilters,
+      savedHeadlineFilters: updatedSavedFilters,
     );
 
     add(AppUserContentPreferencesChanged(preferences: updatedPreferences));
   }
 
-  /// Handles reordering the list of saved filters.
-  Future<void> _onSavedFiltersReordered(
-    SavedFiltersReordered event,
+  /// Handles reordering the list of saved headline filters.
+  Future<void> _onSavedHeadlineFiltersReordered(
+    SavedHeadlineFiltersReordered event,
     Emitter<AppState> emit,
   ) async {
-    _logger.fine('[AppBloc] SavedFiltersReordered event received.');
+    _logger.fine('[AppBloc] SavedHeadlineFiltersReordered event received.');
     // This method modifies the preferences in memory and then delegates the
     // persistence and final state update to the AppUserContentPreferencesChanged event.
     if (state.userContentPreferences == null) {
       _logger.warning(
-        '[AppBloc] Skipping SavedFiltersReordered: UserContentPreferences not loaded.',
+        '[AppBloc] Skipping SavedHeadlineFiltersReordered: '
+        'UserContentPreferences not loaded.',
       );
       return;
     }
 
     final updatedPreferences = state.userContentPreferences!.copyWith(
-      savedFilters: event.reorderedFilters,
+      savedHeadlineFilters: event.reorderedFilters,
     );
 
     add(AppUserContentPreferencesChanged(preferences: updatedPreferences));