From 7e9ea30a0b4eee08322d689450efebe74dec8097 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bla=C5=BE=20Hrastnik?= Date: Thu, 22 Apr 2021 23:26:57 +0900 Subject: [PATCH] auto_pairs: move if cursor, extend if selection. --- helix-core/src/auto_pairs.rs | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/helix-core/src/auto_pairs.rs b/helix-core/src/auto_pairs.rs index 3ba271058..bbd1cea45 100644 --- a/helix-core/src/auto_pairs.rs +++ b/helix-core/src/auto_pairs.rs @@ -69,7 +69,12 @@ fn handle_open( let pos = range.head; let next = next_char(doc, pos); - ranges.push(Range::new(range.anchor, pos + open.len_utf8())); // pos + open + let head = pos + open.len_utf8(); + // if selection, retain anchor, if cursor, move over + ranges.push(Range::new( + if range.is_empty() { head } else { range.anchor }, + head, + )); match next { Some(ch) if !close_before.contains(ch) => { @@ -98,7 +103,12 @@ fn handle_close(doc: &Rope, selection: &Selection, _open: char, close: char) -> let pos = range.head; let next = next_char(doc, pos); - ranges.push(Range::new(range.anchor, pos + close.len_utf8())); // pos + close + let head = pos + close.len_utf8(); + // if selection, retain anchor, if cursor, move over + ranges.push(Range::new( + if range.is_empty() { head } else { range.anchor }, + head, + )); if next == Some(close) { // return transaction that moves past close