Plots2: Style "add a location" button to match style guide

[jywarren]

Pages like https://publiclab.org/wiki/london-uk have a prompt to add a location:

However, the style guide shows we should be showing them in a button like this:

The icon should be: <i class="fa fa-map-marker" style="color:#c40;"></i>

And the text should be (on this page) "Add a location" and not "Add your location".

We'd love help fixing this!

The code for it is here:

https://github.com/publiclab/plots2/blob/1cfd91a87c7333718960a20b560857c88920c0d3/app/views/sidebar/_related.html.erb#L48-L54

[jywarren]

This has been marked as a good candidate for becoming a first-timers-only issue like these, meaning that it's simple, self-contained, and with some extra formatting, could be a great entry point for a new contributor. If you're familiar enough with this code, please consider reformatting or reposting it as a first-timers-only issue, and then ping @publiclab/reviewers to get it labelled. Or, if this is not your first time, try to solve it yourself!

---

[kushaggarwal]

Hi @jywarren
As I understood from the above mentioned information about the issue is to change the "add your location" button into the new format as mentioned by you for which the reference code you have given above right

[jywarren]

That's right! Thanks!

On Wed, Nov 27, 2019 at 12:54 AM kush aggarwal [e-mail]
wrote:

Hi @jywarren https://github.com/jywarren
As I understood from the above mentioned information about the issue is to
change the "add your location" button into the new format as mentioned by
you for which the reference code you have given above right

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/plots2/issues/6856?email_source=notifications&email_token=AAAF6J5RR32LR2YKKDQH5X3QVYDPLA5CNFSM4JR3O6C2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFIMISA#issuecomment-558941256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAF6JYW4DSFFEVSYAPRHQDQVYDPLANCNFSM4JR3O6CQ
.

[SidharthBansal]

Hi @kushaggarwal are you working on this?

[kushaggarwal]

Hello @SidharthBansal yes I am

[SidharthBansal]

Go ahead. Thanks

[jywarren]

Just noting this could be related to #6899 and cc'ing @nstjean !

[jywarren]

Hi @kushaggarwal are you still working on this? Need any help? Thanks!

[jywarren]

If you aren't able to continue, maybe we can open this up for someone else? Thanks, either way works!

[pianeta051]

@kushaggarwal are you still working on this....If not please let us know, because I am interested in this.
thanks

[jywarren]

Hi @pianeta051 we'd love your help, it seems like this is open!

[kushaggarwal]

Hello guys,
Sorry for the answering so late
Got stuck with my university exams therefore wasn't able to reply
@pianeta051 please go ahead as I am facing some problem with setting up the project
so, you can go ahead with the issue meanwhile I will try to solve the problems with setup on my end
Thank you

[dmahajan980]

I would like solving this issue as my first contribution to open source. Can I work on this?

[nstjean]

@dmahajan980 I just assigned another issue to you: https://github.com/publiclab/plots2/issues/6516

[jywarren]

If nobody's taken this up still, it's still available! Please leave a comment and open a pull request if you get started, thanks!

[sssash18]

@jywarren I would like to do this.

[SidharthBansal]

Hi @sssash18 only one FTO/ first timers only issue per person is allowed. Please don't do more ftos
We will merge this one. But don't do more ftos.
You can do as many issues as you want which don't have fto tag nor first-timers-only tag
Thanks

[sssash18]

Ok @SidharthBansal ,I will take care next time .Sorry for the misdeed.