migrate geolocation-demo.js to osm w openlayers #353

Closed
wants to merge 1 commit into
from

3 participants

@l-hedgehog
Mozilla member

Hi,

Sorry for not actively working on this and bug 767767

This is a first attempt to replace Google Maps API with OpenStreetMap (via OpenLayers API). Leaflet mentioned in Deploying your own Slippy Map seems to be a simpler API but requires another key from CloudMade.

I kind of have the impression that OpenLayers users should host their own copy of API files in the production environment. But for now I just link to http://openlayers.org/api/OpenLayers.js directly.

@rik
rik commented Oct 2, 2012

Sorry for not reviewing earlier. Don't apologize for not working on this :) You've been more than helpful on this!

This looks fine to me for the code.
The maps look awful though :(

I'll bring the licensing issue in the bug. Do not merge in the mean time.

@sgarrity

I see more activity in Bug 767767 - is this pull request out-of-date? Can you either close or update?

@l-hedgehog
Mozilla member

Hi @sgarrity , I'll close this one and open a new pull request with leaflet changes I talked about in 767767c6 soon.

@l-hedgehog l-hedgehog closed this Mar 1, 2013
@l-hedgehog l-hedgehog deleted the l-hedgehog:openlayers branch Mar 1, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment