aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChrHorn2022-11-17 01:16:03 +0000
committerGitHub2022-11-17 01:16:03 +0000
commit322e957ea1947108d10699f217b75b831df2a2c0 (patch)
tree1db5a266799843d1c4b5668ed47a0e9b6d42a40f
parent7483c762229516298c4a2d9398b9418dbc57198b (diff)
Remove prefix filtering from autocomplete menu (#4578)
PR #4134 switched the autocomplete menu from alphabetical to fuzzy sorting. This commit removes the still existing filtering by prefix and should enable full fuzzy sorting of the autocomplete menu. closes #3084, #1807 Co-authored-by: Blaž Hrastnik <blaz@mxxn.io>
-rw-r--r--helix-term/src/commands.rs12
1 files changed, 1 insertions, 11 deletions
diff --git a/helix-term/src/commands.rs b/helix-term/src/commands.rs
index 2f2427ec..4e3f0767 100644
--- a/helix-term/src/commands.rs
+++ b/helix-term/src/commands.rs
@@ -3950,7 +3950,6 @@ pub fn completion(cx: &mut Context) {
iter.reverse();
let offset = iter.take_while(|ch| chars::char_is_word(*ch)).count();
let start_offset = cursor.saturating_sub(offset);
- let prefix = text.slice(start_offset..cursor).to_string();
cx.callback(
future,
@@ -3960,7 +3959,7 @@ pub fn completion(cx: &mut Context) {
return;
}
- let mut items = match response {
+ let items = match response {
Some(lsp::CompletionResponse::Array(items)) => items,
// TODO: do something with is_incomplete
Some(lsp::CompletionResponse::List(lsp::CompletionList {
@@ -3970,15 +3969,6 @@ pub fn completion(cx: &mut Context) {
None => Vec::new(),
};
- if !prefix.is_empty() {
- items.retain(|item| {
- item.filter_text
- .as_ref()
- .unwrap_or(&item.label)
- .starts_with(&prefix)
- });
- }
-
if items.is_empty() {
// editor.set_error("No completion available");
return;