[prettierx] invalid indentation in case of nested ternary in test condition

[brodybits]

mirror of prettier/prettier#5257

I have reproduced this issue on prettierx with and without using the --no-align-ternary-lines option.

P.S. I found prettier/prettier#5257 by looking through prettier/prettier#5039 where the ternary formatting was changed while investigating the cause and possible solutions for #41.

[brodybits]

Similar issue on JSX, which seems to be exactly the same on Prettier and prettierx:

Input:

(a ? b : c) ? (
  <Element>
    <Sub />
    <Sub />
    <Sub />
    <Sub />
    <Sub />
    <Sub />
  </Element>
) : (
  <Element2>
    <Sub />
    <Sub />
    <Sub />
  </Element2>
)

Output:

(a ? (
  b
) : (
  c
)) ? (
  <Element>
    <Sub />
    <Sub />
    <Sub />
    <Sub />
    <Sub />
    <Sub />
  </Element>
) : (
  <Element2>
    <Sub />
    <Sub />
    <Sub />
  </Element2>
);

[brodybits]

The issue I found on JSX would definitely affect the more verbose ternary formatting that I proposed in #71.

[brodybits]

Looks like a fix should be coming in prettier/prettier#7087, also seems to resolve the ugly JSX formatting issue that I mentioned in #73 (comment) & prettier/prettier#5257 (comment).

I hope it will be merged soon, even better if they release it soon.