fix(headlines): fixed notification ruting issue

Author: fulleni

lib/headlines-feed/view/headlines_feed_page.dart

@@ -75,20 +75,20 @@ class _HeadlinesFeedViewState extends State<_HeadlinesFeedView> {
 
     return Scaffold(
       appBar: AppBar(
-        centerTitle: true, // Center the title
-        leading: IconButton( // Add leading icon button
-          icon: const Icon(Icons.notifications_outlined),
-          tooltip: l10n.notificationsTooltip, // Add tooltip for accessibility
-          onPressed: () {
-            context.goNamed(Routes.notificationsName);
-          },
-        ),
         title: Text(
           'HT', // TODO(fulleni): Localize this title
           style: textTheme.titleLarge?.copyWith(fontWeight: FontWeight.bold),
         ),
         actions: [
-          // Removed Search IconButton
+          IconButton(
+            icon: const Icon(Icons.notifications_outlined),
+            tooltip: l10n.notificationsTooltip, // Add tooltip for accessibility
+            onPressed: () {
+              context.goNamed(
+                Routes.notificationsName,
+              ); // Ensure correct route name
+            },
+          ),
           BlocBuilder<HeadlinesFeedBloc, HeadlinesFeedState>(
             builder: (context, state) {
               var isFilterApplied = false;

lib/router/router.dart

@@ -76,7 +76,9 @@ GoRouter createRouter({required ValueNotifier<AppStatus> authStatusNotifier}) {
       // Let the initial navigation attempt proceed. The refreshListenable
       // will trigger a redirect check again once the status is known.
       if (appStatus == AppStatus.initial) {
-        print('  Redirect Decision: AppStatus is INITIAL. Allowing navigation.');
+        print(
+          '  Redirect Decision: AppStatus is INITIAL. Allowing navigation.',
+        );
         return null; // Do not redirect during initial phase
       }
 
@@ -261,6 +263,17 @@ GoRouter createRouter({required ValueNotifier<AppStatus> authStatusNotifier}) {
                       return HeadlineDetailsPage(headlineId: id);
                     },
                   ),
+                  // Sub-route for notifications (placeholder) - MOVED HERE
+                  GoRoute(
+                    path: Routes.notifications, // Relative path 'notifications'
+                    name: Routes.notificationsName,
+                    builder: (context, state) {
+                      // TODO(fulleni): Replace with actual NotificationsPage
+                      return const Placeholder(
+                        child: Center(child: Text('NOTIFICATIONS PAGE')),
+                      );
+                    },
+                  ),
                 ],
               ),
             ],
@@ -294,17 +307,6 @@ GoRouter createRouter({required ValueNotifier<AppStatus> authStatusNotifier}) {
                       );
                     },
                   ),
-                  // Sub-route for notifications (placeholder)
-                  GoRoute(
-                    path: Routes.notifications, // Relative path 'notifications'
-                    name: Routes.notificationsName,
-                    builder: (context, state) {
-                      // TODO(fulleni): Replace with actual NotificationsPage
-                      return const Placeholder(
-                        child: Center(child: Text('NOTIFICATIONS PAGE')),
-                      );
-                    },
-                  ),
                 ],
               ),
             ],