Skip to content
This repository was archived by the owner on Oct 22, 2019. It is now read-only.

SASS? #9

Closed
surfjedi opened this issue Jan 25, 2016 · 8 comments
Closed

SASS? #9

surfjedi opened this issue Jan 25, 2016 · 8 comments

Comments

@surfjedi
Copy link

Any chance for SASS support?

@aki77
Copy link
Owner

aki77 commented Jan 26, 2016

I have no plan at the moment.
Welcome new provider.
Please refer to the atom-source-preview-babel.

@danielbayley
Copy link

+1.

@danielbayley
Copy link

I just released source-preview-sass. Although I can't quite seem to get the source mapping to work… @aki77 Any ideas?

@danielbayley
Copy link

Never mind… I just released patch 0.1.1 which fixes source maps. This can be closed now @surfjedi 👍

@surfjedi
Copy link
Author

surfjedi commented May 2, 2016

Cool thanks - but whats to short cut? using source-preview from the palette doesn't do any thing on a .scss file?

@danielbayley
Copy link

danielbayley commented May 3, 2016

but whats to short cut?

Not quite sure what you're asking there… I'm sure you've done apm install source-preview-sass?

I suspect you might be getting a 'Missing binding' error as I suddenly am after updating my homebrew installed version of Node to 6, which is causing a problem with node-sass (the only dependency of my package). It seems this will be fixed in their next release, but in the meantime (if I'm right, and you're on a mac) then you can download the correct darwin-x64-47 binding, rename the downloaded file from darwin-x64-47_binding.node to binding.node and place it inside a darwin-x64-47 folder. Then move this folder to $ATOM_HOME/packages/source-preview-sass/node_modules/node-sass/vendor/darwin-x64-47/binding.node which fixes the problem until the upstream is fixed, at which point I'll update the dependency and the Atom package.

Hope that helps. If there is still an issue then just open one on my package as this issue should be closed.

@danielbayley
Copy link

danielbayley commented May 6, 2016

@surfjedi It turned out that node-sass is currently not compatible with Electron (Atom), so the update didn't fix the problem. It might be in future… at which point I'll probably switch dependencies back to node-sass, but for now I implemented a workaround. So package should work smoothly now, although it takes a while to install.

@aki77 This can be closed.

@aki77
Copy link
Owner

aki77 commented May 6, 2016

@danielbayley Thanks!

@aki77 aki77 closed this as completed May 6, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants