-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Expandable hover #61492
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
Merged
Merged
Expandable hover #61492
Changes from 25 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
91af0ce
Reapply "Proposed expandable hover API" (#61132)
gabritto 92c49e6
WIP: support more cases
gabritto e371469
Fixes
gabritto 5686f68
use declared symbol for class
gabritto bc99b1f
don't expand when truncating; fix tuple
gabritto 44770e7
lint; test
gabritto 06c5711
fix type parameter printing, fix lib type at top level
gabritto e97074b
fix top-level enum expansion
gabritto 8aa7313
expand enum as type
gabritto d4958f6
print top-level classes as declarations
gabritto a7a334f
add class and enum declaration printing
gabritto 45a92a4
add one more case to import
gabritto 4256003
finish impl of top-level declaration expansion
gabritto 25673e9
more fixes + tests
gabritto f9d2309
add top-level truncation
gabritto 794e70b
add basic JS tests
gabritto d3b2a84
format
gabritto 801b84c
Merge branch 'main' into gabritto/hover
gabritto fb7762c
tiny change unmade
gabritto 8bd4821
default to printing 'namespace' instead of 'module'
gabritto 2511624
update protocol comments
gabritto 14d8405
Revert "default to printing 'namespace' instead of 'module'"
gabritto 0def080
fix expansion of namespaces with non-id names
gabritto 0c3edf9
format
gabritto 7562951
update displayparts baselines
gabritto a187291
use enum member initializer if present
gabritto 6af0483
truncation with comment when necessary
gabritto 435418a
comments
gabritto 7fe7ec2
refactor: move things around
gabritto defed5d
renaming, more comments
gabritto 9e2feca
address some CR comments
gabritto 98772e1
deep clone enum initializer when reused for quickinfo expansion
gabritto 6e9f1d7
Merge branch 'main' into gabritto/hover
gabritto File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
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
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
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
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
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.