Fix surround_replace replacing the wrong character on the right. (#571)

Fixes #569.
helix-editor · Aug 11, 2021 · dde2be9 · dde2be9
1 parent 27b551d
commit dde2be9
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/helix-term/src/commands.rs b/helix-term/src/commands.rs
@@ -4017,11 +4017,11 @@ fn surround_replace(cx: &mut Context) {
                     let transaction = Transaction::change(
                         doc.text(),
                         change_pos.iter().enumerate().map(|(i, &pos)| {
-                            if i % 2 == 0 {
-                                (pos, pos + 1, Some(Tendril::from_char(open)))
-                            } else {
-                                (pos.saturating_sub(1), pos, Some(Tendril::from_char(close)))
-                            }
+                            (
+                                pos,
+                                pos + 1,
+                                Some(Tendril::from_char(if i % 2 == 0 { open } else { close })),
+                            )
                         }),
                     );
                     doc.apply(&transaction, view.id);