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

Better inline preview for postfix completion #17073

Merged
merged 2 commits into from
Apr 15, 2024

Conversation

roife
Copy link
Member

@roife roife commented Apr 15, 2024

Better inline preview for postfix completion, a proper implementation of c5686c8.

Here editors may filter completion item with the text within delete_range, so we need to include the receiver text in the lookup (aka FilterText in LSP spec) for editors to find the completion item. (See #17036 (comment), Thanks to pascalkuthe)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 15, 2024
@Veykril
Copy link
Member

Veykril commented Apr 15, 2024

Thanks!
@bors r+

@bors
Copy link
Contributor

bors commented Apr 15, 2024

📌 Commit 8db1c64 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Apr 15, 2024

⌛ Testing commit 8db1c64 with merge af72874...

@bors
Copy link
Contributor

bors commented Apr 15, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing af72874 to master...

@bors bors merged commit af72874 into rust-lang:master Apr 15, 2024
11 checks passed
@davidbarsky
Copy link
Contributor

Just as a heads up, this PR doesn't work too well with VS Code. On c037130, these are the completions I'm getting:

Screenshot 2024-04-15 at 11 23 34 AM

With af72874, I'm getting the postfix completions first:

Screenshot 2024-04-15 at 11 25 57 AM

I'm not sure how intentional that is, but it certainly feels a bit like a bug to me.

davidbarsky added a commit to davidbarsky/rust-analyzer that referenced this pull request Apr 15, 2024
bors added a commit that referenced this pull request Apr 16, 2024
Revert #17073: Better inline preview for postfix completion

See discussion on #17077, but I strongly suspect that the changes to the `TextEdit` ranges caused VS Code's autocomplete to prefer the snippets over method completions. I explain why I think that [here](#17077 (comment)).
lnicola pushed a commit to lnicola/rust-analyzer that referenced this pull request Apr 17, 2024
…kril

Revert rust-lang#17073: Better inline preview for postfix completion

See discussion on rust-lang#17077, but I strongly suspect that the changes to the `TextEdit` ranges caused VS Code's autocomplete to prefer the snippets over method completions. I explain why I think that [here](rust-lang#17077 (comment)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants