Skip to content

Commit eb65f46

Browse files
committed
refactor(account): move saved filters management to account section
- Rename `ManageSavedFiltersPage` to `SavedFiltersPage` - Move file from `lib/headlines-feed/view/` to `lib/account/view/` - Update routing configuration to reflect new location - Adjust documentation comments to reflect naming change
1 parent 8ed6c8d commit eb65f46

File tree

2 files changed

+13
-12
lines changed

2 files changed

+13
-12
lines changed

lib/headlines-feed/view/manage_saved_filters_page.dart renamed to lib/account/view/saved_filters_page.dart

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,17 +6,17 @@ import 'package:flutter_news_app_mobile_client_full_source_code/headlines-feed/w
66
import 'package:flutter_news_app_mobile_client_full_source_code/l10n/l10n.dart';
77
import 'package:ui_kit/ui_kit.dart';
88

9-
/// {@template manage_saved_filters_page}
9+
/// {@template saved_filters_page}
1010
/// A page for managing saved feed filters, allowing users to reorder,
1111
/// rename, or delete them.
1212
///
1313
/// Reordering is handled via a [ReorderableListView], which dispatches a
1414
/// [SavedFiltersReordered] event to the [AppBloc] to persist the new order.
1515
/// Renaming and deletion are handled via a [PopupMenuButton] on each list item.
1616
/// {@endtemplate}
17-
class ManageSavedFiltersPage extends StatelessWidget {
18-
/// {@macro manage_saved_filters_page}
19-
const ManageSavedFiltersPage({super.key});
17+
class SavedFiltersPage extends StatelessWidget {
18+
/// {@macro saved_filters_page}
19+
const SavedFiltersPage({super.key});
2020

2121
@override
2222
Widget build(BuildContext context) {

lib/router/router.dart

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ import 'package:flutter_news_app_mobile_client_full_source_code/headlines-feed/b
3232
import 'package:flutter_news_app_mobile_client_full_source_code/headlines-feed/view/country_filter_page.dart';
3333
import 'package:flutter_news_app_mobile_client_full_source_code/headlines-feed/view/headlines_feed_page.dart';
3434
import 'package:flutter_news_app_mobile_client_full_source_code/headlines-feed/view/headlines_filter_page.dart';
35-
import 'package:flutter_news_app_mobile_client_full_source_code/headlines-feed/view/manage_saved_filters_page.dart';
35+
import 'package:flutter_news_app_mobile_client_full_source_code/account/view/saved_filters_page.dart';
3636
import 'package:flutter_news_app_mobile_client_full_source_code/headlines-feed/view/source_filter_page.dart';
3737
import 'package:flutter_news_app_mobile_client_full_source_code/headlines-feed/view/source_list_filter_page.dart';
3838
import 'package:flutter_news_app_mobile_client_full_source_code/headlines-feed/view/topic_filter_page.dart';
@@ -482,13 +482,6 @@ GoRouter createRouter({
482482
);
483483
},
484484
routes: [
485-
// Sub-route for managing saved filters
486-
GoRoute(
487-
path: Routes.manageSavedFilters,
488-
name: Routes.manageSavedFiltersName,
489-
builder: (context, state) =>
490-
const ManageSavedFiltersPage(),
491-
),
492485
// Sub-route for topic selection
493486
GoRoute(
494487
path: Routes.feedFilterTopics,
@@ -601,6 +594,14 @@ GoRouter createRouter({
601594
);
602595
},
603596
),
597+
// Route for managing saved filters, now under the account section.
598+
GoRoute(
599+
path: Routes.accountSavedFilters,
600+
name: Routes.accountSavedFiltersName,
601+
builder: (context, state) {
602+
return const SavedFiltersPage();
603+
},
604+
),
604605
],
605606
),
606607
],

0 commit comments

Comments
 (0)