leaflet-plugin icon indicating copy to clipboard operation
leaflet-plugin copied to clipboard

Verify browser compatability w/ Leaflet Core

Open riordan opened this issue 10 years ago • 3 comments

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

riordan avatar Aug 10 '15 20:08 riordan

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 avatar Aug 11 '15 17:08 louh

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

louh avatar Aug 31 '15 21:08 louh

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 [email protected] 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 https://github.com/pelias/leaflet-geocoder/issues/27#issuecomment-136512049 .

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

riordan avatar Sep 02 '15 18:09 riordan