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

User completion in commit box is wrong when manually triggered #4026

Closed
joaomoreno opened this issue Oct 7, 2022 · 8 comments · Fixed by #4101
Closed

User completion in commit box is wrong when manually triggered #4026

joaomoreno opened this issue Oct 7, 2022 · 8 comments · Fixed by #4101
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@joaomoreno
Copy link
Member

When typing @, I get suggestions for users, but once I typed a few characters pressing Ctrl D will show me issue suggestions.

4eeea7ac-c585-4a7f-985d-f34366efa2c2

@alexr00
Copy link
Member

alexr00 commented Oct 10, 2022

I think if you scroll down in that issues list you'll see the users below the issues. Since it wasn't triggered by a trigger character we show both.

@alexr00 alexr00 added the info-needed Issue requires more information from poster label Oct 10, 2022
@joaomoreno
Copy link
Member Author

Correct, they are all there. But why is this useful behavior? In other words, why do all those suggestions appear as possibilities for the @sand input? Scrolling down, not even @sandy081 is highlighted in that case:

image

@joaomoreno
Copy link
Member Author

joaomoreno commented Oct 11, 2022

The bug seems to be that the filtering only happens from the moment the widget pops up (either automatically or manually). Notice this example:

Recording 2022-10-11 at 11 17 22

The widget only filters on the moreno sequence that I type after manually opening it and even fails to properly render highlights on user names.

The expected behavior is: when manually opening it, it should be filtered on my initial input (joao) and should further filter down as I type the rest (moreno). This is exactly how it works in the editor:

Recording 2022-10-11 at 11 20 20

@alexr00 alexr00 self-assigned this Oct 11, 2022
@alexr00 alexr00 added bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels Oct 11, 2022
@alexr00 alexr00 added this to the October 2022 milestone Oct 11, 2022
@alexr00 alexr00 added feature-request Request for new features or functionality and removed bug Issue identified by VS Code Team member as probable bug labels Oct 21, 2022
@alexr00
Copy link
Member

alexr00 commented Oct 21, 2022

This occurs for all suggests, not just in the commit box.

@alexr00
Copy link
Member

alexr00 commented Oct 21, 2022

@jrieken I think CompletionProvider would need to add some support for this. Is there a way for an extension to tell the suggest widget what text before the cursor should be used for filtering?

@alexr00 alexr00 modified the milestones: October 2022, On Deck Oct 21, 2022
@jrieken
Copy link
Member

jrieken commented Oct 24, 2022

Is there a way for an extension to tell the suggest widget what text before the cursor should be used for filtering?

Yes, it the CompletionItem#range property.

@alexr00 alexr00 modified the milestones: On Deck, October 2022 Oct 24, 2022
@alexr00 alexr00 added bug Issue identified by VS Code Team member as probable bug and removed feature-request Request for new features or functionality labels Oct 24, 2022
@joyceerhl joyceerhl added the verified Verification succeeded label Oct 27, 2022
@joyceerhl
Copy link
Contributor

In a codespace and desktop RemoteHub I get user suggestions in the commit box, but I can't trigger suggestions automatically or manually in web at all.

@alexr00
Copy link
Member

alexr00 commented Oct 27, 2022

Opened #4119 to fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants