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

A modest attempt to improve the logo aesthetics #2952

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

huettenhain
Copy link

@huettenhain huettenhain commented Nov 8, 2024

To be perfectly honest, I am not a user of k9s. I was working on a project with a colleague and they had it open. I commented on how I don't like some of the decisions that were made for the standard Graffiti font and how I thought the K looked kinda weird. Said colleague suggested I should open a PR if it bugs me so much, and hey, I thought, why not. Feel free to reject this if you disagree, but I think the logo looks a lot smoother with a few simple changes =).

Edit: Here's a quick preview of the old one:

 ____  __.________       
|    |/ _/   __   \______
|      < \____    /  ___/
|    |  \   /    /\___ \ 
|____|__ \ /____//____  >
        \/            \/ 

And this is my suggestion:

 ____  __ ________       
|    |/  /   __   \______
|       /\____    /  ___/
|    \   \  /    /\___  \
|____|\__ \/____//____  /
         \/           \/ 

@derailed
Copy link
Owner

derailed commented Nov 9, 2024

@huettenhain Thank you for volunteering this update!
The new logo looks nice. I'll let folks vote this up and recommend their drathers.
For me, I think the K pops a little more in the original design...

@huettenhain
Copy link
Author

Looks like I didn't properly adjust the tests, though. I'll admit I did this blind, but I'll setup a go environment later and fix that.

@huettenhain
Copy link
Author

Ok, tests should go green now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants