discolabs / cartjs

A Javascript library to power cart management for Shopify themes.
https://cartjs.org
MIT License
479 stars 85 forks source link

Add some improved documentation around cart page possibilities #18

Open gavinballard opened 9 years ago

gavinballard commented 9 years ago

For example, using rv-href to link to products and the productImageSize filter.

bakura10 commented 9 years ago

Hi :),

I've tried to use the productImageSize filter, but it always return me the biggest image:

<img rv-src="item.image | productImageSize small">

What am I doing wrong?

gavinballard commented 9 years ago

This is why it needs more documentation :).

The productImageSize filter relies on Shopify's Shopify.Image.getSizedImageUrl method being available, although I believe that should be available on any page with option_selectors.js included. But anyway, first step would be to double check that method's available by entering that in your console.

Another thing that might be causing a problem here is that reading the upgrade notes for Rivets.js from 0.6.0 (the bundled version prior to v0.3.0 of Cart.js) to 0.8.0, they note that arguments to formatters should now be wrapped in quotes if they're meant to be strings, as in the case of the small argument here. So I'd try changing your example to:

<img rv-src="item.image | productImageSize 'small'">

and seeing if that helps.

bakura10 commented 9 years ago

Bingo ! Adding the quotes worked! Thanks a lot !

Envoyé de mon iPhone

Le 28 juin 2015 à 12:11, Gavin Ballard notifications@github.com a écrit :

This is why it needs more documentation :).

The productImageSize filter relies on Shopify's Shopify.Image.getSizedImageUrl method being available, although I believe that should be available on any page with option_selectors.js included. But anyway, first step would be to double check that method's available by entering that in your console.

Another thing that might be causing a problem here is that reading the upgrade notes for Rivets.js from 0.6.0 (the bundled version prior to v0.3.0 of Cart.js) to 0.8.0, they note that arguments to formatters should now be wrapped in quotes if they're meant to be strings, as in the case of the small argument here. So I'd try changing your example to:

and seeing if that helps.

— Reply to this email directly or view it on GitHub.

michaelrshannon commented 8 years ago

@gavinballard Would you like me to factor this issue into work on #55?

gavinballard commented 8 years ago

@michaelrshannon That would be awesome, thank you!

I'd also like to bring product images into the advanced cart example on the Cart.js homepage.

michaelrshannon commented 8 years ago

Np - I can do that at the same time.

dungle-scrubs commented 6 years ago

I love the plugin but I'm stuck on getting images to appear, tried the method above. Is new documentation still going to happen? Particularly around Rivets.

gavinballard commented 6 years ago

@michaelrshannon? ;)