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

Can't get it to work with LiveScript #53

Closed Industrial closed 8 years ago

Industrial commented 8 years ago

Hi!

I'm mainly coding in LiveScript these days and I have one problem:

Alignment of { x, y } = z does not work.

gulp-plumber     = require \gulp-plumber
gulp-spawn-mocha = require \gulp-spawn-mocha
handle-error     = require \../lib/handle-error
{ some, things, here } = require \thing
adrianlee44 commented 8 years ago

Do you use any additional language package for LiveScript?

Industrial commented 8 years ago

Yes, I use https://atom.io/packages/language-livescript

adrianlee44 commented 8 years ago

For clarification, you are looking for the = in { x, y } = z to align with previous =s, right?

Industrial commented 8 years ago

Yes, that would be great :-)

adrianlee44 commented 8 years ago

The issue is with the grammar. Currently the grammar is parsing } = as one token instead of 2. I'll look into creating a PR for that grammar.

adrianlee44 commented 8 years ago

It also seems like the package is not maintained for a while now so creating a PR might not fix this issue.

Industrial commented 8 years ago

Okay, I will look into continuing/owning that package this weekend :-)

adrianlee44 commented 8 years ago

Let me know if you do as I have a quick patch for the grammar.

I'm going to close this for now.