Mixin "EQSTATECHECK" does not accept a content block.

lmartins commented 8 years ago

Hi Sam,

Just to let you know that using latest beta releases from gulp-sass and node-sass the eq mixin throws the following error:

events.js:141
      throw er; // Unhandled 'error' event
      ^
Error: jspm_packages/github/Snugug/eq.js@1.7.1/sass/_eq.scss
  31:12  Mixin "EQSTATECHECK" does not accept a content block.

Backtrace:
    jspm_packages/github/Snugug/eq.js@1.7.1/sass/_eq.scss:31, in mixin `eq`
    src/sass/_shame.scss:469
    at options.error (/Users/luismartins/Sites/rulergy.dev/wp-content/themes/rulergy-marketing/node_modules/gulp-sass/node_modules/node-sass/lib/index.js:277:32)

This with something like the following on my Sass file:

.banner-home{
    @include eq-pts((
        small: 400,
        medium: 500,
        large: 700
    ));

    @include eq('small', 'medium', 'large') {
      font-size: 1em;
    }
}

jonscottclark commented 8 years ago

If you're lazy like me, in the meantime you can make a copy of the mixins file from the bower/npm repo and remove the error-checking function (what's the worst that could happen? Famous last words.)

https://gist.github.com/jonscottclark/6255c80b46371f2d4ac6

lmartins commented 8 years ago

Thanks for the tip @jonscottclark I've abandoned the idea of using those mixins though. Because im now unable to provide any feedback about it, will close this.

Snugug / eq.js

Mixin "EQSTATECHECK" does not accept a content block. #59