This repository has been archived by the owner on Aug 30, 2018. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 473
v1.3.0 #80
Merged
Merged
v1.3.0 #80
Changes from 13 commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
14bc64a
Minor helpers and removal of defined anchor styles
cshold 7d91a20
Needs the defaults in the preset or they break
cshold 018558f
main content and breadcrumb margins
cshold dfaba42
Cleaner mobile styles
cshold 95089b6
IE10 header table layout fix
cshold f63eef7
Moved button css under links and button header
cshold b2cba6a
Consolidated .rte styles
cshold 776ece1
Swap the product photo with the thumbnails
cshold e4168aa
Product page photo sizes
cshold e975b4b
Updated footer styles
cshold 0c41e2f
Cleaned up product page price markup
cshold 9c56c35
Found a rogue space
cshold b76d180
restructured new JS file
cshold 25eaf9e
Focus styles on buttons. Specific button class helpers
cshold bf7944e
Updated into to theme settings text
cshold bdf0763
Restructured theme settings
cshold 346b2fe
Updated theme colors
cshold 0b8ec8d
Minor settings updates
cshold 55b851e
Collection sorting and views. Ajax cart border update
cshold 249e834
Hide list/grid view links in oldIE
cshold dbf846d
Empty product page text
cshold 446ebee
Globalized JS variables
cshold eaad033
Updated JS variables
cshold 2b5067a
Removed push/pull from small breakpoint
cshold 17fe5c2
More specific modal overlay selector
cshold 83a3409
Cleaner sorting styles
cshold 492a463
Updated cart page layout a little
cshold 47f3cce
Added type=search to styles
cshold 9619eff
Changed search bar to type=search
cshold 1763465
Minor cart style updates. Renamed class in product-list-item
cshold 2e068be
Multi-screen-size modal sizing
cshold 9f41e8b
More social theme settings
cshold 9270fd5
Moved search-bar styles into header section
cshold 2ab9097
Added minimal icon set for social/payments
cshold ba00104
Removed old payment icon css
cshold 5562ce9
New americna express and mastercard logos
cshold ff976a3
Fixed goolge credit card icon
cshold c44238c
CSS at-query formating
cshold 0241ab8
Cleaned product price code
cshold 725370b
Minor header style update
cshold 0fa0adb
Bolded theme setting headers
cshold 15b2270
Removed question marks from settings
cshold 01f1475
Added related product snippet. Cleaned settings markup
cshold 71a3f4e
More specific JS selectors
cshold 04ee025
Added Fancy social setting. Updated icon set
cshold 1097df7
Longer delay for modal close button position
cshold 3acd390
Cleaned up cart template and styles
cshold 43f7580
Less specific selectors on buttons
cshold c1f9bf7
Basic color settings added
cshold 0f06d17
Footer colors
cshold 204cb4a
Social/payment icon colors
cshold 82bc344
Cleaned social icons styles
cshold 9936572
Keep logo floated
cshold 17879fd
Consistent tab spaces. Escape twitter/fb meta tags
cshold 4a88388
More indent fixes
cshold 42e9d8b
Remember form fields on contact page if there is an error
cshold 9850400
Hidden (accessible) labels
cshold 73fb485
Organized address form. More hidden labels
cshold 0f3285e
Autofocus login/register forms
cshold 920e441
Max logo width
cshold 3a46f64
Added ability to easily change collection-grid-item widths
cshold bea95cd
Frontpage empty state
cshold File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,36 @@ | ||
$(function() { | ||
window.timber = window.timber || {}; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Looks like I lost the line comment after my update, but appreciate the notes. Rich, I assume this is what you meant by splitting the methods up? Cached elements are more aptly named. Will consider moving other selectors up there too, though I sometimes prefer keeping the relevant ones with their methods.. just preference there. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Comments are still in the conversation view under "Show outdated diff", but yep, that's what I meant! |
||
|
||
timber.cache = { | ||
html: $('html'), | ||
body: $('body') | ||
} | ||
|
||
timber.init = function () { | ||
timber.cache.html.removeClass('no-js').addClass('js'); | ||
|
||
// Run on load | ||
timber.productImageSwitch(); | ||
} | ||
|
||
$('html').removeClass('no-js').addClass('js'); | ||
timber.productImageSwitch = function () { | ||
var mainImage = $('#productPhoto img'), | ||
thumbImages = $('#productThumbs a'), | ||
newImage; | ||
|
||
if (!thumbImages.length) { | ||
return; | ||
} | ||
|
||
// Switch the main image with one of the thumbnails | ||
// Note: this does not change the variant selected, just the image | ||
thumbImages.on('click', function(e) { | ||
e.preventDefault(); | ||
newImage = $(this).attr('href'); | ||
mainImage.attr({ src: newImage }); | ||
}); | ||
} | ||
|
||
// Initialize Timber's JS on docready | ||
$(function() { | ||
window.timber.init(); | ||
}); |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this won't work with IE8, assuming that's ok for Timber?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, just sets the border as transparent to get ready for a CSS transition so it can be ignored for IE8.