Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Snippets in assists don't work anymore #8208

Closed
jonas-schievink opened this issue Mar 26, 2021 · 4 comments
Closed

Snippets in assists don't work anymore #8208

jonas-schievink opened this issue Mar 26, 2021 · 4 comments
Labels
A-assists A-vscode vscode plugin issues

Comments

@jonas-schievink
Copy link
Contributor

We have a custom LSP extension for snippets in assists, it doesn't seem to work anymore:

fn f() {
    $0()$0
}

Apply "Extract variable"

fn f() {
    let $0var_name = ();
    var_name
}

...but there's no snippet

@jonas-schievink jonas-schievink added A-vscode vscode plugin issues A-assists labels Mar 26, 2021
@ivan770
Copy link
Contributor

ivan770 commented Mar 27, 2021

Is this reproducible in editors other than VS Code? Trace logs show that we send snippet insert text format

@ivan770
Copy link
Contributor

ivan770 commented Mar 27, 2021

Got it narrowed down to placeholderLength always being at zero, thus putting cursor at the beginning of placeholder but without selecting it

@jonas-schievink
Copy link
Contributor Author

It seems that this was never supported like I thought it was, but I've fixed some bugs around this in #8210

@jonas-schievink
Copy link
Contributor Author

Closing, since the known issues were fixed in #8210, even though that didn't end up using this feature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-assists A-vscode vscode plugin issues
Projects
None yet
Development

No branches or pull requests

2 participants