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

Added ability to show both faces of the card at the same time. #120

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cvrabie
Copy link

@cvrabie cvrabie commented Jan 19, 2014

I'm currently working on a project where the user base is skewed toward people with very little experience with computers. After doing some user testing I discovered that the only negative comment about our payment section is that not everybody understands right away that they need to flip the card (even though there's a big red/green button that says so ... i know).

I also saw a few other people make similar comments in the issues section, and a few websites like pingdom which chose to go with a two-face visual card option.

So I've made this patch that allows you to disable the flip and show both faces at the same time. It seemed to be better received by our targeted customers.
Skeuocard two-face.

The solution is completely CSS based. You can make it work by simply adding the two-face class on the skeuocard element. Alternatively you can also specify it in the initialization parameters like this

new Skeuocard($("#skeuocard"), {
  twoFace: true
});

Not sure if you're interested in merging such a feature in the main branch but here it is. Also, thank you very much for making Skeuocard available in the first place!

@brightcommerce
Copy link

Nice work Cristian, I like it. I think would be our preferred way to display it too. Thanks for sharing.

…d changes included.

Since skeuocard basically works only for IE10+ we almost always have to provide a good no-js alternative. In order to style the inputs we will often include them in divs or other containers which will be removed by the initialisation JS. Thanks @leek for the fix.
Refs kenkeiter#103 kenkeiter#101
Fixes kenkeiter#108
@jackspirou
Copy link
Collaborator

@cvrabie this is an interesting feature. Can you merge into dev instead of master? Thx!

@Zakay
Copy link

Zakay commented Nov 4, 2014

Great! Exactly what it needed!

@ghost
Copy link

ghost commented Dec 9, 2014

Just what I needed. Thanks.

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