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 show in inline suggestions even though they are turned off #175190

Closed
TheColorRed opened this issue Feb 23, 2023 · 13 comments · Fixed by #203418
Closed

Snippets show in inline suggestions even though they are turned off #175190

TheColorRed opened this issue Feb 23, 2023 · 13 comments · Fixed by #203418
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug suggest IntelliSense, Auto Complete verified Verification succeeded

Comments

@TheColorRed
Copy link

TheColorRed commented Feb 23, 2023

Does this issue occur when all extensions are disabled?: Yes

  • VS Code Version: Windows 11
  • OS Version: 1.75.1

Steps to Reproduce:

Using the following settings:
image
image

  1. open a typescript file
  2. type fo within a method/function
  3. the following gets output (might have to navigate through some of the suggestions) even though snippets are off:

image

@jrieken jrieken added the suggest IntelliSense, Auto Complete label Feb 23, 2023
@jrieken jrieken added the info-needed Issue requires more information from poster label Dec 5, 2023
@jrieken
Copy link
Member

jrieken commented Dec 5, 2023

Thanks for the patience. I cannot reproduce, please share more of your settings that might influence this. Also share data in a copy-paste friendly way (text over images)

@TheColorRed
Copy link
Author

TheColorRed commented Dec 7, 2023

@jrieken Here are the settings that I am using to reproduce (all extensions disabled code --disable-extensions):

{
    "editor.inlineSuggest.enabled": true,
    "editor.snippetSuggestions": "none",
    "editor.quickSuggestions": {
        "other": "inline",
        "comments": "inline",
        "strings": "inline"
    },
    "editor.inlineSuggest.showToolbar": "always"
}

Once typing fo and going to the 3rd suggestion, I get this as a suggestion:

image

@jrieken jrieken added bug Issue identified by VS Code Team member as probable bug and removed info-needed Issue requires more information from poster labels Dec 7, 2023
@jrieken
Copy link
Member

jrieken commented Dec 7, 2023

Thanks for clarifying, looks like we aren't passing the filter here:

new CompletionOptions(undefined, undefined, triggerCharacterInfo?.providers),

jrieken added a commit that referenced this issue Dec 14, 2023
@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label Dec 14, 2023
@VSCodeTriageBot VSCodeTriageBot added this to the December / January 2024 milestone Dec 14, 2023
@VSCodeTriageBot VSCodeTriageBot added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Dec 15, 2023
@amunger amunger added verified Verification succeeded verification-steps-needed Steps to verify are needed for verification and removed verified Verification succeeded labels Jan 24, 2024
@amunger
Copy link
Contributor

amunger commented Jan 24, 2024

I can't figure out how to get that ghost text to show up for suggestions to verify this. I've never seen that ghost text for anything but copilot completions
Do I need a suggestion provider extension?

@jrieken
Copy link
Member

jrieken commented Jan 25, 2024

You are right. I made a refactoring after this fix and that broke this feature 😨 Thanks for the diligence on this one!

@jrieken jrieken reopened this Jan 25, 2024
@VSCodeTriageBot VSCodeTriageBot removed the insiders-released Patch has been released in VS Code Insiders label Jan 25, 2024
jrieken added a commit that referenced this issue Jan 25, 2024
jrieken added a commit that referenced this issue Jan 25, 2024
* restore inline suggestion

fixes #175190

* fix leak in test
@VSCodeTriageBot VSCodeTriageBot added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Jan 25, 2024
@jrieken jrieken removed the verification-steps-needed Steps to verify are needed for verification label Jan 26, 2024
@jrieken
Copy link
Member

jrieken commented Jan 26, 2024

To verify, use the steps from #175190 (comment). Make sure no snippets completions are ghost-texted with "editor.snippetSuggestions": "none" and that they are when the setting is set accordingly

@meganrogge meganrogge added verified Verification succeeded and removed verified Verification succeeded labels Jan 26, 2024
@meganrogge
Copy link
Contributor

I still cannot get snippet completions to show up @jrieken

@meganrogge meganrogge reopened this Jan 26, 2024
@VSCodeTriageBot VSCodeTriageBot removed the insiders-released Patch has been released in VS Code Insiders label Jan 26, 2024
@jrieken
Copy link
Member

jrieken commented Jan 26, 2024

@meganrogge You need to say what you are trying because I do see them working just fine

Screenshot 2024-01-26 at 18 14 32

@amunger
Copy link
Contributor

amunger commented Jan 26, 2024

This works correctly for me now, no snippets with "editor.snippetSuggestions": "none",, and they do show up without it.

@jrieken
Copy link
Member

jrieken commented Jan 26, 2024

Bang on. The whole point is that the editor.snippetSuggestions-setting is honored also for ghost suggestions

@jrieken jrieken closed this as completed Jan 26, 2024
@meganrogge
Copy link
Contributor

editor.snippetSuggestions:inline

Screenshot 2024-01-26 at 9 19 47 AM

@jrieken
Copy link
Member

jrieken commented Jan 26, 2024

@meganrogge this is about ghost text, please see #175190 (comment) and the comment linked from there

@amunger amunger added the verified Verification succeeded label Jan 26, 2024
@meganrogge
Copy link
Contributor

🤦🏼‍♀️ there was a syntax error in my settings.json file, so the settings did not get applied. Now it works 👍🏼

@aiday-mar aiday-mar added this to the December / January 2024 milestone Feb 6, 2024
@microsoft microsoft locked and limited conversation to collaborators Jun 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug suggest IntelliSense, Auto Complete verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants