Improve the app header - Githubissues

tinkerhub / TinkerHub-Learning-Paths

TinkerHub Learning Paths repo contains curated learning paths for learning technologies

https://paths.tinkerhub.org

MIT License

61 stars 63 forks source link

Improve the app header #40

Closed gautamkrishnar closed 3 years ago

gautamkrishnar commented 3 years ago

Following are the list of suggestions to improve the crrent header of https://tinkerhub.org/learn

Current behaviour

Desktop

Mobile

Suggestions

Make the search bar visible in all screen-sizes including mobile
The 'add new path' link is very large we should use a '+' icon instead of it

Source: @LunaticHacker's coment on https://github.com/tinkerhub-org/TinkerHub-Learning-Paths/pull/38#pullrequestreview-656303407

ToDo

[ ] Design a wireframe - Add as a comment to this issue
[ ] Implement the design - Open a PR once the design is approved

adarshajit commented 3 years ago

Hello there..... I'm attaching the link to the Figma file herewith. Just an attempt. Do let me know what you think! Cheers. https://www.figma.com/file/XsN5TxDG4gDIbFMm6bwYc0/TinkerHub-Navbar?node-id=0%3A1

gautamkrishnar commented 3 years ago

Looks great @adarshajit Great job, Can you please make the search box square instead of rounded borders to match the existing UI.

Screenshot 2021-05-12 at 12 04 02 PM

adarshajit commented 3 years ago

Hey.....As per your feedback, I changed it. Take a look!

gautamkrishnar commented 3 years ago

Looks great. Attaching it here.

Screenshot 2021-05-12 at 12 40 36 PM

LunaticHacker commented 3 years ago

@adarshajit are you working on the implementation? if not, i can work on it.

gautamkrishnar commented 3 years ago

@LunaticHacker you can work on it, @adarshajit was just designing the wireframe.

adarshajit commented 3 years ago

Actually, I was trying to implement it. I was busy with other stuff. Is there a deadline lol.

gautamkrishnar commented 3 years ago

@adarshajit nope, but it would be better if we can get it done earlier. Since you are busy lets have @LunaticHacker work on it. I said you are not working on it because i didnt see you forking the repository.