andrey/helix-mirror
1
0
Fork 0
mirror of https://github.com/helix-editor/helix.git synced 2024-11-22 09:26:19 +04:00
Code Issues Projects Releases Wiki Activity

feat(lsp): LSP preselected items appear first in completion menu (#4480)

Browse Source 
* feat(lsp): LSP preselected items appear first in completion menu

* fix: shorter diff
This commit is contained in:
lazytanuki 2022-10-27 16:16:55 +02:00 committed by GitHub
parent b1ffbbd49f
commit b4a3dd8f89
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
helix-term/src/ui/completion.rs
View File

@ -92,11 +92,15 @@ impl Completion {
pub fn new(
editor: &Editor,
items: Vec<CompletionItem>,
mut items: Vec<CompletionItem>,
offset_encoding: helix_lsp::OffsetEncoding,
start_offset: usize,
trigger_offset: usize,
) -> Self {
// Sort completion items according to their preselect status (given by the LSP server)
items.sort_by_key(|item| !item.preselect.unwrap_or(false));
// Then create the menu
let menu = Menu::new(items, (), move |editor: &mut Editor, item, event| {
fn item_to_transaction(
doc: &Document,

3
helix-term/src/ui/menu.rs
View File

@ -108,7 +108,8 @@ pub fn score(&mut self, pattern: &str) {
.map(|score| (index, score))
}),
);
self.matches.sort_unstable_by_key(|(_, score)| -score);
// Order of equal elements needs to be preserved as LSP preselected items come in order of high to low priority
self.matches.sort_by_key(|(_, score)| -score);
// reset cursor position
self.cursor = None;