Vertical alignment doesn't treat , correctly

[arrdem]

(def ^:private start-element
  {:meta "^", :meta* "#^", :vector "[",       :map "{"
   :list "(", :eval "#=",  :uneval "#_",      :fn "#("
   :set "#{", :deref "@",  :reader-macro "#", :unquote "~"
   :var "#'", :quote "'",  :syntax-quote "`", :unquote-splicing "~@"})

indents with align-vertically to

(def ^:private start-element
  {:meta "^"  , :meta* "#^" , :vector        "["  ,       :map        "{"
   :list "("  , :eval  "#=" ,  :uneval       "#_" ,      :fn          "#("
   :set  "#{" , :deref "@"  ,  :reader-macro "#"  , :unquote          "~"
   :var  "#'" , :quote "'"  ,  :syntax-quote "`"  , :unquote-splicing "~@"})

since , is lexical whitespace, the ,s should remain attached to the preceding forms.

[Malabarba]

That happens because clojure-forward-logical-sexp doesn't skip over the comma. I wonder whether we should fix that in forward-sexp, clojure-forward-logical-sexp, or in clojure-align.

[bbatsov]

Seems to me forward-sexp would be the best place to fix this, as everything else builds on top of it.

[arrdem]

I'll take a swing at this after I get 8hrs.

[bbatsov]

@arrdem I guess you meant to write 8days. :-)

[arrdem]

squints, drags self out of bed