mirror of https://github.com/microsoft/vscode.git
Remove expandable hover setting
Seems like there's no downside in keeping this enabled by default for everyone
This commit is contained in:
parent
4bda9b5f58
commit
7094490c32
|
@ -458,15 +458,6 @@
|
|||
"default": true,
|
||||
"markdownDescription": "%configuration.updateImportsOnPaste%"
|
||||
},
|
||||
"typescript.experimental.expandableHover": {
|
||||
"type": "boolean",
|
||||
"default": true,
|
||||
"description": "%configuration.expandableHover%",
|
||||
"scope": "window",
|
||||
"tags": [
|
||||
"experimental"
|
||||
]
|
||||
},
|
||||
"js/ts.hover.maximumLength": {
|
||||
"type": "number",
|
||||
"default": 500,
|
||||
|
|
|
@ -225,7 +225,6 @@
|
|||
"configuration.tsserver.web.typeAcquisition.enabled": "Enable/disable package acquisition on the web. This enables IntelliSense for imported packages. Requires `#typescript.tsserver.web.projectWideIntellisense.enabled#`. Currently not supported for Safari.",
|
||||
"configuration.tsserver.nodePath": "Run TS Server on a custom Node installation. This can be a path to a Node executable, or 'node' if you want VS Code to detect a Node installation.",
|
||||
"configuration.updateImportsOnPaste": "Automatically update imports when pasting code. Requires TypeScript 5.6+.",
|
||||
"configuration.expandableHover": "Enable expanding/contracting the hover to reveal more/less information from the TS server. Requires TypeScript 5.9+.",
|
||||
"configuration.hover.maximumLength": "The maximum number of characters in a hover. If the hover is longer than this, it will be truncated. Requires TypeScript 5.9+.",
|
||||
"walkthroughs.nodejsWelcome.title": "Get started with JavaScript and Node.js",
|
||||
"walkthroughs.nodejsWelcome.description": "Make the most of Visual Studio Code's first-class JavaScript experience.",
|
||||
|
|
|
@ -33,9 +33,8 @@ class TypeScriptHoverProvider implements vscode.HoverProvider {
|
|||
return undefined;
|
||||
}
|
||||
|
||||
const enableExpandableHover = vscode.workspace.getConfiguration('typescript').get<boolean>('experimental.expandableHover', true);
|
||||
let verbosityLevel: number | undefined;
|
||||
if (enableExpandableHover && this.client.apiVersion.gte(API.v590)) {
|
||||
if (this.client.apiVersion.gte(API.v590)) {
|
||||
verbosityLevel = Math.max(0, this.getPreviousLevel(context?.previousHover) + (context?.verbosityDelta ?? 0));
|
||||
}
|
||||
const args = { ...typeConverters.Position.toFileLocationRequestArgs(filepath, position), verbosityLevel };
|
||||
|
|
Loading…
Reference in New Issue