Why does $baseColWidth have to be set manually and not automatically?

[sebastian-ruiz]

In the _var.scss, $baseColWidth: 40px, for when the $baseWidth: 960px.
However, supposing we wanted to have a $baseWidth of 1200px, we would have to change $baseColWidth: 55px.

May I suggest the following:

$baseColWidth: ($baseWidth/$baseColCount) - $baseGutterWidth;

This could also be applied in the @mixin _fixedGrid, used in _dependencies.scss, which would mean the 28px value could be removed in skeleton.scss:

@include grid($tabletWidth, false, 28px); // 28px no longer necessary if $colWidth set dynamically.

[atomicpages]

Love it, @vephinx ! Making this happen now!

[atomicpages]

Update, since @include grid() can be called without parameters, we need the defaults to stay in _vars.scss. We can run the calculations inside the grid mixin since _fixedGrid takes $width, $colWidth, $gutterWidth, $colCount as parameters. We can simply remove the $colWidth parameter from grid like so:

@mixin grid($width: $baseWidth, $fluid: $isFluid, $gutterWidth: $baseGutterWidth, $colCount: $baseColCount) {
    @if ( $fluid == true ) {
        @include _fluidGrid($colCount);
    } @else {
        // we no longer need to set a value for $colWdith because it is generated dynamically in _dependencies.scss
        // @see https://github.com/atomicpages/skeleton-sass/issues/7
        $colWidth: ($width / $colCount) - $gutterWidth;
        @include _fixedGrid($width, $colWidth, $gutterWidth, $colCount);
    }
}