breakpoint-slicer icon indicating copy to clipboard operation
breakpoint-slicer copied to clipboard

Eyeglass support

Open polarathene opened this issue 9 years ago • 6 comments

Any chance of providing support for Eyeglass? It's a JS equivalent to Compass from what I understand. The support would make it possible to use Breakpoint Slicer via LibSass on Node.js I think?

Details on how to support Eyeglass is here: https://github.com/sass-eyeglass/eyeglass/blob/master/docs/eyeglass_modules/index.md#bringing-eyeglass-to-existing-projects

polarathene avatar Jun 20 '15 10:06 polarathene

I am counting on support for Eyeglass too :) thanks.

ManasN avatar Jun 22 '15 05:06 ManasN

Hi.

I've published the project on npm and created an Eyeglass config. I'm not using Eyeglass yet, so I dunno if the config is correct.

Can you please try installing and using it via Eyeglass?

lolmaus avatar Nov 21 '15 15:11 lolmaus

Yeah, it doesn't work :(

It appears that for some reason going via eyeglass causes node-sass to choke on the indented syntax you use. Looking around, this issue looks like it might be it: https://github.com/sass/node-sass/issues/1192

stevetweeddale avatar Jul 08 '16 14:07 stevetweeddale

I've just whipped up a quick fork that sidesteps the issue by converting to the scss bracketed syntax.

I don't care much for the older syntax, but I know people sometimes feel strongly about such things. So I can make it a pull request if you'd like to exchange indented sass syntax for working eyeglass support! Otherwise, once that issue is fixed you should be golden.

The other change I made was addressing the following complaint made by eyeglass on compilation:

The following modules did not declare an eyeglass version:
  breakpoint-slicer
Please add the following to the module's package.json:
  "eyeglass": { "needs": "^0.8.3" }

stevetweeddale avatar Jul 08 '16 15:07 stevetweeddale

This sucks. :( All those curlies and semicolons do not bring any benefit, only visual noise.

OK, I'll accept a PR.

lolmaus avatar Jul 08 '16 21:07 lolmaus

I'm sorry for neglecting this.

I have rewritten Breakpoint Slicer from scratch with support for Sass modules.

Please tell me if it works with Eyeglass. If it doesn't, please file a PR and I promise I'll tackle it within a week.

lolmaus avatar Feb 22 '21 07:02 lolmaus