Skip to content

Commit

Permalink
Dbaeumer/vicious-mole-white (#1866)
Browse files Browse the repository at this point in the history
* Recommend using [start, end] offset over substring.

* Fix formatting
  • Loading branch information
dbaeumer authored Dec 18, 2023
1 parent 4b034c5 commit 80b9d4a
Showing 1 changed file with 9 additions and 5 deletions.
14 changes: 9 additions & 5 deletions _specifications/lsp/3.18/language/signatureHelp.md
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ export interface SignatureHelpClientCapabilities {
* @since 3.18.0
*/
noActiveParameterSupport?: boolean;

};

/**
Expand Down Expand Up @@ -228,18 +228,18 @@ export interface SignatureHelp {

/**
* The active parameter of the active signature.
*
*
* If `null`, no parameter of the signature is active (for example a named
* argument that does not match any declared parameters). This is only valid
* since 3.18.0 and if the client specifies the client capability
* `textDocument.signatureHelp.noActiveParameterSupport === true`
*
*
* If omitted or the value lies outside the range of
* `signatures[activeSignature].parameters` defaults to 0 if the active
* signature has parameters.
*
*
* If the active signature has no parameters it is ignored.
*
*
* In future version of the protocol this property might become
* mandatory (but still nullable) to better express the active parameter if
* the active signature does have any.
Expand Down Expand Up @@ -308,6 +308,10 @@ export interface ParameterInformation {
* offsets are based on a UTF-16 string representation as `Position` and
* `Range` does.
*
* To avoid ambiguities a server should use the [start, end] offset value
* instead of using a substring. Whether a client support this is
* controlled via `labelOffsetSupport` client capability.
*
* *Note*: a label of type string should be a substring of its containing
* signature label. Its intended use case is to highlight the parameter
* label part in the `SignatureInformation.label`.
Expand Down

0 comments on commit 80b9d4a

Please sign in to comment.