Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds dynamic management of TapRegion on navigation #155297

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

Mairramer
Copy link
Contributor

Fixes #153093

This PR implements a solution to dynamically manage TapRegion when navigating between pages in the app. By using a custom TapRegionNavigatorObserver, TapRegion instances are automatically registered and unregistered during push and pop route events. This ensures that onTapOutside callbacks are triggered only for the active regions on the current page, preventing false positives and improving the consistency of touch region behavior across the app.

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@github-actions github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
framework flutter/packages/flutter repository. See also f: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TapRegion will fire callback on other navigation's stack
2 participants