Skip to content

Commit 9241723

Browse files
committed
refactor(headlines-feed): update navigation flow with ads
- Modify onHeadlineTap to handle ad display and navigation asynchronously - Add check for widget tree membership before navigating - Improve user experience by proceeding with navigation only after ad is closed
1 parent be1ed31 commit 9241723

File tree

1 file changed

+11
-3
lines changed

1 file changed

+11
-3
lines changed

lib/headlines-feed/view/headlines_feed_page.dart

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -244,9 +244,17 @@ class _HeadlinesFeedPageState extends State<HeadlinesFeedPage> {
244244
);
245245
}
246246

247-
void onHeadlineTap(Headline headline) {
248-
context.read<InterstitialAdManager>().onPotentialAdTrigger();
249-
context.goNamed(
247+
Future<void> onHeadlineTap(Headline headline) async {
248+
// Await for the ad to be shown and dismissed.
249+
await context
250+
.read<InterstitialAdManager>()
251+
.onPotentialAdTrigger();
252+
253+
// Check if the widget is still in the tree before navigating.
254+
if (!context.mounted) return;
255+
256+
// Proceed with navigation after the ad is closed.
257+
await context.pushNamed(
250258
Routes.articleDetailsName,
251259
pathParameters: {'id': headline.id},
252260
extra: headline,

0 commit comments

Comments
 (0)