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

Changed Logo #32

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

Changed Logo #32

wants to merge 2 commits into from

Conversation

Notenlish
Copy link

@Notenlish Notenlish commented Apr 7, 2025

Description

Changed the logo. Also removed the height="300" attribute on the readme file, as it messed with the aspect ratio.

Here's the new logo:
logo

There aren't any issues on the repo relating to changing the logo, but I saw your twitter post and thought I'd help out.

How Has This Been Tested?

Tested by viewing it on Brave, on both desktop and mobile(emulated via devtools)

Checklist:

What I'm doing is not really a code change.

Notes

I can also send favicon files in case you want to use it if you create a docs website for MCP Tools. I can send the files as an SVG or I can share my Figma file.

Here are some screenshots of my "design process" ig, lol. I can change the logo to any one you choose below, if you want to use those instead.

image
image

@f
Copy link
Owner

f commented Apr 8, 2025

Oh these are beautitful! Someone created a logo already and I've updated but I'll keep this PR open as well to decide the exact one.

I love the "cube" structure, like it's a box that keeps some tools in it. Can you improve the current one?

@Notenlish
Copy link
Author

Oh these are beautitful! Someone created a logo already and I've updated but I'll keep this PR open as well to decide the exact one.

I love the "cube" structure, like it's a box that keeps some tools in it. Can you improve the current one?

Would any of these work?
image
image

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