Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change navbar links to icons #3328

Closed
grvsachdeva opened this issue Sep 14, 2018 · 16 comments
Closed

Change navbar links to icons #3328

grvsachdeva opened this issue Sep 14, 2018 · 16 comments

Comments

@grvsachdeva
Copy link
Member

grvsachdeva commented Sep 14, 2018

We are preparing to participate in Google Code-in, and have reserved this issue for participants in GCI - but we'd love to have your help with another one! Please check out https://code.publiclab.org to see more.

To make the Navbar more compact replace the following navbar links:

  1. profile link with the icon of the person
  2. Dashboard link with the house icon
  3. We want to hear about this point from you. Suggest which other navbar links can be changed 🤔

Hint: You need to make changes in this https://github.com/publiclab/plots2/blob/master/app/views/layouts/_header.html.erb file

Mention @publiclab/community-reps after solving as getting their consent is essential for choosing icon and for finalizing the view too. Feel free to ask for help, if required. Thanks!

@SidharthBansal
Copy link
Member

Hi @gauravano can you please break this into 2 different issues. So that two people can work on it.
Thanks

@grvsachdeva
Copy link
Member Author

Hi @SidharthBansal, I think it's fine as it is, as the issue just need replacing links with icons at some places, its already an easy task 😅 . What do you say?

@SidharthBansal
Copy link
Member

Ok

@SidharthBansal
Copy link
Member

Published on GCI dashboard

@SidharthBansal
Copy link
Member

An icon in addition to text will be awesome. What do you say?

@jywarren
Copy link
Member

Yes and we can make this a multi-use issue to go to https://fontawesome.com/v4.7.0/ and find an icon, and paste it in here with a link and a screenshot to suggest one for each item. +1!

@jywarren
Copy link
Member

jywarren commented Oct 22, 2018 via email

@jonxuxu
Copy link
Member

jonxuxu commented Oct 29, 2018

Hi, I'm working on solving this issue for codein!

How is this for the user icon:
image
And for the dashboard icon:
image

Respective links:
https://fontawesome.com/v4.7.0/icon/user
https://fontawesome.com/v4.7.0/icon/home

I think I've made the neccessary changes in the file but can't view it as I'm having trouble running Phusion Passenger. I've followed the standard installation for plots2 until step 9, when I get the following error:
https://pastebin.com/AaeeizLL

@jywarren
Copy link
Member

jywarren commented Oct 30, 2018 via email

@SidharthBansal
Copy link
Member

@JonathanXu1 have you worked on this? Where is the pr?

@jonxuxu
Copy link
Member

jonxuxu commented Dec 9, 2018

Yeah I worked on it a long time ago but the task expired before I could claim it

@SidharthBansal
Copy link
Member

SidharthBansal commented Dec 9, 2018 via email

@jonxuxu
Copy link
Member

jonxuxu commented Dec 9, 2018

By the time @jywarren commented , my task had expired, but I found the icons and was adding them to the codebase. Should I still make a PR if the changes have already been solved by #4199 ?

@SidharthBansal
Copy link
Member

I will give you reward for your work.

@SidharthBansal
Copy link
Member

https://codein.withgoogle.com/dashboard/tasks/5446379464818688/
please claim it, I will approve it

@jonxuxu
Copy link
Member

jonxuxu commented Dec 9, 2018

Thank you :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants