search

thecointech / thecoin

Mono-repo for TheCoin project
https://thecoin.io
GNU Affero General Public License v3.0
4 stars 1 forks source link

Fix/landing header #341

Closed FrozenKiwi closed 2 years ago

FrozenKiwi commented 2 years ago

Tweaks landing page header to fix overflow.

Removed "Home", merged "FAQ" & "Article" into "Help", tweaked the padding/margins of everything.

https://user-images.githubusercontent.com/4876160/143656266-59092233-9111-4dc6-8ef1-761c0cca2768.mp4

MarieLoupG commented 2 years ago

@FrozenKiwi I thought you wanted to get rid of 'se connecter' link in the header that would leave more room for the links. I am worried about the help one a bit too small and not not a lot of space between the others.

FrozenKiwi commented 2 years ago

No, I thought we needed at least 1 link to the app in header. "Se Connecter" is probably smaller than "Create Account" so thought that one made sense. The crowding is pretty small though, I'll test it on a device when the new tablet turns up

FrozenKiwi commented 2 years ago

I didn't want to have "Log In" popping in and out as we shrink/grow. Consistency > Pretty, if that choice has to be made?

FrozenKiwi commented 2 years ago

Merging this to keep our list of PR's down, will email you to verify whether you think 'se connecter' should change