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

Verify browser compatability w/ Leaflet Core #27

Open
13 tasks
riordan opened this issue Aug 10, 2015 · 3 comments
Open
13 tasks

Verify browser compatability w/ Leaflet Core #27

riordan opened this issue Aug 10, 2015 · 3 comments

Comments

@riordan
Copy link
Contributor

riordan commented Aug 10, 2015

Leaflet supports

  • Chrome
  • Firefox
  • Safari 5+
  • Opera 12+
  • IE 7-11

We've got to go back and check all of them.

  • Chrome
  • Firefox
  • Safari
    • Safari 5
    • Safari 6
    • Safari 7
    • Safari 8
  • Internet Explorer
    • IE 7
    • IE 8
    • IE 9
    • IE 10
    • IE 11
@louh
Copy link
Contributor

louh commented Aug 11, 2015

Compatibility notes (ongoing, will evolve as I learn more)

  • Placeholders in input box are not supported on IE 7-9 (http://caniuse.com/#feat=input-placeholder). This degrades gracefully so we don't need to remove it. If we want to support IE 7-9 with placeholders, we can either polyfill it inside the library or note it as a caveat (so that developers who integrate the library can choose their polyfill method of choice or ignore it).
  • The element.querySelector() and element.querySelectorAll() methods do not exist in IE7 (http://caniuse.com/#feat=queryselector). This will cause the library to throw errors, so we must either work around it or polyfill the method.
  • a JSONP response alternative might be required for old IE. (Would CORS support also help?)

@louh
Copy link
Contributor

louh commented Aug 31, 2015

Yo @riordan, IE8/IE9 support is ready for testing. IE7 will have to be not supported due to lack of JSONP primarily.

@riordan
Copy link
Contributor Author

riordan commented Sep 2, 2015

Sweet! Time to fire up the IE VM's. I'll take a look this evening.

On Mon, Aug 31, 2015 at 5:55 PM, Lou Huang notifications@github.com wrote:

Yo @riordan https://github.com/riordan, IE8/IE9 support is ready for
testing. IE7 will have to be not supported due to lack of JSONP primarily.


Reply to this email directly or view it on GitHub
#27 (comment)
.

David Riordan | Product Manager - Search | dave.riordan@mapzen.com |
@riordan https://twitter.com/riordan | gpg 235D9DC95EF6277C
https://keybase.io/riordan
Mapzen | https://mapzen.com | @mapzen https://twitter.com/search

@dianashk dianashk added this to the Leaflet Geocoder milestone Apr 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants