aboutsummaryrefslogtreecommitdiff
path: root/helix-term/src/commands.rs
diff options
context:
space:
mode:
Diffstat (limited to 'helix-term/src/commands.rs')
-rw-r--r--helix-term/src/commands.rs604
1 files changed, 346 insertions, 258 deletions
diff --git a/helix-term/src/commands.rs b/helix-term/src/commands.rs
index 9b72a8e9..57df47a7 100644
--- a/helix-term/src/commands.rs
+++ b/helix-term/src/commands.rs
@@ -1,9 +1,6 @@
use helix_core::{
comment, coords_at_pos, find_first_non_whitespace_char, find_root, graphemes, indent,
- line_ending::{
- get_line_ending_of_str, line_end_char_index, rope_end_without_line_ending,
- str_is_line_ending,
- },
+ line_ending::{get_line_ending_of_str, line_end_char_index, str_is_line_ending},
match_brackets,
movement::{self, Direction},
object, pos_at_coords,
@@ -124,7 +121,7 @@ enum Align {
}
fn align_view(doc: &Document, view: &mut View, align: Align) {
- let pos = doc.selection(view.id).cursor();
+ let pos = doc.selection(view.id).cursor(doc.text().slice(..));
let line = doc.text().char_to_line(pos);
let relative = match align {
@@ -330,7 +327,8 @@ fn move_char_left(cx: &mut Context) {
let count = cx.count();
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
+
+ let selection = doc.selection(view.id).clone().transform(|range| {
movement::move_horizontally(text, range, Direction::Backward, count, Movement::Move)
});
doc.set_selection(view.id, selection);
@@ -340,7 +338,8 @@ fn move_char_right(cx: &mut Context) {
let count = cx.count();
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
+
+ let selection = doc.selection(view.id).clone().transform(|range| {
movement::move_horizontally(text, range, Direction::Forward, count, Movement::Move)
});
doc.set_selection(view.id, selection);
@@ -350,7 +349,8 @@ fn move_line_up(cx: &mut Context) {
let count = cx.count();
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
+
+ let selection = doc.selection(view.id).clone().transform(|range| {
movement::move_vertically(text, range, Direction::Backward, count, Movement::Move)
});
doc.set_selection(view.id, selection);
@@ -360,7 +360,8 @@ fn move_line_down(cx: &mut Context) {
let count = cx.count();
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
+
+ let selection = doc.selection(view.id).clone().transform(|range| {
movement::move_vertically(text, range, Direction::Forward, count, Movement::Move)
});
doc.set_selection(view.id, selection);
@@ -368,84 +369,67 @@ fn move_line_down(cx: &mut Context) {
fn goto_line_end(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
+ let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
- let text = doc.text();
- let line = text.char_to_line(range.head);
+ let selection = doc.selection(view.id).clone().transform(|range| {
+ let line = range.head_line(text);
- let pos = line_end_char_index(&text.slice(..), line);
- let pos = graphemes::nth_prev_grapheme_boundary(text.slice(..), pos, 1);
- let pos = range.head.max(pos).max(text.line_to_char(line));
+ let mut pos = line_end_char_index(&text, line);
+ if doc.mode != Mode::Select {
+ pos = graphemes::prev_grapheme_boundary(text, pos);
+ }
- Range::new(
- match doc.mode {
- Mode::Normal | Mode::Insert => pos,
- Mode::Select => range.anchor,
- },
- pos,
- )
- });
+ pos = range.head.max(pos).max(text.line_to_char(line));
+ range.put(text, pos, doc.mode == Mode::Select)
+ });
doc.set_selection(view.id, selection);
}
fn goto_line_end_newline(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
+ let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
- let text = doc.text();
- let line = text.char_to_line(range.head);
+ let selection = doc.selection(view.id).clone().transform(|range| {
+ let line = range.head_line(text);
- let pos = line_end_char_index(&text.slice(..), line);
- Range::new(pos, pos)
+ let mut pos = text.line_to_char((line + 1).min(text.len_lines()));
+ if doc.mode != Mode::Select {
+ pos = graphemes::prev_grapheme_boundary(text, pos);
+ }
+ range.put(text, pos, doc.mode == Mode::Select)
});
-
doc.set_selection(view.id, selection);
}
fn goto_line_start(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
+ let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
- let text = doc.text();
- let line = text.char_to_line(range.head);
+ let selection = doc.selection(view.id).clone().transform(|range| {
+ let line = range.head_line(text);
// adjust to start of the line
let pos = text.line_to_char(line);
- Range::new(
- match doc.mode {
- Mode::Normal => range.anchor,
- Mode::Select | Mode::Insert => pos,
- },
- pos,
- )
+ range.put(text, pos, doc.mode == Mode::Select)
});
-
doc.set_selection(view.id, selection);
}
fn goto_first_nonwhitespace(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
+ let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
- let text = doc.text();
- let line_idx = text.char_to_line(range.head);
+ let selection = doc.selection(view.id).clone().transform(|range| {
+ let line = range.head_line(text);
- if let Some(pos) = find_first_non_whitespace_char(text.line(line_idx)) {
- let pos = pos + text.line_to_char(line_idx);
- Range::new(
- match doc.mode {
- Mode::Normal => pos,
- Mode::Select => range.anchor,
- Mode::Insert => unreachable!(),
- },
- pos,
- )
+ if let Some(pos) = find_first_non_whitespace_char(text.line(line)) {
+ let pos = pos + text.line_to_char(line);
+ range.put(text, pos, doc.mode == Mode::Select)
} else {
range
}
});
-
doc.set_selection(view.id, selection);
}
@@ -491,8 +475,9 @@ fn move_next_word_start(cx: &mut Context) {
let selection = doc
.selection(view.id)
+ .clone()
+ .min_width_1(text)
.transform(|range| movement::move_next_word_start(text, range, count));
-
doc.set_selection(view.id, selection);
}
@@ -503,8 +488,9 @@ fn move_prev_word_start(cx: &mut Context) {
let selection = doc
.selection(view.id)
+ .clone()
+ .min_width_1(text)
.transform(|range| movement::move_prev_word_start(text, range, count));
-
doc.set_selection(view.id, selection);
}
@@ -515,8 +501,9 @@ fn move_next_word_end(cx: &mut Context) {
let selection = doc
.selection(view.id)
+ .clone()
+ .min_width_1(text)
.transform(|range| movement::move_next_word_end(text, range, count));
-
doc.set_selection(view.id, selection);
}
@@ -527,8 +514,9 @@ fn move_next_long_word_start(cx: &mut Context) {
let selection = doc
.selection(view.id)
+ .clone()
+ .min_width_1(text)
.transform(|range| movement::move_next_long_word_start(text, range, count));
-
doc.set_selection(view.id, selection);
}
@@ -539,8 +527,9 @@ fn move_prev_long_word_start(cx: &mut Context) {
let selection = doc
.selection(view.id)
+ .clone()
+ .min_width_1(text)
.transform(|range| movement::move_prev_long_word_start(text, range, count));
-
doc.set_selection(view.id, selection);
}
@@ -551,8 +540,9 @@ fn move_next_long_word_end(cx: &mut Context) {
let selection = doc
.selection(view.id)
+ .clone()
+ .min_width_1(text)
.transform(|range| movement::move_next_long_word_end(text, range, count));
-
doc.set_selection(view.id, selection);
}
@@ -565,9 +555,7 @@ fn goto_file_start(cx: &mut Context) {
fn goto_file_end(cx: &mut Context) {
push_jump(cx.editor);
let (view, doc) = current!(cx.editor);
- let text = doc.text();
- let last_line = text.line_to_char(text.len_lines().saturating_sub(2));
- doc.set_selection(view.id, Selection::point(last_line));
+ doc.set_selection(view.id, Selection::point(doc.text().len_chars()));
}
fn extend_next_word_start(cx: &mut Context) {
@@ -575,12 +563,15 @@ fn extend_next_word_start(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
- let word = movement::move_next_word_start(text, range, count);
- let pos = word.head;
- Range::new(range.anchor, pos)
- });
-
+ let selection = doc
+ .selection(view.id)
+ .clone()
+ .min_width_1(text)
+ .transform(|range| {
+ let word = movement::move_next_word_start(text, range, count);
+ let pos = word.head;
+ range.put(text, pos, true)
+ });
doc.set_selection(view.id, selection);
}
@@ -589,11 +580,15 @@ fn extend_prev_word_start(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
- let word = movement::move_prev_word_start(text, range, count);
- let pos = word.head;
- Range::new(range.anchor, pos)
- });
+ let selection = doc
+ .selection(view.id)
+ .clone()
+ .min_width_1(text)
+ .transform(|range| {
+ let word = movement::move_prev_word_start(text, range, count);
+ let pos = word.head;
+ range.put(text, pos, true)
+ });
doc.set_selection(view.id, selection);
}
@@ -602,12 +597,15 @@ fn extend_next_word_end(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
- let word = movement::move_next_word_end(text, range, count);
- let pos = word.head;
- Range::new(range.anchor, pos)
- });
-
+ let selection = doc
+ .selection(view.id)
+ .clone()
+ .min_width_1(text)
+ .transform(|range| {
+ let word = movement::move_next_word_end(text, range, count);
+ let pos = word.head;
+ range.put(text, pos, true)
+ });
doc.set_selection(view.id, selection);
}
@@ -653,18 +651,10 @@ where
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
- search_fn(text, ch, range.head, count, inclusive).map_or(range, |pos| {
- if extend {
- Range::new(range.anchor, pos)
- } else {
- // select
- Range::new(range.head, pos)
- }
- // or (pos, pos) to move to found val
- })
+ let selection = doc.selection(view.id).clone().transform(|range| {
+ search_fn(text, ch, range.head, count, inclusive)
+ .map_or(range, |pos| range.put(text, pos, extend))
});
-
doc.set_selection(view.id, selection);
})
}
@@ -759,24 +749,30 @@ fn replace(cx: &mut Context) {
_ => None,
};
- if let Some(ch) = ch {
- let transaction =
- Transaction::change_by_selection(doc.text(), doc.selection(view.id), |range| {
- let max_to = rope_end_without_line_ending(&doc.text().slice(..));
- let to = std::cmp::min(max_to, range.to() + 1);
- let text: String = RopeGraphemes::new(doc.text().slice(range.from()..to))
- .map(|g| {
- let cow: Cow<str> = g.into();
- if str_is_line_ending(&cow) {
- cow
- } else {
- ch.into()
- }
- })
- .collect();
+ let text = doc.text().slice(..);
+ let selection = doc.selection(view.id).clone().min_width_1(text);
- (range.from(), to, Some(text.into()))
- });
+ if let Some(ch) = ch {
+ let transaction = Transaction::change_by_selection(doc.text(), &selection, |range| {
+ if !range.is_empty() {
+ let text: String =
+ RopeGraphemes::new(doc.text().slice(range.from()..range.to()))
+ .map(|g| {
+ let cow: Cow<str> = g.into();
+ if str_is_line_ending(&cow) {
+ cow
+ } else {
+ ch.into()
+ }
+ })
+ .collect();
+
+ (range.from(), range.to(), Some(text.into()))
+ } else {
+ // No change.
+ (range.from(), range.to(), None)
+ }
+ });
doc.apply(&transaction, view.id);
doc.append_changes_to_history(view.id);
@@ -786,24 +782,27 @@ fn replace(cx: &mut Context) {
fn switch_case(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
- let transaction =
- Transaction::change_by_selection(doc.text(), doc.selection(view.id), |range| {
- let text: Tendril = range
- .fragment(doc.text().slice(..))
- .chars()
- .flat_map(|ch| {
- if ch.is_lowercase() {
- ch.to_uppercase().collect()
- } else if ch.is_uppercase() {
- ch.to_lowercase().collect()
- } else {
- vec![ch]
- }
- })
- .collect();
+ let selection = doc
+ .selection(view.id)
+ .clone()
+ .min_width_1(doc.text().slice(..));
+ let transaction = Transaction::change_by_selection(doc.text(), &selection, |range| {
+ let text: Tendril = range
+ .fragment(doc.text().slice(..))
+ .chars()
+ .flat_map(|ch| {
+ if ch.is_lowercase() {
+ ch.to_uppercase().collect()
+ } else if ch.is_uppercase() {
+ ch.to_lowercase().collect()
+ } else {
+ vec![ch]
+ }
+ })
+ .collect();
- (range.from(), range.to() + 1, Some(text))
- });
+ (range.from(), range.to(), Some(text))
+ });
doc.apply(&transaction, view.id);
doc.append_changes_to_history(view.id);
@@ -811,12 +810,15 @@ fn switch_case(cx: &mut Context) {
fn switch_to_uppercase(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
- let transaction =
- Transaction::change_by_selection(doc.text(), doc.selection(view.id), |range| {
- let text: Tendril = range.fragment(doc.text().slice(..)).to_uppercase().into();
+ let selection = doc
+ .selection(view.id)
+ .clone()
+ .min_width_1(doc.text().slice(..));
+ let transaction = Transaction::change_by_selection(doc.text(), &selection, |range| {
+ let text: Tendril = range.fragment(doc.text().slice(..)).to_uppercase().into();
- (range.from(), range.to() + 1, Some(text))
- });
+ (range.from(), range.to(), Some(text))
+ });
doc.apply(&transaction, view.id);
doc.append_changes_to_history(view.id);
@@ -824,12 +826,15 @@ fn switch_to_uppercase(cx: &mut Context) {
fn switch_to_lowercase(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
- let transaction =
- Transaction::change_by_selection(doc.text(), doc.selection(view.id), |range| {
- let text: Tendril = range.fragment(doc.text().slice(..)).to_lowercase().into();
+ let selection = doc
+ .selection(view.id)
+ .clone()
+ .min_width_1(doc.text().slice(..));
+ let transaction = Transaction::change_by_selection(doc.text(), &selection, |range| {
+ let text: Tendril = range.fragment(doc.text().slice(..)).to_lowercase().into();
- (range.from(), range.to() + 1, Some(text))
- });
+ (range.from(), range.to(), Some(text))
+ });
doc.apply(&transaction, view.id);
doc.append_changes_to_history(view.id);
@@ -838,7 +843,10 @@ fn switch_to_lowercase(cx: &mut Context) {
fn scroll(cx: &mut Context, offset: usize, direction: Direction) {
use Direction::*;
let (view, doc) = current!(cx.editor);
- let cursor = coords_at_pos(doc.text().slice(..), doc.selection(view.id).cursor());
+ let cursor = coords_at_pos(
+ doc.text().slice(..),
+ doc.selection(view.id).cursor(doc.text().slice(..)),
+ );
let doc_last_line = doc.text().len_lines() - 1;
let last_line = view.last_line(doc);
@@ -867,7 +875,7 @@ fn scroll(cx: &mut Context, offset: usize, direction: Direction) {
.min(last_line.saturating_sub(scrolloff));
let text = doc.text().slice(..);
- let pos = pos_at_coords(text, Position::new(line, cursor.col)); // this func will properly truncate to line end
+ let pos = pos_at_coords(text, Position::new(line, cursor.col), true); // this func will properly truncate to line end
// TODO: only manipulate main selection
doc.set_selection(view.id, Selection::point(pos));
@@ -901,7 +909,8 @@ fn extend_char_left(cx: &mut Context) {
let count = cx.count();
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
+
+ let selection = doc.selection(view.id).clone().transform(|range| {
movement::move_horizontally(text, range, Direction::Backward, count, Movement::Extend)
});
doc.set_selection(view.id, selection);
@@ -911,7 +920,8 @@ fn extend_char_right(cx: &mut Context) {
let count = cx.count();
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
+
+ let selection = doc.selection(view.id).clone().transform(|range| {
movement::move_horizontally(text, range, Direction::Forward, count, Movement::Extend)
});
doc.set_selection(view.id, selection);
@@ -921,7 +931,8 @@ fn extend_line_up(cx: &mut Context) {
let count = cx.count();
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
+
+ let selection = doc.selection(view.id).clone().transform(|range| {
movement::move_vertically(text, range, Direction::Backward, count, Movement::Extend)
});
doc.set_selection(view.id, selection);
@@ -931,7 +942,8 @@ fn extend_line_down(cx: &mut Context) {
let count = cx.count();
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
+
+ let selection = doc.selection(view.id).clone().transform(|range| {
movement::move_vertically(text, range, Direction::Forward, count, Movement::Extend)
});
doc.set_selection(view.id, selection);
@@ -940,7 +952,7 @@ fn extend_line_down(cx: &mut Context) {
fn select_all(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
- let end = rope_end_without_line_ending(&doc.text().slice(..));
+ let end = doc.text().len_chars();
doc.set_selection(view.id, Selection::single(0, end))
}
@@ -980,7 +992,25 @@ fn split_selection_on_newline(cx: &mut Context) {
fn search_impl(doc: &mut Document, view: &mut View, contents: &str, regex: &Regex, extend: bool) {
let text = doc.text();
let selection = doc.selection(view.id);
- let start = text.char_to_byte(selection.cursor());
+ let start = {
+ let range = selection.primary();
+
+ // This is a little bit weird. Due to 1-width cursor semantics, we
+ // would typically want the search to always begin at the visual left-side
+ // of the head. However, when there's already a selection from e.g. a
+ // previous search result, we don't want to include any of that selection
+ // in the subsequent search. The code below makes a compromise between the
+ // two behaviors that hopefully behaves the way most people expect most of
+ // the time.
+ if range.anchor <= range.head {
+ text.char_to_byte(range.head)
+ } else {
+ text.char_to_byte(graphemes::next_grapheme_boundary(
+ text.slice(..),
+ range.head,
+ ))
+ }
+ };
// use find_at to find the next match after the cursor, loop around the end
// Careful, `Regex` uses `bytes` as offsets, not character indices!
@@ -997,12 +1027,10 @@ fn search_impl(doc: &mut Document, view: &mut View, contents: &str, regex: &Rege
return;
}
- let head = end - 1;
-
let selection = if extend {
- selection.clone().push(Range::new(start, head))
+ selection.clone().push(Range::new(start, end))
} else {
- Selection::single(start, head)
+ Selection::single(start, end)
};
doc.set_selection(view.id, selection);
@@ -1065,16 +1093,15 @@ fn extend_line(cx: &mut Context) {
let count = cx.count();
let (view, doc) = current!(cx.editor);
- let pos = doc.selection(view.id).primary();
let text = doc.text();
+ let range = doc.selection(view.id).primary().min_width_1(text.slice(..));
- let line_start = text.char_to_line(pos.anchor);
- let start = text.line_to_char(line_start);
- let line_end = text.char_to_line(pos.head);
- let mut end = line_end_char_index(&text.slice(..), line_end + count.saturating_sub(1));
+ let (start_line, end_line) = range.line_range(text.slice(..));
+ let start = text.line_to_char(start_line);
+ let mut end = text.line_to_char((end_line + count).min(text.len_lines()));
- if pos.anchor == start && pos.head == end && line_end < (text.len_lines() - 2) {
- end = line_end_char_index(&text.slice(..), line_end + 1);
+ if range.from() == start && range.to() == end {
+ end = text.line_to_char((end_line + count + 1).min(text.len_lines()));
}
doc.set_selection(view.id, Selection::single(start, end));
@@ -1083,41 +1110,36 @@ fn extend_line(cx: &mut Context) {
fn extend_to_line_bounds(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
- let text = doc.text();
- let selection = doc.selection(view.id).transform(|range| {
- let start = text.line_to_char(text.char_to_line(range.from()));
- let end = text
- .line_to_char(text.char_to_line(range.to()) + 1)
- .saturating_sub(1);
-
- if range.anchor < range.head {
- Range::new(start, end)
- } else {
- Range::new(end, start)
- }
- });
+ doc.set_selection(
+ view.id,
+ doc.selection(view.id).clone().transform(|range| {
+ let text = doc.text();
- doc.set_selection(view.id, selection);
+ let (start_line, end_line) = range.line_range(text.slice(..));
+ let start = text.line_to_char(start_line);
+ let end = text.line_to_char((end_line + 1).min(text.len_lines()));
+
+ if range.anchor <= range.head {
+ Range::new(start, end)
+ } else {
+ Range::new(end, start)
+ }
+ }),
+ );
}
fn delete_selection_impl(reg: &mut Register, doc: &mut Document, view_id: ViewId) {
- // first yank the selection
- let values: Vec<String> = doc
- .selection(view_id)
- .fragments(doc.text().slice(..))
- .map(Cow::into_owned)
- .collect();
+ let text = doc.text().slice(..);
+ let selection = doc.selection(view_id).clone().min_width_1(text);
+ // first yank the selection
+ let values: Vec<String> = selection.fragments(text).map(Cow::into_owned).collect();
reg.write(values);
// then delete
- let transaction =
- Transaction::change_by_selection(doc.text(), doc.selection(view_id), |range| {
- let alltext = doc.text().slice(..);
- let max_to = rope_end_without_line_ending(&alltext);
- let to = std::cmp::min(max_to, range.to() + 1);
- (range.from(), to, None)
- });
+ let transaction = Transaction::change_by_selection(doc.text(), &selection, |range| {
+ (range.from(), range.to(), None)
+ });
doc.apply(&transaction, view_id);
}
@@ -1145,19 +1167,27 @@ fn change_selection(cx: &mut Context) {
fn collapse_selection(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
- let selection = doc
- .selection(view.id)
- .transform(|range| Range::new(range.head, range.head));
+ let text = doc.text().slice(..);
+ let selection = doc.selection(view.id).clone().transform(|range| {
+ let pos = if range.head > range.anchor {
+ // For 1-width cursor semantics.
+ graphemes::prev_grapheme_boundary(text, range.head)
+ } else {
+ range.head
+ };
+ Range::new(pos, pos)
+ });
doc.set_selection(view.id, selection);
}
fn flip_selections(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
+
let selection = doc
.selection(view.id)
+ .clone()
.transform(|range| Range::new(range.head, range.anchor));
-
doc.set_selection(view.id, selection);
}
@@ -1172,6 +1202,7 @@ fn insert_mode(cx: &mut Context) {
let selection = doc
.selection(view.id)
+ .clone()
.transform(|range| Range::new(range.to(), range.from()));
doc.set_selection(view.id, selection);
}
@@ -1181,18 +1212,15 @@ fn append_mode(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
enter_insert_mode(doc);
doc.restore_cursor = true;
-
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
- Range::new(
- range.from(),
- graphemes::next_grapheme_boundary(text, range.to()), // to() + next char
- )
- });
- let end = text.len_chars();
+ let selection = doc.selection(view.id).clone().min_width_1(text);
- if selection.iter().any(|range| range.head == end) {
+ // Make sure there's room at the end of the document if the last
+ // selection butts up against it.
+ let end = text.len_chars();
+ let last_range = selection.iter().last().unwrap();
+ if !last_range.is_empty() && last_range.head == end {
let transaction = Transaction::change(
doc.text(),
std::array::IntoIter::new([(end, end, Some(doc.line_ending.as_str().into()))]),
@@ -1200,6 +1228,12 @@ fn append_mode(cx: &mut Context) {
doc.apply(&transaction, view.id);
}
+ let selection = selection.transform(|range| {
+ Range::new(
+ range.from(),
+ graphemes::next_grapheme_boundary(doc.text().slice(..), range.to()),
+ )
+ });
doc.set_selection(view.id, selection);
}
@@ -1622,11 +1656,13 @@ mod cmd {
match cx.editor.clipboard_provider.get_contents() {
Ok(contents) => {
+ let selection = doc
+ .selection(view.id)
+ .clone()
+ .min_width_1(doc.text().slice(..));
let transaction =
- Transaction::change_by_selection(doc.text(), doc.selection(view.id), |range| {
- let max_to = rope_end_without_line_ending(&doc.text().slice(..));
- let to = std::cmp::min(max_to, range.to() + 1);
- (range.from(), to, Some(contents.as_str().into()))
+ Transaction::change_by_selection(doc.text(), &selection, |range| {
+ (range.from(), range.to(), Some(contents.as_str().into()))
});
doc.apply(&transaction, view.id);
@@ -2115,10 +2151,10 @@ fn append_to_line(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
enter_insert_mode(doc);
- let selection = doc.selection(view.id).transform(|range| {
- let text = doc.text();
- let line = text.char_to_line(range.head);
- let pos = line_end_char_index(&text.slice(..), line);
+ let selection = doc.selection(view.id).clone().transform(|range| {
+ let text = doc.text().slice(..);
+ let line = range.head_line(text);
+ let pos = line_end_char_index(&text, line);
Range::new(pos, pos)
});
doc.set_selection(view.id, selection);
@@ -2178,7 +2214,7 @@ fn open(cx: &mut Context, open: Open) {
let mut offs = 0;
let mut transaction = Transaction::change_by_selection(contents, selection, |range| {
- let line = text.char_to_line(range.head);
+ let line = range.head_line(text);
let line = match open {
// adjust position to the end of the line (next line - 1)
@@ -2252,7 +2288,7 @@ fn normal_mode(cx: &mut Context) {
// if leaving append mode, move cursor back by 1
if doc.restore_cursor {
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
+ let selection = doc.selection(view.id).clone().transform(|range| {
Range::new(
range.from(),
graphemes::prev_grapheme_boundary(text, range.to()),
@@ -2281,6 +2317,23 @@ fn goto_last_accessed_file(cx: &mut Context) {
}
fn select_mode(cx: &mut Context) {
+ let (view, doc) = current!(cx.editor);
+ let text = doc.text().slice(..);
+
+ // Make sure all selections are at least 1-wide.
+ // (With the exception of being in an empty document, of course.)
+ let selection = doc.selection(view.id).clone().transform(|range| {
+ if range.is_empty() && range.head == text.len_chars() {
+ Range::new(
+ graphemes::prev_grapheme_boundary(text, range.anchor),
+ range.head,
+ )
+ } else {
+ range.min_width_1(text)
+ }
+ });
+ doc.set_selection(view.id, selection);
+
doc_mut!(cx.editor).mode = Mode::Select;
}
@@ -2293,7 +2346,7 @@ fn goto_prehook(cx: &mut Context) -> bool {
push_jump(cx.editor);
let (view, doc) = current!(cx.editor);
- let line_idx = std::cmp::min(count.get() - 1, doc.text().len_lines().saturating_sub(2));
+ let line_idx = std::cmp::min(count.get() - 1, doc.text().len_lines().saturating_sub(1));
let pos = doc.text().line_to_char(line_idx);
doc.set_selection(view.id, Selection::point(pos));
true
@@ -2367,7 +2420,11 @@ fn goto_definition(cx: &mut Context) {
let offset_encoding = language_server.offset_encoding();
- let pos = pos_to_lsp_pos(doc.text(), doc.selection(view.id).cursor(), offset_encoding);
+ let pos = pos_to_lsp_pos(
+ doc.text(),
+ doc.selection(view.id).cursor(doc.text().slice(..)),
+ offset_encoding,
+ );
// TODO: handle fails
let future = language_server.goto_definition(doc.identifier(), pos, None);
@@ -2404,7 +2461,11 @@ fn goto_type_definition(cx: &mut Context) {
let offset_encoding = language_server.offset_encoding();
- let pos = pos_to_lsp_pos(doc.text(), doc.selection(view.id).cursor(), offset_encoding);
+ let pos = pos_to_lsp_pos(
+ doc.text(),
+ doc.selection(view.id).cursor(doc.text().slice(..)),
+ offset_encoding,
+ );
// TODO: handle fails
let future = language_server.goto_type_definition(doc.identifier(), pos, None);
@@ -2441,7 +2502,11 @@ fn goto_implementation(cx: &mut Context) {
let offset_encoding = language_server.offset_encoding();
- let pos = pos_to_lsp_pos(doc.text(), doc.selection(view.id).cursor(), offset_encoding);
+ let pos = pos_to_lsp_pos(
+ doc.text(),
+ doc.selection(view.id).cursor(doc.text().slice(..)),
+ offset_encoding,
+ );
// TODO: handle fails
let future = language_server.goto_implementation(doc.identifier(), pos, None);
@@ -2478,7 +2543,11 @@ fn goto_reference(cx: &mut Context) {
let offset_encoding = language_server.offset_encoding();
- let pos = pos_to_lsp_pos(doc.text(), doc.selection(view.id).cursor(), offset_encoding);
+ let pos = pos_to_lsp_pos(
+ doc.text(),
+ doc.selection(view.id).cursor(doc.text().slice(..)),
+ offset_encoding,
+ );
// TODO: handle fails
let future = language_server.goto_reference(doc.identifier(), pos, None);
@@ -2537,7 +2606,7 @@ fn goto_next_diag(cx: &mut Context) {
let editor = &mut cx.editor;
let (view, doc) = current!(editor);
- let cursor_pos = doc.selection(view.id).cursor();
+ let cursor_pos = doc.selection(view.id).cursor(doc.text().slice(..));
let diag = if let Some(diag) = doc
.diagnostics()
.iter()
@@ -2558,7 +2627,7 @@ fn goto_prev_diag(cx: &mut Context) {
let editor = &mut cx.editor;
let (view, doc) = current!(editor);
- let cursor_pos = doc.selection(view.id).cursor();
+ let cursor_pos = doc.selection(view.id).cursor(doc.text().slice(..));
let diag = if let Some(diag) = doc
.diagnostics()
.iter()
@@ -2586,7 +2655,7 @@ fn signature_help(cx: &mut Context) {
let pos = pos_to_lsp_pos(
doc.text(),
- doc.selection(view.id).cursor(),
+ doc.selection(view.id).cursor(doc.text().slice(..)),
language_server.offset_encoding(),
);
@@ -2709,11 +2778,11 @@ pub mod insert {
let (view, doc) = current!(cx.editor);
let text = doc.text();
- let selection = doc.selection(view.id);
+ let selection = doc.selection(view.id).clone().cursors(text.slice(..));
// run through insert hooks, stopping on the first one that returns Some(t)
for hook in HOOKS {
- if let Some(transaction) = hook(text, selection, c) {
+ if let Some(transaction) = hook(text, &selection, c) {
doc.apply(&transaction, view.id);
break;
}
@@ -2733,7 +2802,11 @@ pub mod insert {
// indent by one to reach 4 spaces).
let indent = Tendril::from(doc.indent_unit());
- let transaction = Transaction::insert(doc.text(), doc.selection(view.id), indent);
+ let transaction = Transaction::insert(
+ doc.text(),
+ &doc.selection(view.id).clone().cursors(doc.text().slice(..)),
+ indent,
+ );
doc.apply(&transaction, view.id);
}
@@ -2742,13 +2815,13 @@ pub mod insert {
let text = doc.text().slice(..);
let contents = doc.text();
- let selection = doc.selection(view.id);
+ let selection = doc.selection(view.id).clone().cursors(text);
let mut ranges = SmallVec::with_capacity(selection.len());
// TODO: this is annoying, but we need to do it to properly calculate pos after edits
let mut offs = 0;
- let mut transaction = Transaction::change_by_selection(contents, selection, |range| {
+ let mut transaction = Transaction::change_by_selection(contents, &selection, |range| {
let pos = range.head;
let prev = if pos == 0 {
@@ -2839,8 +2912,10 @@ pub mod insert {
let count = cx.count();
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
+
let selection = doc
.selection(view.id)
+ .clone()
.transform(|range| movement::move_prev_word_start(text, range, count));
doc.set_selection(view.id, selection);
delete_selection(cx)
@@ -2868,9 +2943,13 @@ fn redo(cx: &mut Context) {
fn yank(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
+ let text = doc.text().slice(..);
+
let values: Vec<String> = doc
.selection(view.id)
- .fragments(doc.text().slice(..))
+ .clone()
+ .min_width_1(text)
+ .fragments(text)
.map(Cow::into_owned)
.collect();
@@ -2889,10 +2968,13 @@ fn yank(cx: &mut Context) {
fn yank_joined_to_clipboard_impl(editor: &mut Editor, separator: &str) -> anyhow::Result<()> {
let (view, doc) = current!(editor);
+ let text = doc.text().slice(..);
let values: Vec<String> = doc
.selection(view.id)
- .fragments(doc.text().slice(..))
+ .clone()
+ .min_width_1(text)
+ .fragments(text)
.map(Cow::into_owned)
.collect();
@@ -2920,11 +3002,13 @@ fn yank_joined_to_clipboard(cx: &mut Context) {
fn yank_main_selection_to_clipboard_impl(editor: &mut Editor) -> anyhow::Result<()> {
let (view, doc) = current!(editor);
+ let text = doc.text().slice(..);
let value = doc
.selection(view.id)
.primary()
- .fragment(doc.text().slice(..));
+ .min_width_1(text)
+ .fragment(text);
if let Err(e) = editor.clipboard_provider.set_contents(value.into_owned()) {
bail!("Couldn't set system clipboard content: {:?}", e);
@@ -2965,17 +3049,21 @@ fn paste_impl(
let mut values = values.iter().cloned().map(Tendril::from).chain(repeat);
let text = doc.text();
+ let selection = doc.selection(view.id).clone().min_width_1(text.slice(..));
- let transaction = Transaction::change_by_selection(text, doc.selection(view.id), |range| {
+ let transaction = Transaction::change_by_selection(text, &selection, |range| {
let pos = match (action, linewise) {
// paste linewise before
(Paste::Before, true) => text.line_to_char(text.char_to_line(range.from())),
// paste linewise after
- (Paste::After, true) => text.line_to_char(text.char_to_line(range.to()) + 1),
+ (Paste::After, true) => {
+ let line = range.line_range(text.slice(..)).1;
+ text.line_to_char((line + 1).min(text.len_lines()))
+ }
// paste insert
(Paste::Before, false) => range.from(),
// paste append
- (Paste::After, false) => range.to() + 1,
+ (Paste::After, false) => range.to(),
};
(pos, pos, Some(values.next().unwrap()))
});
@@ -3016,12 +3104,17 @@ fn replace_with_yanked(cx: &mut Context) {
if let Some(values) = registers.read(reg_name) {
if let Some(yank) = values.first() {
- let transaction =
- Transaction::change_by_selection(doc.text(), doc.selection(view.id), |range| {
- let max_to = rope_end_without_line_ending(&doc.text().slice(..));
- let to = std::cmp::min(max_to, range.to() + 1);
- (range.from(), to, Some(yank.as_str().into()))
- });
+ let selection = doc
+ .selection(view.id)
+ .clone()
+ .min_width_1(doc.text().slice(..));
+ let transaction = Transaction::change_by_selection(doc.text(), &selection, |range| {
+ if !range.is_empty() {
+ (range.from(), range.to(), Some(yank.as_str().into()))
+ } else {
+ (range.from(), range.to(), None)
+ }
+ });
doc.apply(&transaction, view.id);
doc.append_changes_to_history(view.id);
@@ -3034,12 +3127,13 @@ fn replace_selections_with_clipboard_impl(editor: &mut Editor) -> anyhow::Result
match editor.clipboard_provider.get_contents() {
Ok(contents) => {
- let transaction =
- Transaction::change_by_selection(doc.text(), doc.selection(view.id), |range| {
- let max_to = rope_end_without_line_ending(&doc.text().slice(..));
- let to = std::cmp::min(max_to, range.to() + 1);
- (range.from(), to, Some(contents.as_str().into()))
- });
+ let selection = doc
+ .selection(view.id)
+ .clone()
+ .min_width_1(doc.text().slice(..));
+ let transaction = Transaction::change_by_selection(doc.text(), &selection, |range| {
+ (range.from(), range.to(), Some(contents.as_str().into()))
+ });
doc.apply(&transaction, view.id);
doc.append_changes_to_history(view.id);
@@ -3086,8 +3180,7 @@ fn get_lines(doc: &Document, view_id: ViewId) -> Vec<usize> {
// Get all line numbers
for range in doc.selection(view_id) {
- let start = doc.text().char_to_line(range.from());
- let end = doc.text().char_to_line(range.to());
+ let (start, end) = range.line_range(doc.text().slice(..));
for line in start..=end {
lines.push(line)
@@ -3215,10 +3308,9 @@ fn join_selections(cx: &mut Context) {
let fragment = Tendril::from(" ");
for selection in doc.selection(view.id) {
- let start = text.char_to_line(selection.from());
- let mut end = text.char_to_line(selection.to());
+ let (start, mut end) = selection.line_range(slice);
if start == end {
- end += 1
+ end = (end + 1).min(text.len_lines() - 1);
}
let lines = start..end;
@@ -3315,13 +3407,14 @@ fn completion(cx: &mut Context) {
};
let offset_encoding = language_server.offset_encoding();
+ let cursor = doc.selection(view.id).cursor(doc.text().slice(..));
- let pos = pos_to_lsp_pos(doc.text(), doc.selection(view.id).cursor(), offset_encoding);
+ let pos = pos_to_lsp_pos(doc.text(), cursor, offset_encoding);
// TODO: handle fails
let future = language_server.completion(doc.identifier(), pos, None);
- let trigger_offset = doc.selection(view.id).cursor();
+ let trigger_offset = cursor;
cx.callback(
future,
@@ -3371,7 +3464,7 @@ fn hover(cx: &mut Context) {
let pos = pos_to_lsp_pos(
doc.text(),
- doc.selection(view.id).cursor(),
+ doc.selection(view.id).cursor(doc.text().slice(..)),
language_server.offset_encoding(),
);
@@ -3437,7 +3530,7 @@ fn match_brackets(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
if let Some(syntax) = doc.syntax() {
- let pos = doc.selection(view.id).cursor();
+ let pos = doc.selection(view.id).cursor(doc.text().slice(..));
if let Some(pos) = match_brackets::find(syntax, doc.text(), pos) {
let selection = Selection::point(pos);
doc.set_selection(view.id, selection);
@@ -3541,7 +3634,7 @@ fn align_view_bottom(cx: &mut Context) {
fn align_view_middle(cx: &mut Context) {
let (view, doc) = current!(cx.editor);
- let pos = doc.selection(view.id).cursor();
+ let pos = doc.selection(view.id).cursor(doc.text().slice(..));
let pos = coords_at_pos(doc.text().slice(..), pos);
const OFFSET: usize = 7; // gutters
@@ -3577,7 +3670,7 @@ fn select_textobject(cx: &mut Context, objtype: textobject::TextObject) {
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id).transform(|range| {
+ let selection = doc.selection(view.id).clone().transform(|range| {
match ch {
'w' => textobject::textobject_word(text, range, objtype, count),
// TODO: cancel new ranges if inconsistent surround matches across lines
@@ -3587,7 +3680,6 @@ fn select_textobject(cx: &mut Context, objtype: textobject::TextObject) {
_ => range,
}
});
-
doc.set_selection(view.id, selection);
}
})
@@ -3602,17 +3694,13 @@ fn surround_add(cx: &mut Context) {
{
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id);
+ let selection = doc.selection(view.id).clone().min_width_1(text);
let (open, close) = surround::get_pair(ch);
let mut changes = Vec::new();
for range in selection.iter() {
- let from = range.from();
- let max_to = rope_end_without_line_ending(&text);
- let to = std::cmp::min(range.to() + 1, max_to);
-
- changes.push((from, from, Some(Tendril::from_char(open))));
- changes.push((to, to, Some(Tendril::from_char(close))));
+ changes.push((range.from(), range.from(), Some(Tendril::from_char(open))));
+ changes.push((range.to(), range.to(), Some(Tendril::from_char(close))));
}
let transaction = Transaction::change(doc.text(), changes.into_iter());
@@ -3638,9 +3726,9 @@ fn surround_replace(cx: &mut Context) {
{
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id);
+ let selection = doc.selection(view.id).clone().min_width_1(text);
- let change_pos = match surround::get_surround_pos(text, selection, from, count)
+ let change_pos = match surround::get_surround_pos(text, &selection, from, count)
{
Some(c) => c,
None => return,
@@ -3672,9 +3760,9 @@ fn surround_delete(cx: &mut Context) {
{
let (view, doc) = current!(cx.editor);
let text = doc.text().slice(..);
- let selection = doc.selection(view.id);
+ let selection = doc.selection(view.id).clone().min_width_1(text);
- let change_pos = match surround::get_surround_pos(text, selection, ch, count) {
+ let change_pos = match surround::get_surround_pos(text, &selection, ch, count) {
Some(c) => c,
None => return,
};