-
Notifications
You must be signed in to change notification settings - Fork 1
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
Homepage 1.0 #62
Homepage 1.0 #62
Conversation
Committed from shop: Konnected
stamped snippet
Committed from shop: Konnected
Committed from shop: Konnected
Committed from shop: Konnected
Committed from shop: Konnected
Committed from shop: Konnected
Committed from shop: Konnected
Committed from shop: Konnected
Committed from shop: Konnected
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.
looks great
…o/shopify-dawn into feature/new-homepage
Committed from shop: Konnected
Fixes/dawnify compare kits
…cted-io/shopify-dawn into fixes/dawnify-compare-kits
This reverts commit 281b27a.
Committed from shop: Konnected
Committed from shop: Konnected
Committed from shop: Konnected
…cted-io/shopify-dawn into fixes/dawnify-compare-kits
Fixes/dawnify compare kits
Committed from shop: Konnected
Committed from shop: Konnected
Committed from shop: Konnected
Committed from shop: Konnected
…o/shopify-dawn into feature/new-homepage
…o/shopify-dawn into feature/new-homepage
@heythisisnate pls check if adaptiveheight feature (on mobile) of the banner slider works fine on your mobile device w/o flickering on slide change. https://nodemcu-smartthings.myshopify.com/?preview_theme_id=162338570555 slider height and dots position should adapt current slide item's height. |
I think it looks good! Yes, we should use the current Black Friday banner |
Preview: https://nodemcu-smartthings.myshopify.com/?preview_theme_id=162338570555
Applied 2 new sections here: