Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

🐛 Formatter: Arrow with parenthesized JSX Body #3377

Closed
1 task done
MichaReiser opened this issue Oct 10, 2022 · 0 comments · Fixed by #3505
Closed
1 task done

🐛 Formatter: Arrow with parenthesized JSX Body #3377

MichaReiser opened this issue Oct 10, 2022 · 0 comments · Fixed by #3505
Labels
A-Formatter Area: formatter S-Bug: confirmed Status: report has been confirmed as a valid bug
Milestone
10.0.0

Comments

@MichaReiser
Copy link
Contributor

MichaReiser commented Oct 10, 2022 •
edited
Loading

Real world usages

https://github.com/ant-design/ant-design/blob/master/components/list/Item.tsx#L111-L125

What happened?

  1. Open the playground
  2. Observe how Rome formats the opening ( on a new line whereas prettier keeps it on the same line as the =>

Expected result

Rome to format the example the same as prettier as it results in more dense formatting.

Code of Conduct

  • I agree to follow Rome's Code of Conduct
@MichaReiser MichaReiser added S-Bug: confirmed Status: report has been confirmed as a valid bug A-Formatter Area: formatter labels Oct 10, 2022
@MichaReiser MichaReiser mentioned this issue Oct 26, 2022
Merged
@MichaReiser MichaReiser added this to the 10.0.0 milestone Oct 27, 2022
@MichaReiser MichaReiser moved this to Done in Rome 2022 Oct 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-Formatter Area: formatter S-Bug: confirmed Status: report has been confirmed as a valid bug
Projects
Rome 2022
Status: Done
Milestone
10.0.0
Development

Successfully merging a pull request may close this issue.

fix(rome_js_formatter): Arrow with JSX Body rome/tools
1 participant
@MichaReiser