Elm-format: Comments cause the value in record to go line below

[amitaibu]

(from here)

[avh4]

Can you clarify what part of the code that comment refers to? If it's meant to go with nextPage, then would the following be reasonable?

type alias Model =
  { login : Login.Model
  , -- If the user is anonymous, we want to know where to redirect them.
    nextPage : Maybe Page
  }

Currently elm-format thinks the comment is associated with Login.Model because the comment comes before the comma.

[amitaibu]

Can you clarify what part of the code that comment refers to?

It refers to the line below it (nextPage).

then would the following be reasonable?

It's looks slightly weird, and not consistent - I'd expect the leading comma before the actual key.
My expectations would be to keep it as is, but of course, I guess it's easier said than done ;)

(p.s. many thanks for this lib - it's awesome!)

[Janiczek]

Related:

type Unit
    -- metric
  = Cm Float
  | Mm Float
  | M Float
    -- imperial
  | In Float
  | Ft Float

results in

type
  Unit
  -- metric
  = Cm Float
  | Mm Float
  | M Float
    -- imperial
  | In Float
  | Ft Float

(To be fair, I don't know what's the right thing to do here. Particularly where to put the first comment in the first code snippet.)

[amitaibu]

I think the "proper" way should be:

type Unit
  -- metric
  = Cm Float
  | Mm Float
  | M Float
  -- imperial
  | In Float
  | Ft Float

[avh4]

This is now a duplicate of https://github.com/avh4/elm-format/issues/97

[avh4]

The original issue about records has been fixed.

The example about type Unit seems contrived, and should be a separate issue if it's still a concern.