inuitcss/trumps.widths — pull request #2 by emagnier (2013-12-30 06:56:07+00:00)
Title: Uses a mixin to generate all widths

I played a bit in this file/repo and created a mixin to generate dynamically all widths. This is related to https://github.com/inuitcss/trumps.widths/issues/1.

It uses Sass 3.3 and the new map data type (this new version of Sass will probably be released sometime [in January](https://gist.github.com/nex3/8050187)).

`@include widths(2, 3, 4, 5, 6, 8, 9, 10, 12)` can accept any numbers.
So we can easily generate only the widths we need, including after 12. E.g.: `@include widths(4, 12, 16)`.
It also support spoken-word format up to 12 (after 12 the mixin will warn the user and the Sass compiler will crash).

Of course it combine selectors which have the same widths (like the previous implementation).

The arguments in the `width` mixin could be stored in a variable defined in the `_defaults.scss` (if still exists in version 6).

I was not sure, but if I follow your architecture, this mixin should have a better place in `_tools.mixins.scss`. Also, I was wondering why you split Inuit.css in a lot of different repos, because I think it will be more difficult for the community to follow this frameworks, and particularly send pull requests across this project (but this is my point of view and you probably have good reasons to do that).

Feel free to accept or not my PR, and correct my bad English :)