Skip to content
This repository has been archived by the owner on Dec 2, 2018. It is now read-only.

Add an option to align vertically the braces #21

Open
PhiLhoSoft opened this issue Sep 8, 2015 · 5 comments
Open

Add an option to align vertically the braces #21

PhiLhoSoft opened this issue Sep 8, 2015 · 5 comments

Comments

@PhiLhoSoft
Copy link

I wonder why it isn't there already... 😄

h1 
{
    color: red;
}
@ben-eb
Copy link
Owner

ben-eb commented Sep 8, 2015

Would accept a PR with this option.

@ghost
Copy link

ghost commented Oct 6, 2016

You can just copy applyExpanded.js to some applyVerticalAlign.js, and replace in 101 row:
rule.raws.between = ' '; => rule.raws.between = '\n';

@ben-eb
Copy link
Owner

ben-eb commented Oct 6, 2016

Sure, this is on the easy end of the spectrum which is why I tagged it as help wanted. I can't do everything on this list.

@ghost
Copy link

ghost commented Oct 6, 2016

Man, your job is awesome, without sarcasm.
I created temp fork for use on my job, by this issue and #53 issues:
https://github.com/dmitriy-lodyanov/perfectionist
This commits probably help you. Or i can make pull request later, if you want.

@ben-eb
Copy link
Owner

ben-eb commented Oct 6, 2016

@dmitriy-lodyanov A pull request would be great! 😄

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

2 participants