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

ESRI base maps hides overlays #795

Closed
zstadler opened this issue Sep 4, 2018 · 3 comments
Closed

ESRI base maps hides overlays #795

zstadler opened this issue Sep 4, 2018 · 3 comments
Assignees
Labels
bug Medium Should be added/fixed in the future
Milestone

Comments

@zstadler
Copy link
Member

zstadler commented Sep 4, 2018

Bug

What I expect to happen

Map overlays should be visible on top of ESRI base maps

What really happened

ESRI base maps are covering map overlays

What I did that caused the issue - step by step

  1. Open the https://israelhiking.osm.org.il/share/JoKWORRFxK
  2. You may be able to see the overlay downloaded first
    2018-09-04 10_15_49-israel hiking map
    and then the ESRI base map covers it:
    2018-09-04 10_15_51-israel hiking map

This also happens in private/anonymous mode

My environment

  • Operating System and version: Win 10 home
  • Web Browser and version:
    • Firefox 56.0.2 (64-bit)
    • Microsoft Edge 42.17134.1.0
    • Chrome 68.0.3440.106 (Official Build) (64-bit)
@HarelM
Copy link
Member

HarelM commented Sep 4, 2018

Probably needs the following implementation of panes, I truly wonder why it works as expected for other layers... :-)
https://leafletjs.com/examples/map-panes/
http://esri.github.io/esri-leaflet/examples/layer-ordering.html

@HarelM HarelM self-assigned this Sep 4, 2018
@HarelM HarelM added bug Medium Should be added/fixed in the future labels Sep 4, 2018
@HarelM HarelM added this to the Next Release milestone Sep 4, 2018
@HarelM
Copy link
Member

HarelM commented Sep 13, 2018

Also depending on #669 - if we migrate to open layers this should be implemented differently.

@HarelM
Copy link
Member

HarelM commented Dec 7, 2018

I don't think this is still relevant in beta code. relevant issue is now #860.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Medium Should be added/fixed in the future
Projects
None yet
Development

No branches or pull requests

2 participants