-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Fire mouseout events #2777
Fire mouseout events #2777
Conversation
Helps hide popups / highlighted areas which can clutter the screen, on a mouse out
Great! Thanks @alcoheca! |
@@ -37,6 +38,11 @@ module.exports = function bindHandlers(map, options) { | |||
el.addEventListener('dblclick', onDblClick, false); | |||
el.addEventListener('contextmenu', onContextMenu, false); | |||
|
|||
function onMouseOut(e) { | |||
map.stop(); |
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.
Let's remove this line. By default, animated zooms/pans should not stop if the cursor leaves the canvas.
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.
sorry - I'll be honest i wasn't 100% about this line..
should I do anything?
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.
see #2789
Helps hide popups / highlighted areas which can clutter the screen, on a mouse out
Allows developers to hide popups / clear highlighted areas which can clutter the screen after the user moves the mouse pointer out of context.