Skip to content

Use passive event listeners instead of preventDefault#1146

Merged
stepankuzmin merged 5 commits intomapbox:mainfrom
danielsippel:main
Jan 25, 2023
Merged

Use passive event listeners instead of preventDefault#1146
stepankuzmin merged 5 commits intomapbox:mainfrom
danielsippel:main

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