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

IE8 tile loading bug #3332

Closed
Trufi opened this issue Mar 27, 2015 · 6 comments
Closed

IE8 tile loading bug #3332

Trufi opened this issue Mar 27, 2015 · 6 comments
Labels
blocker Critical issue or PR that must be resolved before the next release
Milestone

Comments

@Trufi
Copy link
Contributor

Trufi commented Mar 27, 2015

Can you look at leaflet (master branch) in IE8? Tiles sometimes freeze and stop loading after several drags and zooms. It looks like:

screenshot from 2015-03-27 14 30 50
Windows XP, Service Pack 3, IE8

@perliedman
Copy link
Member

I can reproduce this with latest master on IE8/Win7 (virtual machine from MS in Virtual Box) with the /debug/map/map.html example.

An interesting question is if Leaflet still aims to support IE8, which starts to get really old, and it seems some libraries are dropping support for it.

I'll try to have a look, but given the state of IE8 debugger, I can't promise anything :)

@mourner
Copy link
Member

mourner commented Mar 27, 2015

Yeah, Leaflet still has to support IE8 since A LOT of government/enterprise apps depend on this. Thanks for looking into this.

@perliedman
Copy link
Member

Follow up: I ran the test specs on IE8, and there are quite a few errors. Some of them appear in GridLayer, which I guess might be the issue we're seeing here.

@mourner mourner added this to the 1.0 milestone Mar 27, 2015
@mourner mourner added the blocker Critical issue or PR that must be resolved before the next release label Mar 27, 2015
@perliedman
Copy link
Member

So far, I've found out that the tiles load, but for some reason the leaflet-tile-loaded class is never applied to the element. I'll try to find some more time to dig into why.

As a side note, this issue can be debugged in IE11 if you set the debugger's document mode to IE8, which makes things a lot easier.

@osvodef
Copy link

osvodef commented Apr 1, 2015

I think I've managed to pinpoint the bug, here is a possible solution: #3346

@mourner
Copy link
Member

mourner commented Apr 2, 2015

@RLRR thanks a lot! Merged.

@mourner mourner closed this as completed Apr 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker Critical issue or PR that must be resolved before the next release
Projects
None yet
Development

No branches or pull requests

4 participants