Skip to content
This repository has been archived by the owner on Aug 30, 2018. It is now read-only.

QR code on printed gift card template #333

Merged
merged 2 commits into from
Feb 18, 2015
Merged

QR code on printed gift card template #333

merged 2 commits into from
Feb 18, 2015

Conversation

cshold
Copy link
Contributor

@cshold cshold commented Feb 17, 2015

Added QR code to printed gift cards. A few other very minor style updates.

  • Center text/logo on print version
  • Reduce header padding so printed version fits on one page

Demo - https://checkout.shopify.com/gift_cards/4790585/preview#

@stevebosworth @carolineschnapp

cshold added a commit that referenced this pull request Feb 18, 2015
QR code on printed gift card template
@cshold cshold merged commit 82debf9 into master Feb 18, 2015
@cshold cshold deleted the gift-card-qr branch February 18, 2015 16:15
@@ -368,6 +368,17 @@ $gutter: 24px;
vertical-align: baseline;
}

#QrCode {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know it's a bit late in the game, but I was wondering about the fact this says #QrCode and not #qr-code.

In the default template provided by Shopify they're using #qr-code. Also, in the mark up Kevin asked us to copy from, the id used is #qr-code.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Timber follows our internal styleguide, and in it we use PascalCase for IDs.

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