Skip to content

Commit d773ebe

Browse files
committed
fix(headlines-search): check context mount and improve navigation handling
- Add context.mounted check before navigation to prevent errors - Convert onHeadlineTap function to async - Use context.pushNamed instead of context.goNamed for better navigation control - Improve code comments for clarity
1 parent dda8e8c commit d773ebe

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

lib/headlines-search/view/headlines_search_page.dart

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -310,11 +310,15 @@ class _HeadlinesSearchViewState extends State<_HeadlinesSearchView> {
310310
.feedPreferences
311311
.headlineImageStyle;
312312
Widget tile;
313-
void onHeadlineTap() {
314-
context
313+
Future<void> onHeadlineTap() async {
314+
await context
315315
.read<InterstitialAdManager>()
316316
.onPotentialAdTrigger();
317-
context.goNamed(
317+
318+
// Check if the widget is still in the tree before navigating.
319+
if (!context.mounted) return;
320+
321+
await context.pushNamed(
318322
Routes.searchArticleDetailsName,
319323
pathParameters: {'id': feedItem.id},
320324
extra: feedItem,

0 commit comments

Comments
 (0)