Additional IntelliSense fixes and ToolTip overhaul #1809
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Now shows word to complete and match count instead of the literal type name for `SMA.CommandCompletion`.
Type completion when utilizing `using namespace` statements was really hit or miss due to replacement text not matching the filter text. Also add support for completing and retaining `$PSScriptRoot`. Requires a change in PowerShell that is not yet merged to work, but does not break without the changes needed.
andyleejordan
approved these changes
May 20, 2022
Member
andyleejordan
left a comment
There was a problem hiding this comment.
Reviewed via Teams call!
|
@SeeminglyScience OMG This is so wonderful! Especially the methods view! Thank you! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some minorish changes
SMA.CommandCompletion).using namespacestatements$PSScriptRootand retaining it in the result (requires Use AST extent forPSScriptRootpath completion PowerShell#17376)Overhaul completion tooltips for members
Types before
Types after
Properties before
Properties after
Methods before 😬
Methods after 🎉