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

Footer Formatting #293

Merged
merged 3 commits into from Jan 27, 2019
Merged

Footer Formatting #293

merged 3 commits into from Jan 27, 2019

Conversation

finnsky
Copy link
Contributor

@finnsky finnsky commented Nov 29, 2018

what

  • Fixed footer layout. It was wrong aligned.

why

@finnsky
Copy link
Contributor Author

finnsky commented Nov 29, 2018

Image from Gyazo

@finnsky
Copy link
Contributor Author

finnsky commented Nov 29, 2018

Also we may add some border top in footer. or small shadow at least? What you think @osterman ?

@osterman osterman changed the title Footer fixes 275 Footer Formatting Nov 29, 2018
@osterman
Copy link
Member

border top in footer. or small shadow at least

Good idea. Can you show a couple examples?

@osterman
Copy link
Member

image

Tags should not be under the categories.

Maybe we should move them somewhere else on the page?

Copy link
Member

@osterman osterman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tags are in a weird place

@osterman
Copy link
Member

What do you think about this?

image
image

@finnsky
Copy link
Contributor Author

finnsky commented Dec 3, 2018

Image from Gyazo

@osterman
Copy link
Member

osterman commented Dec 3, 2018

I don't like mixing the concept of the "badge" and overloading it for "tags". In github, badges are one thing.

Maybe we should borrow the GitHub UI for this? See tags here....

image

@osterman
Copy link
Member

osterman commented Dec 3, 2018

Also, we really need some padding below the title and the content. It's so scrunched up

@osterman
Copy link
Member

@finnsky lets first do this: #306

@osterman osterman added this to In progress in UI/UX via automation Dec 23, 2018
@osterman
Copy link
Member

@finnsky rebase

@finnsky finnsky moved this from In progress to Needs review in UI/UX Jan 26, 2019
@finnsky
Copy link
Contributor Author

finnsky commented Jan 26, 2019

Image from Gyazo

UI/UX automation moved this from Needs review to Reviewer approved Jan 27, 2019
@osterman osterman merged commit 82c6dfb into master Jan 27, 2019
UI/UX automation moved this from Reviewer approved to Done Jan 27, 2019
@osterman osterman deleted the footer-fixes-275 branch January 12, 2020 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
UI/UX
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants