search

adrianlee44 / atom-aligner

Easily align multi-line with support for different operators and custom configurations
https://atom.io/packages/aligner
MIT License
77 stars 3 forks source link

CSS/LESS selector with double-colon... #50

Closed emveeoh closed 8 years ago

emveeoh commented 8 years ago

Selectors that require two colons (see example below) are improperly split apart. The second colon is incorrectly aligned with the rest of the css properties. The extra whitespace will break the LESS compiler. Specifically, this happens in LESS documents (.less). Thank you! :)

For example:

atom-text-editor::shadow .cursor { }

Becomes this after applying Aligner:

atom-text-editor: (((((unintended blank white space here))))) :shadow .cursor { }

adrianlee44 commented 8 years ago

Thanks for reporting this. Let me take a look.

adrianlee44 commented 8 years ago

It seems like language-less doesn't support ::shadow properly. I'm going submit a PR to look into fixing it.

adrianlee44 commented 8 years ago

Opened atom/language-less#55 which should fix this issue.

adrianlee44 commented 8 years ago

The PR has been merged and the language package has been bumped. This should be fixe in the next release of Atom.