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 banks brand colors #284

Open
ai opened this issue Oct 17, 2016 · 13 comments
Open

Add banks brand colors #284

ai opened this issue Oct 17, 2016 · 13 comments

Comments

@ai
Copy link

ai commented Oct 17, 2016

There is a good and open banks BINs database with banks names and brand colors.

I think we could do form UX better if we will show bank name and color while user input a card number (we could get bank by first 6 card numbers, BIN). Here is a demo: https://ramoona.github.io/banks-db-demo/

In fact, in Russia few big banks already use this UX: Alfabank, Tinkoff and Rocket bank. And all my friends love this behavior.

@jessepollak what do you think about it?

@HTMLGuyLLC
Copy link

HTMLGuyLLC commented Oct 31, 2016

Probably not good for production yet in America. Good idea tho. I tried a couple of my own cards with no luck.

@ai
Copy link
Author

ai commented Oct 31, 2016

@VersatilityWerks sure. But it is a open source database. Feel free to add your BINs there.

@HTMLGuyLLC
Copy link

Yea, every DB starts somewhere. Just saying this one needs some work. I'll contribute when I get a moment.

@HTMLGuyLLC
Copy link

I'm sure there are a lot more, but I took a couple minutes to add SunTrust and Navy Federal. Also, the list of BINs I found for SunTrust may have only been for their Debit Cards. May need to have another set of them added for Credit Cards. Not sure on that.

@ai
Copy link
Author

ai commented Nov 1, 2016

Great, thanks!

@jessepollak what do you think about this idea?

@jessepollak
Copy link
Owner

@ai I think it's a good idea, though I'm also wary of switching to this if it degrades the ability to detect card providers etc. Are you suggesting we switch entirely to this for card detection or add it as an additional layer on top of our current card detection? Also, do you know how large the dataset is to do this detection?

@HTMLGuyLLC
Copy link

@jessepollak - They only have a couple/few for the United States and it's a very incomplete library. Might be cool to add on top of existing detection to show specific banks when they exist, but I would NOT recommend switching completely.

@ai
Copy link
Author

ai commented Nov 1, 2016

@VersatilityWerks I agree that this library don’t have all banks, but it is the most complete database of banks brand colors.

Card library right now has some different database for banks names?

@ai
Copy link
Author

ai commented Nov 1, 2016

@jessepollak “card providers” is bank names (Citi Bank, Bank of America, …) or card types (Visa/MasterCard/…)?

@HTMLGuyLLC
Copy link

HTMLGuyLLC commented Nov 1, 2016

Assuming you're suggesting the bank's color be added as a background...a concern for me would be lack of control to make sure the font is contrasting. If a bank's color is navy blue, you probably can't have black text over the top. There will need to be additional considerations.

@ai
Copy link
Author

ai commented Nov 1, 2016

@VersatilityWerks a11n is a good concern.

This is why banks-db suggests to use postcss-contrast for CSS-only or contrast for CSS-in-JS implementation.

background: color,
color: contrast(color) === 'light' ? 'black' : 'white'

@HTMLGuyLLC
Copy link

HTMLGuyLLC commented Nov 1, 2016

That works? Dang. Never seen that before and I can't get it to work for me. Plus...how cross-browser compatible is it?

Update: after some googling, it looks like you'll probably need a JS library to achieve that.
https://24ways.org/2010/calculating-color-contrast/

@shkupjane
Copy link

how can i use binlist in this plugin @jessepollak ?
i want get bank name. can u help me plz?

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

No branches or pull requests

4 participants