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

Add scalable version of the logo #5709

Closed
wants to merge 1 commit into from
Closed

Add scalable version of the logo #5709

wants to merge 1 commit into from

Conversation

The-Compiler
Copy link

@The-Compiler The-Compiler commented May 14, 2021

I needed a scalable version for a (non-public) presentation, and 128x128 is pretty small.

I couldn't find a source for the logo anywhere, so I decided to quickly redraw it based on the original:

image

and arrived at something which looks pretty close:

aiohttp-svg

I thought I'd add it to the repo in case anyone else needs it too - even though it's not actually used in the docs, next to the .png is the best place to put it, IMHO. If someone had the original logo in a scalable format, that'd be even better, of course.

Redrawn based on the PNG
@The-Compiler The-Compiler requested a review from asvetlov as a code owner May 14, 2021 14:30
@webknjaz
Copy link
Member

Hey Florian, I recently discovered that the SVG logo is actually present in this repo, just isn't used in sphinx for no reason 🤷‍♂️, and it is located poorly — https://github.com/aio-libs/aiohttp/blob/master/docs/aiohttp-plain.svg.
I've filed an issue to make it actually used (#5677) and somebody even attempted to address it (#5678).

That said, I'm going to close this PR. Thanks for caring!

@webknjaz webknjaz closed this May 14, 2021
@The-Compiler
Copy link
Author

Ah, turns out I had searched for "logo" in the repo, but not for "svg"... Even better, thanks! 👍

@The-Compiler The-Compiler deleted the svg-logo branch May 18, 2021 07:47
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