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
Fix cursor state on popup.remove() (#12223) #12230
Fix cursor state on popup.remove() (#12223) #12230
Conversation
- add in function to remove 'mapboxgl-track-pointer' from map element class on popup.remove()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please fix the failing unit tests in marker.test.js
?
I saw that. I think a simple conditional should fix it. Testing it now. |
- added conditional to map property canvasContainer in case map is null
@camouflagedName let's avoid the optional chaining operator for now since it's not yet used anywhere else in the codebase and it started being supported across browsers relatively recently, so introducing it might have unexpected consequences. |
- added conditional to check map property canvasContainer exists in case undefined
Launch Checklist
mapbox-gl-js
changelog:<changelog>returns cursor to original state after a popup is removed</changelog>