Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

selector_separator_newline: true for Sass doesn't work #563

Closed
joseluisq opened this issue Oct 20, 2014 · 5 comments
Closed

selector_separator_newline: true for Sass doesn't work #563

joseluisq opened this issue Oct 20, 2014 · 5 comments

Comments

@joseluisq
Copy link

Hi, I'm using ST3 and selector_separator_newline: true for Sass doesn't work.

Now:

It should be like:

@bitwiseman
Copy link
Member

Cool, good report, thanks.

@silveringsea
Copy link

is it solved ?

@joseluisq
Copy link
Author

@bitwiseman I hope that team can solve this soon :)
Regards

@darlanalves
Copy link

This is about other config, called newline_between_rules. The latest version shipped with the "HTML-JS-CSS Prettify" plugin on ST3 don't have it, and causes the same problem. I've replaced the js-beautify folder under plugin folder/scripts/node_modules with a clone of this repo and it works.

Check if adding newline_between_rules: true to your config solves the problem.

For the ST3 plugin, the issue is the current version of js-beautify published on npm, which also don't have the config, so publishing a new version of this repo on npm to fix the issue is also necessary.

@joseluisq
Copy link
Author

@darlanalves yes, it is solved by newline_between_rules at 44df1f1
But, The current Js Beautify version is 1.5.4 and the commit that solves this (I suspect) will be released in the next 1.5.5 version.

@bitwiseman bitwiseman added this to the v1.5.5 milestone Jun 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants