fix: don't add extra whitespace around fields

closes #8785

diff --git a/crates/ide/src/diagnostics/fixes/fill_missing_fields.rs b/crates/ide/src/diagnostics/fixes/fill_missing_fields.rs
index 123c2f0af0762d1b4f1ac0c64e865b53c7a4e65e..37a0e37a9f65ea3d2e99655e1fddfa4d5bfcbc72 100644 (file)
--- a/crates/ide/src/diagnostics/fixes/fill_missing_fields.rs
+++ b/crates/ide/src/diagnostics/fixes/fill_missing_fields.rs
@@ -186,6 +186,31 @@ fn test_fn() {
     let one = 1;
     let s = TestStruct{ ..a };
 }
+"#,
+        );
+    }
+
+    #[test]
+    fn test_fill_struct_fields_blank_line() {
+        check_fix(
+            r#"
+struct S { a: (), b: () }
+
+fn f() {
+    S {
+        $0
+    };
+}
+"#,
+            r#"
+struct S { a: (), b: () }
+
+fn f() {
+    S {
+        a: (),
+        b: (),
+    };
+}
 "#,
         );
     }

diff --git a/crates/syntax/src/ast/edit_in_place.rs b/crates/syntax/src/ast/edit_in_place.rs
index 14624c68252c745b0d50a0e7f33970aa78f0e816..2676ed8c981e865971e40802c3056e1054db2790 100644 (file)
--- a/crates/syntax/src/ast/edit_in_place.rs
+++ b/crates/syntax/src/ast/edit_in_place.rs
@@ -378,6 +378,10 @@ pub fn add_field(&self, field: ast::RecordExprField) {
             make::tokens::single_space()
         };
 
+        if is_multiline {
+            normalize_ws_between_braces(self.syntax());
+        }
+
         let position = match self.fields().last() {
             Some(last_field) => {
                 let comma = match last_field