Update to paddings on wrap utility class

FundingCircle / sass-common

Common Sass mixins and files

BSD 3-Clause "New" or "Revised" License

14 stars 5 forks source link

Update to paddings on wrap utility class #95

Closed lroliphant closed 8 years ago

lroliphant commented 8 years ago

keep default padding of 16px on mobile devices
increase padding to 24px on screen widths larger than 640px
small fix - 1st step towards wider design/dev review of column padding across site
bump patch version

lroliphant commented 8 years ago

@thejuanitalee for review from US side please :eyes:

thejuanitalee commented 8 years ago

LGTM.

One question, do we push the dist changes as well? It's been awhile for me.

Sent from my Rotary Phone On Jan 29, 2016 6:42 AM, "Lucy Oliphant" notifications@github.com wrote:

@thejuanitalee https://github.com/thejuanitalee for review from US side please [image: :eyes:]

— Reply to this email directly or view it on GitHub https://github.com/FundingCircle/sass-common/pull/95#issuecomment-176788217 .

igoratron commented 8 years ago

@lroliphant can we merge this?

lroliphant commented 8 years ago

@igoratron just waiting for Adam to confirm he is happy and then yes

RazvanDH commented 8 years ago

LGTM :sheep: :it: