Static html table menu

[rarrunategu1]

Fixes #192 (<=== Add issue number here)

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

• PR is descriptively titled 📑 and links the original issue above 🔗
• tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
• code is in uniquely-named feature branch and has no merge conflicts 📁
• screenshots/GIFs are attached 📎 in case of UI updation
• ask @publiclab/reviewers for help, in a comment below

Still a working progress. I have a few more things to add so it's not quite ready for review but I'm open to any suggestions based on what may be needed. I do have a png file that is not being used but that's because I couldn't find the original for better quality. I may delete it later.

[welcome]

Thanks for opening this pull request! This space is protected by our Code of Conduct - and we're here to help.
Dangerbot will test out your code and reply in a bit with some pointers and requests.
There may be some errors, but don't worry! We'll work through them with you! 👍🎉😄
It would be great if you can tell us your Twitter handle so we can thank you properly?

[jywarren]

Oooooohhhhh! @ananyaarun @sagarpreet-chadha @stefannibrasil this is pretty awesome, just static HTML to build on top of -- what do you think?

[stefannibrasil]

that looks awesome!
We can potentially use it on the new front-page, it would fit really well 💅

[ananyaarun]

@rarrunategu1 this is great !!

[rarrunategu1]

Thank you. Today I'll be putting the finishing touches on it. I planned on it yesterday but I wasn't feeling well. If you have any suggestions, please feel free to share them. :)

[ananyaarun]

@rarrunategu1 is there anything I can contribute to or help with this PR ?
Do let me know :)

[rarrunategu1]

@jywarren @sagarpreet-chadha @stefannibrasil @ananyaarun I've added the collapsible rows and basically just have a template ready to push. Please take a look at it and let me know if there's anything I may have missed that you feel should be included.

[jywarren]

Thanks! Ananya, can you give it a try? If this looks good we can merge and address any other issues in a follow-up PR!

…

On Tue, Jun 4, 2019, 12:12 PM rarrunategu1 ***@***.***> wrote: @jywarren <https://github.com/jywarren> @sagarpreet-chadha <https://github.com/sagarpreet-chadha> @stefannibrasil <https://github.com/stefannibrasil> @ananyaarun <https://github.com/ananyaarun> I've added the collapsible rows and basically just have a template ready to push. Please take a look at it and let me know if there's anything I may have missed that you feel should be included. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#193?email_source=notifications&email_token=AAAF6J2YTLHTLMLJZTOSGJ3PY2H6PA5CNFSM4HQWUJ4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW5CSNY#issuecomment-498739511>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAF6J5CV2WQPUPWQ6B6I2TPY2H6PANCNFSM4HQWUJ4A> .

[sagarpreet-chadha]

I love this 😍 !

@rarrunategu1 ...can we use hosted URL of the image in the code instead of adding the image in the library ? You can remove the image from this PR .

Some follow up issues we can discuss :

• Clicking Add a Report button should open a modal which has all the fields that the POST request of the API wants . (Will be different for different API's) .

• Google maps also has a similar feature to add report --- we can use UI similar to that (especially in mobile view) .

• Writing AJAX call to POST report as a common function which can be used for all API's .
  One function where we pass different parameters including the URL of API , and it sends POST request to that API .

• Integrating to projects --- mapknitter and plots2 :)

Thank you !

[ananyaarun]

I tried this out locally. It looks great!!!
Here is a working gif

[jywarren]

Ok! Want to offer an approving review and we can go ahead and merge it then? Thanks!!!

…

On Thu, Jun 6, 2019, 4:51 AM Ananya Arun ***@***.***> wrote: I tried this out locally. It looks great!!! Here is a working gif [image: static] <https://user-images.githubusercontent.com/32260628/59030407-8af3c880-887e-11e9-8aad-e9c7012ea793.gif> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#193?email_source=notifications&email_token=AAAF6J5CETNOTZWEPTSDPCDPZD2YXA5CNFSM4HQWUJ4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXCTCIY#issuecomment-499462435>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAF6JZFVIN6OFZMNX53SEDPZD2YXANCNFSM4HQWUJ4A> .

[ananyaarun]

This looks good to merge once the image is removed from templates and replaced with the hosted URL instead as @sagarpreet-chadha suggested.
Thanks !!

[rarrunategu1]

Hello @sagarpreet-chadha, above you suggested that we use hosted URL of the image in the code instead of adding the image in the library ? Then I see an approving review. I'm still trying to understand what all of this means. Were the changes made or should I remove the image at the top to be able to merge this?

[sagarpreet-chadha]

Hey!
No it seems the changes are not made . You can update the PR and then we can merge this :)
Thanks

[rarrunategu1]

I've made the changes @sagarpreet-chadha. I appreciate everyone's feedback. It's now ready to merge @jywarren :)

[sagarpreet-chadha]

Awesome !!!

[welcome]

Congrats on merging your first pull request! 🙌🎉⚡️
Your code will likely be published to https://publiclab.github.io/leaflet-environmental-layers/example/ in the next few days.
Do join our weekly check-in to share your this week goal and the awesome work you did 😃. Please find the link pinned in the issue section 📝
Now that you've completed this, you can help someone else take their first step!
Reach out to someone else working on theirs on Public Lab's code welcome page. Thanks!

Help others take their first step

Now that you've merged your first pull request, you're the perfect person to help someone else out with this challenging first step. 🙌

https://code.publiclab.org

Try looking at this list of `first-timers-only` issues, and see if someone else is waiting for feedback, or even stuck! 😕

People often get stuck at the same steps, so you might be able to help someone get unstuck, or help lead them to some documentation that'd help. Reach out and be encouraging and friendly! 😄 🎉

Read about how to help support another newcomer here, or find other ways to offer mutual support here.

[jywarren]

Awesome work, folks!

…

On Tue, Jun 11, 2019 at 1:22 PM welcome[bot] ***@***.***> wrote: Congrats on merging your first pull request! 🙌🎉⚡️ Your code will likely be published to https://publiclab.github.io/leaflet-environmental-layers/example/ in the next few days. Do join our weekly check-in to share your this week goal and the awesome work you did 😃. Please find the link *pinned in the issue section* 📝 Now that you've completed this, you can help someone else take their first step! Reach out to someone else working on theirs on Public Lab's code welcome page <https://code.publiclab.org#r=all>. Thanks! *Help others take their first step* Now that you've merged your first pull request, you're the perfect person to help someone else out with this challenging first step. 🙌 https://code.publiclab.org <https://code.publiclab.org#r=all> Try looking at this list of `first-timers-only` issues, and see if someone else is waiting for feedback, or even stuck! 😕 People often get stuck at the same steps, so you might be able to help someone get unstuck, or help lead them to some documentation that'd help. Reach out and be encouraging and friendly! 😄 🎉 Read about how to help support another newcomer here <publiclab/plots2#3458>, or find other ways to offer mutual support here <https://github.com/publiclab/plots2/labels/support>. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#193?email_source=notifications&email_token=AAAF6J4NEUMATBHGHC34B3LPZ7NLTA5CNFSM4HQWUJ4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXN4HVY#issuecomment-500941783>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAF6J2PGDFUPVABNJ2C57DPZ7NLTANCNFSM4HQWUJ4A> .