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

conformed XMPP logo addition #315

Merged
merged 7 commits into from
Oct 15, 2019
Merged

conformed XMPP logo addition #315

merged 7 commits into from
Oct 15, 2019

Conversation

wuxmedia
Copy link
Contributor

finally.

@edent
Copy link
Owner

edent commented Oct 15, 2019

It is really really close! One last tweak, I promise! Oh, and it is my fault - not yours.

If you set the rx="50%" you'll see that the logo goes outside the bounds.

I've put guidelines on https://github.com/edent/SuperTinyIcons#guidelines - but they're not in contributing. That's my bad and I'll fix it.

updated to fit inside guide.
@wuxmedia
Copy link
Contributor Author

ok i've updated my version which fits. do i burn this PR or will it merge?

@wuxmedia
Copy link
Contributor Author

I need that tshirt man :D

@edent edent merged commit 79f9b83 into edent:master Oct 15, 2019
@edent
Copy link
Owner

edent commented Oct 15, 2019

Hahaha! You've earned that shirt :-)

@alexanderadam alexanderadam mentioned this pull request Oct 15, 2019
@Neustradamus
Copy link

@wuxmedia, @edent: I am not sure that you use the good logo version...

Can you look the good one?

@kuubeu
Copy link
Collaborator

kuubeu commented Aug 27, 2023

@Neustradamus Do you mean the gradient, the text or the overall shape?

@Neustradamus
Copy link

@kuubeu: Old version, there was a problem:

@kuubeu kuubeu mentioned this pull request Aug 27, 2023
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.

4 participants