Evan Rutledge Borden
Evan Rutledge Borden
@lpil the issue tracking it :point_right: https://github.com/lspitzner/brittany/issues/145
There are three areas where space can exist, before the first bracket, after the first bracket and before the second bracket. The reasonable options are: ```hs Record{..} Record {..} Record...
I'm not against the "big change" as it binds the record identifier tightly with the opening bracket. For the spacious bracket you'd have: ```hs match Record {} = 1 match...
@lspitzner one way to avoid inconsistency on your previous layout is to have spacious wildcards, but tight identifier/bracket binding. ```hs match Record{} = 1 match Record{ .. } = 2...
This is the big win for me. The fact that record update/construction binds tighter than application makes removing the spurious space (between the identifier and the opening bracket) a positive...
@lspitzner given your :+1:, it looks like you endorse this layout: https://github.com/lspitzner/brittany/issues/251#issuecomment-527611371  @simonchatts has proposed opening a PR, should he move forward with this layout?
> As we are still pre 1.0, I say we accept this breaking change, even without making it configurable. Should people dislike this change, we can always come back and...
Variations ```yaml conf_layout: lconfig_experimental: true ``` or ```yaml conf_layout: lconfig_unstable: true ```
:point_up: unnassigned since `assignees` will be used for active work.
@mrkgnao `brittany` does this in a number of places, but you'll generally be looking in `src/Language/Haskell/Brittany/Internal/Layouters/.`