search

adrianlee44 / atom-aligner

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

Sass/SCSS, alignment only works in Bracket context #21

Closed olafgleba closed 9 years ago

olafgleba commented 9 years ago

Hi,

atom-aligner + atom-aligner-scss installed. I am not sure which package causes following behaviour:

Works:

.foo {
  // Class context, alignment works well with any <key|property> combination

  $table-padding:      $base-spacing-unit !default;
  $table-border-width: $base-font-size !default;
}

Fails:

// No context (e.g. declaring variables in a configuration.scss file)

$table-padding: $base-spacing-unit !default;
$table-border-width: $base-font-size !default;

Am i doing something stupid or is it intended behaviour? Thx for your answer.

adrianlee44 commented 9 years ago

This has to do with language-sass not parsing the second case correctly. I'll make a PR with atom/language-sass

olafgleba commented 9 years ago

Thanks for your effort. And for making this package.

adrianlee44 commented 9 years ago

Created atom/language-sass#72. Will update aligner-scss when the PR is merged.

adrianlee44 commented 9 years ago

With that latest Atom v1.0.3 update, it comes with language-sass v0.40.0 which fixes :.

Updating Atom to v1.0.3 and aligner-sass to v1.0.2 should fix this.