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

Update to hi-res logo & favicon as well as optimize all image files #5748

Merged
merged 1 commit into from Mar 7, 2019

Conversation

@kyleknighted
Copy link
Contributor

commented Mar 6, 2019

Description

  • The logo was replaced with an SVG version using latest logo provided by marketing
  • The favicon was updated to use latest logo provided by marketing
  • All images where ran through the image optimization tool ImageOptim
  • Closes #5747

Motivation and Context

The current logo and favicon are very blurry on retina and 4k+ resolution

Screenshots (if appropriate):

Current

image

This PR

image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@kyleknighted kyleknighted self-assigned this Mar 6, 2019

@kyleknighted kyleknighted requested a review from dennisoelkers Mar 6, 2019

@kyleknighted kyleknighted removed their assignment Mar 6, 2019

@kyleknighted kyleknighted requested review from edmundoa and kmerz Mar 6, 2019

@kmerz
kmerz approved these changes Mar 7, 2019
Copy link
Member

left a comment

LG2M 👍

@kmerz kmerz merged commit d995689 into master Mar 7, 2019

4 checks passed

ci-web-linter Jenkins build graylog-pr-linter-check 3431 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
graylog-project/pr Jenkins build graylog-project-pr-snapshot 3274 has succeeded
Details
license/cla Contributor License Agreement is signed.
Details

@kmerz kmerz deleted the issue-5747 branch Mar 7, 2019

kyleknighted added a commit that referenced this pull request Mar 9, 2019
Remove commented code
Previous PR #5748 for this issue had commented code.
This PR removes commented code only.
@kyleknighted kyleknighted referenced this pull request Mar 9, 2019
1 of 9 tasks complete
edmundoa added a commit that referenced this pull request Mar 14, 2019
Remove commented code (#5756)
Previous PR #5748 for this issue had commented code.
This PR removes commented code only.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.