refactor: moved notifications feed entry point from the account to the feed page

Author: fulleni

lib/account/view/account_page.dart

@@ -61,8 +61,6 @@ class _AccountView extends StatelessWidget {
           const SizedBox(height: 24),
 
           // --- Action Tiles ---
-          _buildNotificationsTile(context),
-          const Divider(),
           _buildSettingsTile(context),
           const Divider(),
           if (isAnonymous)
@@ -171,20 +169,6 @@ class _AccountView extends StatelessWidget {
     );
   }
 
-  /// Builds the ListTile for navigating to Notifications.
-  Widget _buildNotificationsTile(BuildContext context) {
-    final l10n = context.l10n;
-    return ListTile(
-      leading: const Icon(Icons.notifications_outlined),
-      title: Text(l10n.accountNotificationsTile),
-      trailing: const Icon(Icons.chevron_right), // Suggests navigation
-      onTap: () {
-        // Navigate to the new notifications route (placeholder)
-        context.goNamed(Routes.notificationsName);
-      },
-    );
-  }
-
   /// Helper to convert AuthenticationStatus enum to a display string.
   String _authenticationStatusToString(
     BuildContext context,

lib/headlines-feed/view/headlines_feed_page.dart

@@ -1,9 +1,11 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
+import 'package:go_router/go_router.dart';
 import 'package:ht_headlines_repository/ht_headlines_repository.dart';
 import 'package:ht_main/headlines-feed/bloc/headlines_feed_bloc.dart';
 import 'package:ht_main/headlines-feed/widgets/headline_item_widget.dart';
 import 'package:ht_main/l10n/l10n.dart';
+import 'package:ht_main/router/routes.dart';
 import 'package:ht_main/shared/constants/constants.dart';
 import 'package:ht_main/shared/widgets/failure_state_widget.dart';
 import 'package:ht_main/shared/widgets/loading_state_widget.dart';
@@ -73,7 +75,14 @@ class _HeadlinesFeedViewState extends State<_HeadlinesFeedView> {
 
     return Scaffold(
       appBar: AppBar(
-        // Removed leadingWidth and leading Row
+        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),

lib/l10n/arb/app_ar.arb

@@ -279,5 +279,9 @@
     "headlinesSearchActionTooltip": "بحث",
     "@headlinesSearchActionTooltip": {
         "description": "Tooltip text for the search icon button in the search page AppBar"
+    },
+    "notificationsTooltip": "عرض الإشعارات",
+    "@notificationsTooltip": {
+        "description": "Tooltip text for the notifications icon button in the feed page AppBar"
     }
 }

lib/l10n/arb/app_en.arb

@@ -279,5 +279,9 @@
     "headlinesSearchActionTooltip": "Search",
     "@headlinesSearchActionTooltip": {
         "description": "Tooltip text for the search icon button in the search page AppBar"
+    },
+    "notificationsTooltip": "View notifications",
+    "@notificationsTooltip": {
+        "description": "Tooltip text for the notifications icon button in the feed page AppBar"
     }
 }