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

now osm displays with google, but not the other way #1389

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

h4ck3rm1k3
Copy link

The showing of osm with google works, but showing of google with osm still needs an include. that needs to be figured out, I am thinking about making an a list of src url that will be included and not just one. But for now this should solve your issue #1378 @brianherbert

@brianherbert
Copy link
Contributor

We should always allow OSM but we don't necessarily need to do it the other way around. If that's the case, then we're always including libraries for baselayers that are more than likely never going to be toggled. Alternatively, we could load those libraries on demand but that's a much larger issue.

I haven't had a chance to review this yet but I will.

@h4ck3rm1k3
Copy link
Author

Ok, I will look into adding osm always next. Please test what I did and let
me know.

On Wed, Jul 2, 2014 at 2:03 PM, Brian Herbert notifications@github.com
wrote:

We should always allow OSM but we don't necessarily need to do it the
other way around. If that's the case, then we're always including libraries
for baselayers that are more than likely never going to be toggled.
Alternatively, we could load those libraries on demand but that's a much
larger issue.

I haven't had a chance to review this yet but I will.

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

James Michael DuPont
Member of Free Libre Open Source Software Kosova http://www.flossk.org
Saving Wikipedia(tm) articles from deletion http://SpeedyDeletion.wikia.com

@brianherbert
Copy link
Contributor

First off, apologies it took a while to get back. I was away for the 4th.

On a default install of Ushahidi, where the default map is OSM (osm_mapnik), the map doesn't load since the google library isn't being included in the header. Since we only want to show OSM as an additional provider when the site is set to a separate provider (Google, Bing, etc), we shouldn't include Google when OSM is set.

screen shot 2014-07-07 at 9 31 45

@h4ck3rm1k3
Copy link
Author

Hope you had a great independance day.

On Mon, Jul 7, 2014 at 8:31 AM, Brian Herbert notifications@github.com
wrote:

the map doesn't load since the google library isn't being included in the
header

yes, I understand. I will turn it on for the other providers.
mike

James Michael DuPont
Member of Free Libre Open Source Software Kosova http://www.flossk.org
Saving Wikipedia(tm) articles from deletion http://SpeedyDeletion.wikia.com

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

Successfully merging this pull request may close these issues.

None yet

2 participants