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

Use passive event listeners instead of preventDefault #1146

Merged
merged 5 commits into from
Jan 25, 2023

Conversation

danielsippel
Copy link
Contributor

@danielsippel danielsippel commented Jan 18, 2023

fixes #1019

  • popups not opening by click event on mobile browsers if mapbox-draw activated
  • chrome mobile console error "Unable to preventDefault inside passive event listener invocation."

More information about the issue & solution:

@danielsippel
Copy link
Contributor Author

danielsippel commented Jan 25, 2023

Done.

As mentioned, now my fork is not working as a dependency any longer.. 😅
So I would be glad if you could merge the PR soon..

Going to use a branch in my fork next time I supply a bugfix :-)

@stepankuzmin stepankuzmin changed the title fixes #1019 Use passive event listeners instead of preventDefault Jan 25, 2023
@stepankuzmin stepankuzmin merged commit 84e35b7 into mapbox:main Jan 25, 2023
@stepankuzmin
Copy link
Contributor

Excellent, thanks for the fix, @danielsippel! You can use this repo directly until we release a new version.

stepankuzmin added a commit that referenced this pull request Feb 14, 2023
stepankuzmin added a commit that referenced this pull request Feb 14, 2023
danielsippel added a commit to danielsippel/mapbox-gl-draw that referenced this pull request Apr 17, 2024
danielsippel added a commit to danielsippel/mapbox-gl-draw that referenced this pull request Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to preventDefault inside passive event listener invocation.
2 participants