Skip to content

Custom tabexpansion tooltip shows in ISE but not VS Code #2405

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

Closed
dfinke opened this issue Jan 8, 2020 · 5 comments · Fixed by PowerShell/PowerShellEditorServices#1152
Closed

Comments

@dfinke
Copy link
Contributor

dfinke commented Jan 8, 2020

ISE

image

VS Code

image

Script

function MyTestCompletion {
	param($wordToComplete, $commandAst)

	$commandTree = & {
		    New-CommandTree build "Compile the current package" {
			    New-CommandTree "--quiet" "output printed to stdout"
			    New-CommandTree "--package" "Package to build (see `cargo help pkgid`)"
			    New-CommandTree "--all" "for --workspace (deprecated)"
			    New-CommandTree "-workspace" "all packages in the workspace"
			    New-CommandTree "-exclude" "Exclude packages from the build"
	    }
    }

	Get-CommandTreeCompletion $wordToComplete $commandAst $commandTree
}

Register-ArgumentCompleter -CommandName mytest -Native -ScriptBlock $function:MyTestCompletion
@ghost ghost added the Needs: Triage Maintainer attention needed! label Jan 8, 2020
@SydneyhSmith
Copy link
Collaborator

@dfinke thanks for opening this issue, and providing the screenshots...it would be really helpful if you could also attach the logs so that we can see where the message is getting lost--instructions for how to do that are here

@dfinke
Copy link
Contributor Author

dfinke commented Jan 10, 2020

Here's the log, let me know if I need to provide other info.

vscode.log

@TylerLeonhardt
Copy link
Member

I've got a fix out for this. ^

@dfinke
Copy link
Contributor Author

dfinke commented Jan 18, 2020

What version of the ext is this expected in? Tried in the latest, doesn't seem to work.

@ghost ghost added the Needs: Maintainer Attention Maintainer attention needed! label Jan 18, 2020
@TylerLeonhardt
Copy link
Member

The next version.

@TylerLeonhardt TylerLeonhardt removed the Needs: Maintainer Attention Maintainer attention needed! label Jan 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants