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

Allow OpenStreetMap as an option *alongside* Google maps in the layer picker #1378

Open
harry-wood opened this issue May 21, 2014 · 8 comments
Labels

Comments

@harry-wood
Copy link

I often see ushahidi instances where the map alternatives in the top right picker are Google Maps Normal/Satellite/Hybrid/Physical (Indeed http://demo.ushahidi.com is set up this way) Then I see others where the admin has chosen to set them up to use OpenStreetMap instead. I'm inferring from this, that ushahidi offers this choice to admins. Google Maps or OpenStreetMap, as if they exclude each other entirely.

I have never seen an ushahidi instance offering BOTH within the layer picker. This seems like a missed opportunity, given that you have gone to the trouble of writing all your map logic using OpenLayers. That's something which is perfectly possible with OpenLayers as shown in this example: http://openlayers.org/dev/examples/osm-google.html

So when an admin user decides that google maps is better (which always makes me sad) could we at least get OpenStreetMap in there as a layer the end user can pick?

I imagine the most important thing to offer admins, is the ability to choose their default basemap. Then another possible option, of marginal importance, would be the ability to switch off certain layer options, so their end users can't use them (not sure why they would really)

@srutto srutto added the Feature label Jun 30, 2014
@brianherbert
Copy link
Contributor

Bountysource

There is now a $5 bounty to solve this issue.

@h4ck3rm1k3
Copy link

I have a first version working
I have been able to add osm to the google layer but not yet google to the osm layer
h4ck3rm1k3@d2552b3
basically I use the field $layer->openlayers only for grouping and
$layer->openlayers_class for the name of the class. Change the code to use class. adding duplicate layers for the alternative views. Let me know if this solves your problem.

@brianherbert
Copy link
Contributor

Thanks for working on this, @h4ck3rm1k3 - This will probably work. If you're ready for a review, you can submit a pull request.

@h4ck3rm1k3
Copy link

Righty Right.

@h4ck3rm1k3
Copy link

#1389

@bodacea
Copy link

bodacea commented Aug 7, 2014

I've added this to the V3 features wishlist at https://phabricator.ushahidi.com/T541; please feel free to continue working on this for V2.

@harry-wood
Copy link
Author

Don't know if this ever got fixed in this repo since 2014, but this blog post seems related: https://www.ushahidi.com/blog/2017/04/06/using-mapbox-at-ushahidi Sounds like the new version of ushahidi (this repo?) will be a lot more OpenStreetMappy.

@rjmackay
Copy link
Contributor

rjmackay commented Apr 6, 2017

@harry-wood the new version of Ushahidi lives over at https://github.com/ushahidi/platform and https://github.com/ushahidi/platform-client

At the moment we only have support for OSM via mapbox baselayers, and no support for Google Maps.

I think we migrated this issue over to the new repo if you wanted to follow it there: ushahidi/platform#581

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants