Yep, it won't work without the colon. The other themes should all have colons as well. 

On Aug 6, 2013, at 12:52 PM, Kelley Reynolds <notifications@github.com> wrote:

> In source/cart.html:
> 
> > @@ -4,7 +4,7 @@
> >    <ul id="cart_contents" class="clearfix">
> >      {% for item in cart.items %}<li id="item_{{ item.id }}" class="{% cycle 'row', 'row_alt' %}">
> >        <div class="item_wrap">
> > -        <div class="thumbnail"><a href="{{ item.product.url }}" title="View {{ item.product.name | escape }}"><img src="{{ item.product.image | product_image_url size:"thumb" }}" /><span class="stilt"></span></a></div>
> > +        <div class="thumbnail"><a href="{{ item.product.url }}" title="View {{ item.product.name | escape }}"><img src="{{ item.product.image | product_image_url: "thumb" }}" /><span class="stilt"></span></a></div>
> Is this still supposed to have a colon in it? I already merged one theme and didn't notice it then.
> 
> —
> Reply to this email directly or view it on GitHub.