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

z-index #36

Closed
reyemtm opened this issue Nov 10, 2015 · 5 comments · Fixed by #78
Closed

z-index #36

reyemtm opened this issue Nov 10, 2015 · 5 comments · Fixed by #78

Comments

@reyemtm
Copy link

reyemtm commented Nov 10, 2015

I am needing to set the close button's z-index in leaflet-beta-2 to 800 to show up. Everything else seems to work fine.

@zedd45
Copy link
Collaborator

zedd45 commented Aug 2, 2018

Looks like this issue may be related to #51. I think we can make an adjustment for this. I am not sure what changed, but this is a fairly easy fix, I believe.

@zedd45
Copy link
Collaborator

zedd45 commented Apr 19, 2019

@Turbo87 should we bump the z-index value, or is it preferable to let users specify a higher z-index in their own CSS file for the selector: .leaflet-sidebar .close?

@themacboy
Copy link

same problem here. I have solved the problem with:

.close {
z-index: 800 !important;
}

@serl
Copy link

serl commented Jan 15, 2021

It looks like the z-index for .leaflet-control was changed from 7 to 800 since v1.0.0: Leaflet/Leaflet@64f7ef5
Apart from that, this library works perfectly with the last version of Leaflet (v1.7.1) 🥰

@zedd45
Copy link
Collaborator

zedd45 commented Jan 19, 2021

Good call @serl ; I'll make the update. Looks like this one fell off the radar for a while.

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 a pull request may close this issue.

4 participants