leaflet-environmental-layers icon indicating copy to clipboard operation
leaflet-environmental-layers copied to clipboard

Open control on start and provide popups

Open kevinzluo opened this issue 7 years ago • 7 comments

Hello. I've been looking at the comments on this thread and found some features that I wanted to add.

This PR will open the layer control on load provided no layers have already been loaded and also opens two popups explaining how to select layers.

fixlcontrol

kevinzluo avatar Dec 06 '18 03:12 kevinzluo

@kevinzluo ...so coool 🎉 !

I see there is a gif added in the PR , can you upload that gif somewhere and give the link to the url and remove the gif from the PR . Thanks !

sagarpreet-chadha avatar Dec 06 '18 07:12 sagarpreet-chadha

Sure thing!

kevinzluo avatar Dec 06 '18 12:12 kevinzluo

I have switched it now 😄 .

kevinzluo avatar Dec 07 '18 04:12 kevinzluo

@kevinzluo please resolve conflicts so that @sagarpreet-chadha can give his final review. Thanks!

grvsachdeva avatar Dec 22 '18 10:12 grvsachdeva

There will be merge conflicts after #104 is merged. Please merge that one first and then I will fix the conflicts here.

kevinzluo avatar Dec 22 '18 16:12 kevinzluo

Hi! This is really awesome!

Is this something we could integrate into the main codebase, do you think, rather than in an independent index.js file?

jywarren avatar Feb 20 '19 00:02 jywarren

I'm sorry it took me so long to respond!!!

jywarren avatar Feb 20 '19 00:02 jywarren