Integrating Leaflet-Environmental-Library .

[sagarpreet-chadha]

Continued from publiclab/leaflet-environmental-layers#18

Link to npm : https://www.npmjs.com/package/leaflet-environmental-layers

Prototype :

[sagarpreet-chadha]

Hi @jywarren ...I guess we have to add the library to bower too .

[jywarren]

I think that's right. Give it a try? 👍

[jywarren]

Looking good on production!

A couple small issues:

1. looks like non-latin characters are converted to ??? maybe?
2. looks like the click-through to get to the actual map is not working?

Looks amazing though!!!

[jywarren]

Let's also add a few follow-ups!

• let's make the other layers visible in the layers menu, so you can turn them on
• let's make usernames appear on the popups?
• let's make the current view on that map (after zooming and panning) embeddable on other pages with a </> button!

[sagarpreet-chadha]

Thanks 😄 .
Working on follow ups !!!

[sagarpreet-chadha]

In the API , the location in non-latin characters are converted to ??????? :

Should i open a new issue on plots2 ?

[jywarren]

Ah, yes i think we should! But is this the MapKnitter api, not plots2? sorry, we're doing so many things at once!

[jywarren]

Just FYI i published MapKnitter again yesterday!

[jywarren]

And just adding one more -- we should either:

1. display only maps that have images (this would be an API change or parameter?)
2. display how many images maps have, in their info bubble

[grvsachdeva]

Hi @jywarren @sagarpreet-chadha, you want to update the checklist so that new issues can be formed from here? Thanks!

[SidharthBansal]

Part of #300