add relevant L parts - Githubissues

jonatkins / s2-geometry-javascript

Porting Google's S2 Geometry Library to Javascript

31 stars 17 forks source link

Closed coolaj86 closed 8 years ago

coolaj86 commented 8 years ago

Added

Changed

jonatkins commented 8 years ago

Not sure about this. Likely to clash with the real leafletjs library if that's included?

I suggest just removing the dependency entirely

our own DEG_TO_RAD/RAD_TO_DEG, local to this file
just returning a simple Object with lat and lng properties instead of using L.latLng (and no, it's not a typo - L.latLng is a function that constructs the object, L.LatLng is the prototype class)

coolaj86 commented 8 years ago

Hmm... Looking at the code I didn't see L.latLng lowercase in the source, but it seems to work with L.LatLng (with the updates I just made).

Updated to not redefine L.