fix: These changes should ensure correct navigation from the "Similar Headlines" section and prevent the erroneous display of the "headline removed from saved" snackbar.

Author: fulleni

lib/headline-details/view/headline_details_page.dart

@@ -4,6 +4,7 @@
 import 'package:flutter/foundation.dart' show kIsWeb; // Import kIsWeb
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
+import 'package:go_router/go_router.dart'; // Import GoRouter
 import 'package:ht_main/account/bloc/account_bloc.dart'; // Import AccountBloc
 import 'package:ht_main/headline-details/bloc/headline_details_bloc.dart'; // Import BLoC
 import 'package:ht_main/headline-details/bloc/similar_headlines_bloc.dart'; // Import SimilarHeadlinesBloc
@@ -73,10 +74,6 @@ class _HeadlineDetailsPageState extends State<HeadlineDetailsPage> {
             listenWhen: (previous, current) {
               final detailsState = context.read<HeadlineDetailsBloc>().state;
               if (detailsState is HeadlineDetailsLoaded) {
-                if (current.status == AccountStatus.failure &&
-                    previous.status != AccountStatus.failure) {
-                  return true;
-                }
                 final currentHeadlineId = detailsState.headline.id;
                 final wasPreviouslySaved =
                     previous.preferences?.savedHeadlines.any(
@@ -88,9 +85,21 @@ class _HeadlineDetailsPageState extends State<HeadlineDetailsPage> {
                           (h) => h.id == currentHeadlineId,
                         ) ??
                         false;
-                return (wasPreviouslySaved != isCurrentlySaved) ||
-                    (current.status == AccountStatus.success &&
-                        previous.status != AccountStatus.success);
+
+                // Condition 1: Actual change in saved status for this headline
+                if (wasPreviouslySaved != isCurrentlySaved) {
+                  // Only trigger if the status is success (to show confirmation)
+                  // or failure (to show error). Avoid triggering if status is just loading.
+                  return current.status == AccountStatus.success ||
+                         current.status == AccountStatus.failure;
+                }
+
+                // Condition 2: A specific save/unsave operation just failed
+                // This triggers if an operation was attempted (loading) and then failed.
+                if (current.status == AccountStatus.failure &&
+                    previous.status == AccountStatus.loading) {
+                  return true;
+                }
               }
               return false;
             },
@@ -487,7 +496,15 @@ class _HeadlineDetailsPageState extends State<HeadlineDetailsPage> {
                     ),
                     child: HeadlineItemWidget(
                       headline: similarHeadline,
-                      targetRouteName: Routes.articleDetailsName,
+                      // Use the onTap callback for navigation
+                      onTap: (tappedHeadline) {
+                        context.pushNamed(
+                          Routes.articleDetailsName,
+                          pathParameters: {'id': tappedHeadline.id},
+                          extra: tappedHeadline,
+                        );
+                      },
+                      // targetRouteName: Routes.articleDetailsName, // No longer needed here
                     ),
                   );
                 },

lib/headlines-feed/widgets/headline_item_widget.dart

@@ -9,16 +9,22 @@ class HeadlineItemWidget extends StatelessWidget {
   /// Creates a [HeadlineItemWidget].
   const HeadlineItemWidget({
     required this.headline,
-    required this.targetRouteName, // Add targetRouteName
+    this.targetRouteName, // Make optional
+    this.onTap, // Add optional onTap callback
     this.trailing, // Add optional trailing widget
     super.key,
-  });
+  }) : assert(targetRouteName != null || onTap != null, 'Either targetRouteName or onTap must be provided');
 
   /// The headline to display.
   final Headline headline;
 
   /// The named route to navigate to when the item is tapped.
-  final String targetRouteName; // Add targetRouteName
+  /// Used if [onTap] is null.
+  final String? targetRouteName;
+
+  /// Optional callback to handle tap events.
+  /// If provided, this is used instead of navigating via [targetRouteName].
+  final void Function(Headline headline)? onTap;
 
   /// An optional widget to display at the end of the row.
   final Widget? trailing;
@@ -47,11 +53,15 @@ class HeadlineItemWidget extends StatelessWidget {
           cardTheme.clipBehavior ?? Clip.antiAlias, // Use theme's clip behavior
       child: InkWell(
         onTap: () {
-          context.goNamed(
-            targetRouteName, // Use the new parameter here
-            pathParameters: {'id': headline.id},
-            extra: headline, // Pass the full headline object
-          );
+          if (onTap != null) {
+            onTap!(headline);
+          } else {
+            context.goNamed(
+              targetRouteName!, // Not null due to assert
+              pathParameters: {'id': headline.id},
+              extra: headline, // Pass the full headline object
+            );
+          }
         },
         child: Padding(
           padding: const EdgeInsets.all(AppSpacing.md), // Internal padding