search

suitcss / suit

Style tools for UI components
http://suitcss.github.io/
3.8k stars 225 forks source link

Update STYLE.md to allow function arguments to be split across multiple lines #123

Closed jeddy3 closed 8 years ago

jeddy3 commented 8 years ago

Hello,

This small PR adds long, comma separated function arguments to the list of things that can be separated across multiple lines if it aids in the readability of the code e.g. 

.selector {
  background-image: repeating-linear-gradient(
    -45deg,
    transparent,
    transparent 25px,
    rgba(255, 255, 255, 1) 25px,
    rgba(255, 255, 255, 1) 50px
  );
}

We've added a couple of rules to stylelint recently that should cover all the things listed in the "Exceptions and slight deviations" section of STYLE.md.

Once this PR is decided upon I'll update stylelint-config-suitcss to reflect this and get version 1.0.0 out.

Please don’t hesitate to give me a holler if you stumble into any issues with the config when you get around to #122, as I’ll be happy to help :)

simonsmith commented 8 years ago

This looks good to me!

Thanks for your efforts on the stylelint config too. We'll get that implemented once a bigger task of switching to postcss is out the way.

giuseppeg commented 8 years ago

@jeddy3 :+1: