Inconsistent formatting for object properties with ternaries

[bradzacher]

Prettier 2.5.1
Playground link

--parser babel
--print-width 25

Input:

const x = {
  prop1: foo == null ? bar : baz,
  prop1: !fooooooooo ? bar : baz,
};

Output:

const x = {
  prop1:
    foo == null
      ? bar
      : baz,
  prop1: !fooooooooo
    ? bar
    : baz,
};

Expected behavior:

Both should look the same - either:

• the condition is on the same line as the property key, or
• the condition is on a different lint to the property key

[fisker]

Should be caused by

prettier/src/language-js/print/assignment.js

Line 188 in 6ccc462

return isBinaryish(test) && !shouldInlineLogicalExpression(test);

[islandryu]

Should it be the same line in variable declarations?
playground link

actual behavior

input:

const a = foo == null ? bar: baz;
const b = !foooooooo ? bar : baz;

output:

const a =
  foo == null
    ? bar
    : baz;
const b = !foooooooo
  ? bar
  : baz;

[thorn0]

Looks like special casing for binaries was added in #2228 to prevent indentation like this:

const x = {
  prop1: foo == null &&
    bar == null
    ? bar
    : baz
};