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.

Sign up for GitHub

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

Jump to bottom

Member list reduce vetical spacing of logo. Seems 1/3 too tall #30

Closed
mikemccamon opened this issue Jul 13, 2019 · 1 comment
Closed

Member list reduce vetical spacing of logo. Seems 1/3 too tall #30

mikemccamon opened this issue Jul 13, 2019 · 1 comment

Comments

@mikemccamon
Copy link

No description provided.

@scorchingoyster
Copy link
Owner

scorchingoyster commented Jul 13, 2019 •
edited
Loading

I changed it from 10px to 6px, but I wanted to let you know that I originally gave it that much space to prevent edge cases where the logos would be too close with each other vertically, like in the case of UPS logo siding with another tall logo—worth noting that I already I added transparent area around the image to make it take up less space, too.

image

I just decided to increase the transparent area a bit more on taller logos images, so let me know if you'd like to see this tweaked further.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@scorchingoyster @mikemccamon