-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Tooltip: make Map.openTooltip to consider interactive
option
#7531
Conversation
41099bd
to
99c5303
Compare
…Layer.openTooltip)
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.
Looks good.
Only what we can think of is to rename leaflet-clickable
to leaflet-interactive
because leaflet-clickable
is only used for tooltips. But then also CSS needs to be changed: .leaflet-tooltip.leaflet-clickable{
And I would expect that I can use layer.getTooltip().on('click',(e)=>console.log(e))
but it is not working. But that is a other "bug" and not part of this PR
Good point, to be done in separate PR (after this merged) |
May be because of 'preclick', see #7532 |
@mourner your part to approve and merge 😉 |
(like
Layer.openTooltip
)Fix #5974.
This change is