This repository has been archived by the owner on Jan 25, 2024. It is now read-only.

fix completion for empty prefix #104

Closed

milahu wants to merge 1 commit into nix-community:master from milahu:fix-completion-empty-prefix

Contributor

milahu commented Oct 9, 2022 •

edited

Loading

fix #103

before:
with empty prefix, the TextEdit range was wrong

mostly copy-paste of the let mut completions = Vec::new(); branch, except

                     text_edit: Some(CompletionTextEdit::Edit(TextEdit {
-                        range: utils::range(content, node.node().text_range()),
+                        range: Range {
+                            start: params.position,
+                            end: params.position,
+                        },
                         new_text: var.clone(),
                     })),

maybe refactor the 2 branches?


          fix completion for empty prefix

037c39e

Member

aaronjanse commented Mar 9, 2023

Closed upon request of PR author

aaronjanse closed this

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet