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

Team 4188 logo #136

Closed
wants to merge 3 commits into from
Closed

Team 4188 logo #136

wants to merge 3 commits into from

Conversation

kingbrandon14
Copy link

No description provided.

ErikBoesen
ErikBoesen requested changes Jun 10, 2018
View reviewed changes
Copy link
Member

@ErikBoesen ErikBoesen left a comment

Choose a reason for hiding this comment

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

A couple things:

  • Your logo should have a transparent background.
  • Your logo shouldn't include words or the team name. They won't be readable on the map and they just make things harder on the eyes.

@ErikBoesen
Copy link
Member

Given that no improvements have been made to this PR, I'm going to close it.

Also, an alternate logo was submitted in #136, and we may be moving away from storing our own logos altogether with #140.

@ErikBoesen ErikBoesen closed this Jul 3, 2018
@ErikBoesen ErikBoesen added the logo Adding a team logo label Jul 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ErikBoesen ErikBoesen ErikBoesen requested changes

Assignees
No one assigned
Labels
logo Adding a team logo
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kingbrandon14 @ErikBoesen