-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 awattar icons and logos - https://www.awattar.at #5269
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When I look at the awatter website, they have an icon without text and a logo with the text to the right next to it and not below it.
./Klaas
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
i took the images from here (section logos) https://www.awattar.at/company/press so basically there are no icons ... i thought of just using the rings as icon, but i am not sure if this is ok for them as its not in their ci ... ? |
On Twitter they also only use the rings as a profile image and an icon with only the rings will also look better visually later in the integration list. ./Klaas |
In terms of logo, I would prefer the wide variant with text on the right side. ./Klaas |
Yes, /me too, but then you don't have the "energy in sync with nature" text below, but i can change it if you want. |
Ok, sorry, i must have been blind ... one second, i will replace it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ Approved. Can be merged as soon as the parent PR gets merged 👍
Thanks! |
Proposed change
Type of change
Python wheels repository
Additional information
Checklist
icon.png
)icon@2x.png
)logo.png
)logo@2x.png
)