[JavaScript] Make consistent format for object property values with a string literal and template string

[dimaMachina]

Description

Fixes https://github.com/prettier/prettier/issues/12408 Consistent format, do not break a new line for long string literals

Checklist

• [x] I’ve added tests to confirm my change works.
• [x] (If changing the API or CLI) I’ve documented the changes I’ve made (in the docs/ directory).
• [x] (If the change is user-facing) I’ve added my changes to changelog_unreleased/*/XXXX.md file following changelog_unreleased/TEMPLATE.md.
• [x] I’ve read the contributing guidelines.

✨Try the playground for this PR✨

[dimaMachina]

@fisker @sosukesuzuki Hey guys 👋, it's been over a month since I created my small PR that fixes the current inconsistency, can someone review it and say what you think?

[fisker]

Sorry for the long delay.

I'm not sure if we want to change it, at least not until a better solution found. Making an exception here is really bad. People will ask why long identifiers/numbers/bigints don't format like this. If you want to change the output, add more tests, we'll see if it's fine to accept.

[dimaMachina]

@fisker which tests would you prefer that I add? Only for properties of an object?

[fisker]

long identifiers/numbers/bigints

[dimaMachina]

@fisker added more tests

[dimaMachina]

@fisker @sosukesuzuki friendly ping

[thorn0]

The current behavior for string literals is intentional. Please see https://prettier.io/blog/2021/05/09/2.3.0.html#prevent-wrapping-object-properties-with-short-keys-10335httpsgithubcomprettierprettierpull10335-by-thorn0httpsgithubcomthorn0

If we're going to change something here, that should be the behavior for template literals.