A minor nit pick as well...but i'd like to ask a question regarding design
elements... what purpose does the container within the container solve?
There is a gray global background with a white content body container and
within the white container, a gray container. It seems like the inner gray
container adds little design value given that the content within those
containers either a) can belong to the parent container or b) has buttons
within the container that can stand on their own. Just had this thought
after looking at it a second time. Not being difficult, but i do think
every component in a design should have a clear argument for existence :)


On Thu, Feb 20, 2014 at 5:43 PM, tristanoneil <notifications@github.com>wrote:

> In app/assets/stylesheets/mailers.css.scss:
>
> > @@ -177,7 +166,7 @@ a { color: orange; }
> >
> >  .button:hover, .button-small:hover { opacity: .8; }
> >  .button:hover, .button-small:hover{opacity:.8;}
> > -.lightbutton {background-color: #a0a9ba;}
> > +.lightbutton {background-color: #a0a9ba; }
>
> But yea these look really [image: :fire:]
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/opscode/supermarket/pull/138/files#r9928530>
> .
>



-- 

VP of Product   |   cramerdevelopment.com
Skype: gabe.weaver   |  + 1 (515) 292-0050 x 1301