Skip to content
This repository has been archived by the owner on Sep 20, 2021. It is now read-only.

Hipercard - A Brazilian Credit Card #7

Merged
merged 1 commit into from
Sep 12, 2016

Conversation

tinogomes
Copy link
Contributor

No description provided.

@muffinresearch
Copy link
Owner

muffinresearch commented Sep 6, 2016

@tinogomes thanks for the PR.

Any chance you could update it so the Hipercard mark is used with a card shape for the flat, mono and outline versions for consistency with the other icons?

For reference here's the existing set http://muffinresear.ch/payment-icons/ so you can see how they look together.

@tinogomes
Copy link
Contributor Author

I will do that. My mistake!

@muffinresearch
Copy link
Owner

I will do that. My mistake!

No worries at all, I appreciate the contribution!

@tinogomes
Copy link
Contributor Author

@muffinresearch fixed outline, flat, and mono images.

@muffinresearch
Copy link
Owner

@tinogomes looks good good to me. For the flat version, is the grey background + text rgba(238, 238, 238) colour part of the hipercard colour scheme? If that's not a colour dictated by the logo I wonder if nothing else the text should be #fff.

@tinogomes
Copy link
Contributor Author

@muffinresearch You are right! DONE!

@muffinresearch
Copy link
Owner

@tinogomes - Muito obrigado! I'll merge this, generate the minified files and cut a release shortly!

@muffinresearch muffinresearch merged commit d1e8a01 into muffinresearch:master Sep 12, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants