Added an optional uri property to the DocumentSymbol.#425
Added an optional uri property to the DocumentSymbol.#425kittaakos wants to merge 1 commit intomicrosoft:masterfrom
uri property to the DocumentSymbol.#425Conversation
Clients can use this when the URI of the text document cannot be inferred from the request context. Signed-off-by: Akos Kitta <[email protected]>
|
I understand why you are requesting this (because of the type hierarchy request) but IMO this is not the right way to go. DocumentSymbols are as the name implies scoped to a document and we shouldn't start changing this. I would rather opt to have a new type Any objection to close this ? |
Yes :)
This was the initial idea.
I do not mind adopting the proposals. What's your take on that, @svenefftinge? |
IMO the name |
It should be still possible with the Presumably, you are talking about the Ctrl/Cmd+o functionality in Eclipse. It is your outline: If you want the inherited members (Ctrl/Cmd+o and Ctrl/Cmd+o again), you can get the supertypes of the current document: |
Deciding on whether a symbol is inherited or overridden requires knowledge of language semantics, so I'd say this is something to be provided by the language server and not the client. |
|
Given the various listed use cases I still think it would be reasonable to have an abstraction for 'a symbol in a document'. But I see @dbaeumer point about the |
|
The LSP actually tries hard to stay way from this. All the types it declares are bound to a tool action and usually return UI data types. The For most servers that request doesn't return all symbols defined in a document (for example locals are usually not part of this). I will close the issue. Please ping if someone disagrees. |
Clients can use this when the URI of the text document
cannot be inferred from the request context.
Issue: microsoft/language-server-protocol#582
Signed-off-by: Akos Kitta [email protected]