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

fix(rome_js_formatter): shothanded named import line break with default import #3826

Merged
merged 9 commits into from
Nov 23, 2022
21 changes: 21 additions & 0 deletions crates/rome_js_formatter/src/js/module/import_named_clause.rs
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,27 @@ impl FormatNodeRule<JsImportNamedClause> for FormatJsImportNamedClause {
write!(f, [space(), r_curly_token.format()])
}
}
(
Ok(JsAnyNamedImportSpecifier::JsNamedImportSpecifier(specifier)),
Ok(separator),
) => {
if f.comments().has_comments(specifier.syntax()) {
write!(f, [named_import.format()])
} else {
let JsNamedImportSpecifiersFields {
l_curly_token,
specifiers: _,
r_curly_token,
} = specifiers.as_fields();
write!(f, [l_curly_token.format(), space(), specifier.format(),])?;

if let Some(separator) = separator {
format_removed(separator).fmt(f)?;
}

write!(f, [space(), r_curly_token.format()])
}
}
_ => write![f, [named_import.format()]],
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,6 @@ export { loooooooooooooooooooooooooooooooooooooooooooooooooong, } from "looooooo

export {
// comment
loooooooooooooooooooooooooooooooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooong"
loooooooooooooooooooooooooooooooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooong"

export { a as b } from "loooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong"
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@ export { loooooooooooooooooooooooooooooooooooooooooooooooooong, } from "looooooo
export {
// comment
loooooooooooooooooooooooooooooooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooong"

export { a as b } from "loooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong"
```


Expand Down Expand Up @@ -64,11 +66,14 @@ export {
loooooooooooooooooooooooooooooooooooooooooooooooooong,
} from "loooooooooooooooooooooooooooooooooooooooooooooong";

export { a as b } from "loooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong";


## Lines exceeding width of 80 characters

13: export { loooooooooooooooooooooooooooooooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooong";
15: export { loooooooooooooooooooooooooooooooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooong";
22: export { a as b } from "loooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong";
```


Original file line number Diff line number Diff line change
Expand Up @@ -25,4 +25,8 @@ import { loooooooooooooooooooooooooooooooooooooooooooooooooong, } from "looooooo

import {
// comment
loooooooooooooooooooooooooooooooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooong"
loooooooooooooooooooooooooooooooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooong"

import { a as b } from "loooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong"

import a, { loooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooooonnoong"
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,11 @@ import { loooooooooooooooooooooooooooooooooooooooooooooooooong, } from "looooooo
import {
// comment
loooooooooooooooooooooooooooooooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooong"

import { a as b } from "loooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong"

import a, { loooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooooonnoong"

```


Expand Down Expand Up @@ -90,11 +95,18 @@ import {
loooooooooooooooooooooooooooooooooooooooooooooooooong,
} from "loooooooooooooooooooooooooooooooooooooooooooooong";

import { a as b } from "loooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong";

import a, {
loooooooooooooooooooooong,
} from "loooooooooooooooooooooooooooooooooooooooooooooooonnoong";


## Lines exceeding width of 80 characters

29: import { loooooooooooooooooooooooooooooooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooong";
31: import { loooooooooooooooooooooooooooooooooooooooooooooooooong } from "loooooooooooooooooooooooooooooooooooooooooooooong";
38: import { a as b } from "loooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong";
```