Reason: Placement of closing `>` in JSX

[alexfedoseev]

/* Now */
<Component
  prop1
  prop2>
  ("child" |> ReasonReact.stringToElement)
</Component>

/* VS */
<Component
  prop1
  prop2
>
  ("child" |> ReasonReact.stringToElement)
</Component>

I used to the latter, imo it's easier to spot this place this way. Also, it's consistent with:

<Component
  prop1
  prop2
/>

[zploskey]

Another option would be to further indent the properties.

<Component
    prop1
    prop2>
  ("child" |> ReasonReact.stringToElement)
</Component>

This makes the distinction between props and children more clear without adding an extra line. The Google style guide suggests formatting tags this way.

[alexfedoseev]

I'd be consistent w/ Prettier and /> 😛

[zploskey]

I agree it would be best if /> and > formatted consistently.

[alexfedoseev]

@IwanKaramazow @rickyvetter TBH current behavior introduces a lot of inconvenience:

• it's not possible to delete last prop w/o moving closing > to the next line
• it's not possible to comment out last prop w/o moving closing > to the next line
• it's not possible to add new prop after the last on by pressing o (vim) or Cmd + Enter w/o moving closing > to the next line.

Can it be reconsidered?