Skip to content

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

Open
h4ck3rm1k3 wants to merge 1 commit into
ushahidi:developfrom
h4ck3rm1k3:issue_1378_google_osm
Open

now osm displays with google, but not the other way#1389
h4ck3rm1k3 wants to merge 1 commit into
ushahidi:developfrom
h4ck3rm1k3:issue_1378_google_osm

Conversation

@h4ck3rm1k3

Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

2 participants