diff --git a/.github/ISSUE_TEMPLATE/bug_report.md b/.github/ISSUE_TEMPLATE/bug_report.md
index de7ad30b9be..b40ba5ddd48 100644
--- a/.github/ISSUE_TEMPLATE/bug_report.md
+++ b/.github/ISSUE_TEMPLATE/bug_report.md
@@ -20,4 +20,4 @@ Steps to Reproduce:
Does this issue occur when all extensions are disabled?: Yes/No
-
+
diff --git a/.vscode/notebooks/endgame.github-issues b/.vscode/notebooks/endgame.github-issues
index c0a624c60c7..9183207f7a0 100644
--- a/.vscode/notebooks/endgame.github-issues
+++ b/.vscode/notebooks/endgame.github-issues
@@ -8,7 +8,7 @@
{
"kind": 2,
"language": "github-issues",
- "value": "$REPOS=repo:microsoft/vscode repo:microsoft/vscode-internalbacklog repo:microsoft/vscode-js-debug repo:microsoft/vscode-remote-release repo:microsoft/vscode-pull-request-github repo:microsoft/vscode-settings-sync-server repo:microsoft/vscode-emmet-helper\n\n$MILESTONE=milestone:\"March 2021\"",
+ "value": "$REPOS=repo:microsoft/vscode repo:microsoft/vscode-internalbacklog repo:microsoft/vscode-js-debug repo:microsoft/vscode-remote-release repo:microsoft/vscode-pull-request-github repo:microsoft/vscode-settings-sync-server repo:microsoft/vscode-emmet-helper\n\n$MILESTONE=milestone:\"April 2021\"",
"editable": true
},
{
diff --git a/build/darwin/create-universal-app.js b/build/darwin/create-universal-app.js
index 6e4acb28f35..d455a5cef70 100644
--- a/build/darwin/create-universal-app.js
+++ b/build/darwin/create-universal-app.js
@@ -33,6 +33,7 @@ async function main() {
'Credits.rtf',
'CodeResources',
'fsevents.node',
+ 'Info.plist', // TODO@deepak1556: regressed with 11.4.2 internal builds
'.npmrc'
],
outAppPath,
diff --git a/build/darwin/create-universal-app.ts b/build/darwin/create-universal-app.ts
index 0ec9e2a5877..b13e80100e6 100644
--- a/build/darwin/create-universal-app.ts
+++ b/build/darwin/create-universal-app.ts
@@ -38,6 +38,7 @@ async function main() {
'Credits.rtf',
'CodeResources',
'fsevents.node',
+ 'Info.plist', // TODO@deepak1556: regressed with 11.4.2 internal builds
'.npmrc'
],
outAppPath,
diff --git a/extensions/emmet/package.json b/extensions/emmet/package.json
index fd5f31ecc84..11f1689649a 100644
--- a/extensions/emmet/package.json
+++ b/extensions/emmet/package.json
@@ -454,5 +454,8 @@
"image-size": "^0.5.2",
"vscode-emmet-helper": "^2.3.0",
"vscode-languageserver-textdocument": "^1.0.1"
+ },
+ "workspaceTrust": {
+ "request": "never"
}
}
diff --git a/extensions/image-preview/src/preview.ts b/extensions/image-preview/src/preview.ts
index f4b589e0fb8..6d61c9898ae 100644
--- a/extensions/image-preview/src/preview.ts
+++ b/extensions/image-preview/src/preview.ts
@@ -212,6 +212,7 @@ class Preview extends Disposable {
const nonce = Date.now().toString();
+ const cspSource = this.webviewEditor.webview.cspSource;
return /* html */`
@@ -225,7 +226,7 @@ class Preview extends Disposable {
-
+
diff --git a/extensions/notebook-markdown-extensions/yarn.lock b/extensions/notebook-markdown-extensions/yarn.lock
index 591eff686e5..58952667358 100644
--- a/extensions/notebook-markdown-extensions/yarn.lock
+++ b/extensions/notebook-markdown-extensions/yarn.lock
@@ -4,7 +4,7 @@
"@iktakahiro/markdown-it-katex@https://github.com/mjbvz/markdown-it-katex.git":
version "4.0.1"
- resolved "https://github.com/mjbvz/markdown-it-katex.git#d3f98b077a40bc4ee9b42689df33feec62cdfbca"
+ resolved "https://github.com/mjbvz/markdown-it-katex.git#e88925a7cb3fd593a14ed117fb43627c4ba910b6"
dependencies:
katex "^0.13.0"
diff --git a/extensions/theme-abyss/themes/abyss-color-theme.json b/extensions/theme-abyss/themes/abyss-color-theme.json
index be6d15da953..3ee582a0149 100644
--- a/extensions/theme-abyss/themes/abyss-color-theme.json
+++ b/extensions/theme-abyss/themes/abyss-color-theme.json
@@ -362,6 +362,8 @@
// "peekViewTitleLabel.foreground": "",
// "peekViewTitleDescription.foreground": "",
+ // Ports
+ "ports.iconRunningProcessforeground": "#80a2c2",
// Editor: Diff
"diffEditor.insertedTextBackground": "#31958A55",
// "diffEditor.insertedTextBorder": "",
diff --git a/extensions/theme-defaults/themes/dark_vs.json b/extensions/theme-defaults/themes/dark_vs.json
index 061d360e3f0..010f74ed871 100644
--- a/extensions/theme-defaults/themes/dark_vs.json
+++ b/extensions/theme-defaults/themes/dark_vs.json
@@ -16,6 +16,7 @@
"menu.foreground": "#CCCCCC",
"statusBarItem.remoteForeground": "#FFF",
"statusBarItem.remoteBackground": "#16825D",
+ "ports.iconRunningProcessforeground": "#369432",
"sideBarSectionHeader.background": "#0000",
"sideBarSectionHeader.border": "#ccc3",
"tab.lastPinnedBorder": "#ccc3"
diff --git a/extensions/theme-defaults/themes/hc_black.json b/extensions/theme-defaults/themes/hc_black.json
index fcb350ac423..90825da88b6 100644
--- a/extensions/theme-defaults/themes/hc_black.json
+++ b/extensions/theme-defaults/themes/hc_black.json
@@ -9,7 +9,8 @@
"sideBarTitle.foreground": "#FFFFFF",
"selection.background": "#008000",
"editor.selectionBackground": "#FFFFFF",
- "statusBarItem.remoteBackground": "#00000000"
+ "statusBarItem.remoteBackground": "#00000000",
+ "ports.iconRunningProcessforeground": "#FFFFFF",
},
"tokenColors": [
{
@@ -441,7 +442,7 @@
"name": "HC Search Editor context line override",
"scope": "meta.resultLinePrefix.contextLinePrefix.search",
"settings": {
- "foreground": "#CBEDCB",
+ "foreground": "#CBEDCB"
}
}
],
@@ -450,6 +451,6 @@
"newOperator": "#FFFFFF",
"stringLiteral": "#ce9178",
"customLiteral": "#DCDCAA",
- "numberLiteral": "#b5cea8",
+ "numberLiteral": "#b5cea8"
}
}
diff --git a/extensions/theme-defaults/themes/light_vs.json b/extensions/theme-defaults/themes/light_vs.json
index 3b28fe777c0..084e2d27424 100644
--- a/extensions/theme-defaults/themes/light_vs.json
+++ b/extensions/theme-defaults/themes/light_vs.json
@@ -18,6 +18,7 @@
"settings.numberInputBorder": "#CECECE",
"statusBarItem.remoteForeground": "#FFF",
"statusBarItem.remoteBackground": "#16825D",
+ "ports.iconRunningProcessforeground": "#369432",
"sideBarSectionHeader.background": "#0000",
"sideBarSectionHeader.border": "#61616130",
"tab.lastPinnedBorder": "#61616130",
diff --git a/extensions/theme-kimbie-dark/themes/kimbie-dark-color-theme.json b/extensions/theme-kimbie-dark/themes/kimbie-dark-color-theme.json
index 3a7c37367a5..0c6f9cc64c5 100644
--- a/extensions/theme-kimbie-dark/themes/kimbie-dark-color-theme.json
+++ b/extensions/theme-kimbie-dark/themes/kimbie-dark-color-theme.json
@@ -29,6 +29,7 @@
"statusBar.debuggingBackground": "#423523",
"statusBar.noFolderBackground": "#423523",
"statusBarItem.remoteBackground": "#6e583b",
+ "ports.iconRunningProcessforeground": "#369432",
"activityBar.background": "#221a0f",
"activityBar.foreground": "#d3af86",
"sideBar.background": "#362712",
diff --git a/extensions/theme-monokai-dimmed/themes/dimmed-monokai-color-theme.json b/extensions/theme-monokai-dimmed/themes/dimmed-monokai-color-theme.json
index 11fb4a27d06..3d13ab7514a 100644
--- a/extensions/theme-monokai-dimmed/themes/dimmed-monokai-color-theme.json
+++ b/extensions/theme-monokai-dimmed/themes/dimmed-monokai-color-theme.json
@@ -33,6 +33,7 @@
"statusBar.noFolderBackground": "#505050",
"titleBar.activeBackground": "#505050",
"statusBarItem.remoteBackground": "#3655b5",
+ "ports.iconRunningProcessforeground": "#CCCCCC",
"activityBar.background": "#353535",
"activityBar.foreground": "#ffffff",
"activityBarBadge.background": "#3655b5",
diff --git a/extensions/theme-monokai/themes/monokai-color-theme.json b/extensions/theme-monokai/themes/monokai-color-theme.json
index 1172e585ecd..b5facc9d682 100644
--- a/extensions/theme-monokai/themes/monokai-color-theme.json
+++ b/extensions/theme-monokai/themes/monokai-color-theme.json
@@ -51,6 +51,7 @@
"statusBar.noFolderBackground": "#414339",
"statusBar.debuggingBackground": "#75715E",
"statusBarItem.remoteBackground": "#AC6218",
+ "ports.iconRunningProcessforeground": "#ccccc7",
"activityBar.background": "#272822",
"activityBar.foreground": "#f8f8f2",
"sideBar.background": "#1e1f1c",
diff --git a/extensions/theme-quietlight/themes/quietlight-color-theme.json b/extensions/theme-quietlight/themes/quietlight-color-theme.json
index 7a82fea3830..6e9707e5a51 100644
--- a/extensions/theme-quietlight/themes/quietlight-color-theme.json
+++ b/extensions/theme-quietlight/themes/quietlight-color-theme.json
@@ -506,6 +506,7 @@
"statusBar.noFolderBackground": "#705697",
"statusBar.debuggingBackground": "#705697",
"statusBarItem.remoteBackground": "#4e3c69",
+ "ports.iconRunningProcessforeground": "#749351",
"activityBar.background": "#EDEDF5",
"activityBar.foreground": "#705697",
"activityBarBadge.background": "#705697",
diff --git a/extensions/theme-red/themes/Red-color-theme.json b/extensions/theme-red/themes/Red-color-theme.json
index 879093989fc..d0e2d16a410 100644
--- a/extensions/theme-red/themes/Red-color-theme.json
+++ b/extensions/theme-red/themes/Red-color-theme.json
@@ -10,6 +10,7 @@
"statusBar.background": "#700000",
"statusBar.noFolderBackground": "#700000",
"statusBarItem.remoteBackground": "#c33",
+ "ports.iconRunningProcessforeground": "#DB7E58",
"editorGroupHeader.tabsBackground": "#330000",
"titleBar.activeBackground": "#770000",
"titleBar.inactiveBackground": "#772222",
diff --git a/extensions/theme-solarized-dark/themes/solarized-dark-color-theme.json b/extensions/theme-solarized-dark/themes/solarized-dark-color-theme.json
index b5aba98283c..24e54ca9d3b 100644
--- a/extensions/theme-solarized-dark/themes/solarized-dark-color-theme.json
+++ b/extensions/theme-solarized-dark/themes/solarized-dark-color-theme.json
@@ -462,6 +462,7 @@
"statusBar.debuggingBackground": "#00212B",
"statusBar.noFolderBackground": "#00212B",
"statusBarItem.remoteBackground": "#2AA19899",
+ "ports.iconRunningProcessforeground": "#369432",
"statusBarItem.prominentBackground": "#003847",
"statusBarItem.prominentHoverBackground": "#003847",
// "statusBarItem.activeBackground": "",
diff --git a/extensions/theme-solarized-light/themes/solarized-light-color-theme.json b/extensions/theme-solarized-light/themes/solarized-light-color-theme.json
index 7c14e8996b4..3265ceb664e 100644
--- a/extensions/theme-solarized-light/themes/solarized-light-color-theme.json
+++ b/extensions/theme-solarized-light/themes/solarized-light-color-theme.json
@@ -465,6 +465,7 @@
"statusBar.noFolderBackground": "#EEE8D5",
// "statusBar.foreground": "",
"statusBarItem.remoteBackground": "#AC9D57",
+ "ports.iconRunningProcessforeground": "#2AA19899",
"statusBarItem.prominentBackground": "#DDD6C1",
"statusBarItem.prominentHoverBackground": "#DDD6C199",
// "statusBarItem.activeBackground": "",
diff --git a/extensions/theme-tomorrow-night-blue/themes/tomorrow-night-blue-color-theme.json b/extensions/theme-tomorrow-night-blue/themes/tomorrow-night-blue-color-theme.json
index f2ea18b7b1c..8c2772bccfc 100644
--- a/extensions/theme-tomorrow-night-blue/themes/tomorrow-night-blue-color-theme.json
+++ b/extensions/theme-tomorrow-night-blue/themes/tomorrow-night-blue-color-theme.json
@@ -32,6 +32,7 @@
"titleBar.activeBackground": "#001126",
"statusBar.background": "#001126",
"statusBarItem.remoteBackground": "#0e639c",
+ "ports.iconRunningProcessforeground": "#bbdaff",
"statusBar.noFolderBackground": "#001126",
"statusBar.debuggingBackground": "#001126",
"activityBar.background": "#001733",
diff --git a/extensions/typescript-language-features/src/tsServer/serverError.ts b/extensions/typescript-language-features/src/tsServer/serverError.ts
index 2653360c9be..be9a44d276d 100644
--- a/extensions/typescript-language-features/src/tsServer/serverError.ts
+++ b/extensions/typescript-language-features/src/tsServer/serverError.ts
@@ -39,13 +39,15 @@ export class TypeScriptServerError extends Error {
"TypeScriptRequestErrorProperties" : {
"command" : { "classification": "SystemMetaData", "purpose": "FeatureInsight" },
"serverid" : { "classification": "SystemMetaData", "purpose": "PerformanceAndHealth" },
- "sanitizedstack" : { "classification": "SystemMetaData", "purpose": "PerformanceAndHealth" }
+ "sanitizedstack" : { "classification": "SystemMetaData", "purpose": "PerformanceAndHealth" },
+ "badclient" : { "classification": "SystemMetaData", "purpose": "PerformanceAndHealth" }
}
*/
return {
command: this.serverCommand,
serverid: this.serverId,
sanitizedstack: this.sanitizedStack || '',
+ badclient: /\bBADCLIENT\b/.test(this.stack || ''),
} as const;
}
diff --git a/extensions/typescript-language-features/src/typescriptServiceClient.ts b/extensions/typescript-language-features/src/typescriptServiceClient.ts
index a56c6d77f75..430813d3ad6 100644
--- a/extensions/typescript-language-features/src/typescriptServiceClient.ts
+++ b/extensions/typescript-language-features/src/typescriptServiceClient.ts
@@ -710,16 +710,20 @@ export default class TypeScriptServiceClient extends Disposable implements IType
return undefined;
}
- if (resource.scheme === fileSchemes.file || resource.scheme === fileSchemes.untitled) {
- for (const root of roots.sort((a, b) => a.uri.fsPath.length - b.uri.fsPath.length)) {
- if (resource.fsPath.startsWith(root.uri.fsPath + path.sep)) {
- return root.uri.fsPath;
+ switch (resource.scheme) {
+ case fileSchemes.file:
+ case fileSchemes.untitled:
+ case fileSchemes.vscodeNotebookCell:
+ for (const root of roots.sort((a, b) => a.uri.fsPath.length - b.uri.fsPath.length)) {
+ if (resource.fsPath.startsWith(root.uri.fsPath + path.sep)) {
+ return root.uri.fsPath;
+ }
}
- }
- return roots[0].uri.fsPath;
- }
+ return roots[0].uri.fsPath;
- return undefined;
+ default:
+ return undefined;
+ }
}
public execute(command: keyof TypeScriptRequests, args: any, token: vscode.CancellationToken, config?: ExecConfig): Promise> {
diff --git a/extensions/typescript-language-features/src/utils/telemetry.ts b/extensions/typescript-language-features/src/utils/telemetry.ts
index 13527cae125..5231c0fb35a 100644
--- a/extensions/typescript-language-features/src/utils/telemetry.ts
+++ b/extensions/typescript-language-features/src/utils/telemetry.ts
@@ -14,7 +14,7 @@ interface PackageInfo {
}
export interface TelemetryProperties {
- readonly [prop: string]: string | number | undefined;
+ readonly [prop: string]: string | number | boolean | undefined;
}
export interface TelemetryReporter {
diff --git a/package.json b/package.json
index 82ba6ba716f..7f94f3f4a47 100644
--- a/package.json
+++ b/package.json
@@ -42,7 +42,7 @@
"monaco-compile-check": "tsc -p src/tsconfig.monaco.json --noEmit",
"tsec-compile-check": "node node_modules/tsec/bin/tsec -p src/tsconfig.tsec.json",
"valid-layers-check": "node build/lib/layersChecker.js",
- "strict-function-types-watch": "tsc --watch -p src/tsconfig.json --noEmit --strictFunctionTypes",
+ "no-implicit-override-watch": "tsc --watch -p src/tsconfig.json --noEmit --noImplicitOverride",
"update-distro": "node build/npm/update-distro.js",
"web": "node resources/web/code-web.js",
"compile-web": "node --max_old_space_size=4095 ./node_modules/gulp/bin/gulp.js compile-web",
diff --git a/product.json b/product.json
index 7344ed79094..79267cd4652 100644
--- a/product.json
+++ b/product.json
@@ -33,7 +33,7 @@
"builtInExtensions": [
{
"name": "ms-vscode.node-debug",
- "version": "1.44.20",
+ "version": "1.44.23",
"repo": "https://github.com/microsoft/vscode-node-debug",
"metadata": {
"id": "b6ded8fb-a0a0-4c1c-acbd-ab2a3bc995a6",
@@ -63,7 +63,7 @@
},
{
"name": "ms-vscode.references-view",
- "version": "0.0.77",
+ "version": "0.0.78",
"repo": "https://github.com/microsoft/vscode-references-view",
"metadata": {
"id": "dc489f46-520d-4556-ae85-1f9eab3c412d",
diff --git a/src/main.js b/src/main.js
index 9f7e2304720..1e3b9896dce 100644
--- a/src/main.js
+++ b/src/main.js
@@ -18,7 +18,6 @@ perf.mark('code/didStartMain');
const path = require('path');
const fs = require('fs');
const os = require('os');
-const { getNLSConfiguration } = require('./vs/base/node/languagePacks');
const bootstrap = require('./bootstrap');
const bootstrapNode = require('./bootstrap-node');
const { getUserDataPath } = require('./vs/platform/environment/node/userDataPath');
@@ -45,7 +44,9 @@ app.setPath('userData', userDataPath);
const argvConfig = configureCommandlineSwitchesSync(args);
// Configure crash reporter
+perf.mark('code/willStartCrashReporter');
configureCrashReporter();
+perf.mark('code/didStartCrashReporter');
// Set logs path before app 'ready' event if running portable
// to ensure that no 'logs' folder is created on disk at a
@@ -60,9 +61,6 @@ protocol.registerSchemesAsPrivileged([
{
scheme: 'vscode-webview',
privileges: { standard: true, secure: true, supportFetchAPI: true, corsEnabled: true, allowServiceWorkers: true, }
- }, {
- scheme: 'vscode-webview-resource',
- privileges: { secure: true, standard: true, supportFetchAPI: true, corsEnabled: true }
},
{
scheme: 'vscode-file',
@@ -87,6 +85,7 @@ let nlsConfigurationPromise = undefined;
const metaDataFile = path.join(__dirname, 'nls.metadata.json');
const locale = getUserDefinedLocale(argvConfig);
if (locale) {
+ const { getNLSConfiguration } = require('./vs/base/node/languagePacks');
nlsConfigurationPromise = getNLSConfiguration(product.commit, userDataPath, metaDataFile, locale);
}
@@ -579,6 +578,7 @@ async function resolveNlsConfiguration() {
// See above the comment about the loader and case sensitiviness
appLocale = appLocale.toLowerCase();
+ const { getNLSConfiguration } = require('./vs/base/node/languagePacks');
nlsConfiguration = await getNLSConfiguration(product.commit, userDataPath, metaDataFile, appLocale);
if (!nlsConfiguration) {
nlsConfiguration = { locale: appLocale, availableLanguages: {} };
diff --git a/src/vs/base/browser/dnd.ts b/src/vs/base/browser/dnd.ts
index 3753e458b28..c9d07cccfc3 100644
--- a/src/vs/base/browser/dnd.ts
+++ b/src/vs/base/browser/dnd.ts
@@ -42,7 +42,7 @@ export class DelayedDragHandler extends Disposable {
}
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this.clearDragTimeout();
diff --git a/src/vs/base/browser/dom.ts b/src/vs/base/browser/dom.ts
index 0bdba6f8d6c..90f56d92e05 100644
--- a/src/vs/base/browser/dom.ts
+++ b/src/vs/base/browser/dom.ts
@@ -1589,7 +1589,7 @@ export class ModifierKeyEmitter extends Emitter {
return ModifierKeyEmitter.instance;
}
- dispose() {
+ override dispose() {
super.dispose();
this._subscriptions.dispose();
}
diff --git a/src/vs/base/browser/touch.ts b/src/vs/base/browser/touch.ts
index 7166c4ec3a9..8e9c085748a 100644
--- a/src/vs/base/browser/touch.ts
+++ b/src/vs/base/browser/touch.ts
@@ -136,7 +136,7 @@ export class Gesture extends Disposable {
return 'ontouchstart' in window || navigator.maxTouchPoints > 0 || (window as Window).navigator.msMaxTouchPoints > 0;
}
- public dispose(): void {
+ public override dispose(): void {
if (this.handle) {
this.handle.dispose();
this.handle = null;
diff --git a/src/vs/base/browser/ui/actionbar/actionViewItems.ts b/src/vs/base/browser/ui/actionbar/actionViewItems.ts
index 7203b9850c0..98c0c667054 100644
--- a/src/vs/base/browser/ui/actionbar/actionViewItems.ts
+++ b/src/vs/base/browser/ui/actionbar/actionViewItems.ts
@@ -212,7 +212,7 @@ export class BaseActionViewItem extends Disposable implements IActionViewItem {
// implement in subclass
}
- dispose(): void {
+ override dispose(): void {
if (this.element) {
this.element.remove();
this.element = undefined;
@@ -231,7 +231,7 @@ export interface IActionViewItemOptions extends IBaseActionViewItemOptions {
export class ActionViewItem extends BaseActionViewItem {
protected label: HTMLElement | undefined;
- protected options: IActionViewItemOptions;
+ protected override options: IActionViewItemOptions;
private cssClass?: string;
@@ -244,7 +244,7 @@ export class ActionViewItem extends BaseActionViewItem {
this.cssClass = '';
}
- render(container: HTMLElement): void {
+ override render(container: HTMLElement): void {
super.render(container);
if (this.element) {
@@ -276,32 +276,32 @@ export class ActionViewItem extends BaseActionViewItem {
// Only set the tabIndex on the element once it is about to get focused
// That way this element wont be a tab stop when it is not needed #106441
- focus(): void {
+ override focus(): void {
if (this.label) {
this.label.tabIndex = 0;
this.label.focus();
}
}
- blur(): void {
+ override blur(): void {
if (this.label) {
this.label.tabIndex = -1;
}
}
- setFocusable(focusable: boolean): void {
+ override setFocusable(focusable: boolean): void {
if (this.label) {
this.label.tabIndex = focusable ? 0 : -1;
}
}
- updateLabel(): void {
+ override updateLabel(): void {
if (this.options.label && this.label) {
this.label.textContent = this.getAction().label;
}
}
- updateTooltip(): void {
+ override updateTooltip(): void {
let title: string | null = null;
if (this.getAction().tooltip) {
@@ -320,7 +320,7 @@ export class ActionViewItem extends BaseActionViewItem {
}
}
- updateClass(): void {
+ override updateClass(): void {
if (this.cssClass && this.label) {
this.label.classList.remove(...this.cssClass.split(' '));
}
@@ -343,7 +343,7 @@ export class ActionViewItem extends BaseActionViewItem {
}
}
- updateEnabled(): void {
+ override updateEnabled(): void {
if (this.getAction().enabled) {
if (this.label) {
this.label.removeAttribute('aria-disabled');
@@ -365,7 +365,7 @@ export class ActionViewItem extends BaseActionViewItem {
}
}
- updateChecked(): void {
+ override updateChecked(): void {
if (this.label) {
if (this.getAction().checked) {
this.label.classList.add('checked');
@@ -407,23 +407,23 @@ export class SelectActionViewItem extends BaseActionViewItem {
return option;
}
- setFocusable(focusable: boolean): void {
+ override setFocusable(focusable: boolean): void {
this.selectBox.setFocusable(focusable);
}
- focus(): void {
+ override focus(): void {
if (this.selectBox) {
this.selectBox.focus();
}
}
- blur(): void {
+ override blur(): void {
if (this.selectBox) {
this.selectBox.blur();
}
}
- render(container: HTMLElement): void {
+ override render(container: HTMLElement): void {
this.selectBox.render(container);
}
}
diff --git a/src/vs/base/browser/ui/actionbar/actionbar.css b/src/vs/base/browser/ui/actionbar/actionbar.css
index 2afccd2b6bf..180126a4177 100644
--- a/src/vs/base/browser/ui/actionbar/actionbar.css
+++ b/src/vs/base/browser/ui/actionbar/actionbar.css
@@ -16,6 +16,7 @@
height: 100%;
width: 100%;
justify-content: flex-end;
+ align-items: center;
}
.monaco-action-bar.vertical .actions-container {
@@ -23,12 +24,11 @@
}
.monaco-action-bar .action-item {
- display: flex;
+ display: block;
align-items: center;
justify-content: center;
cursor: pointer;
position: relative; /* DO NOT REMOVE - this is the key to preventing the ghosting icon bug in Chrome 42 */
- min-width: 24px;
}
.monaco-action-bar .eaction-item.disabled {
@@ -37,12 +37,14 @@
.monaco-action-bar .action-item .icon,
.monaco-action-bar .action-item .codicon {
- display: inline-block;
+ display: block;
}
.monaco-action-bar .action-item .codicon {
display: flex;
align-items: center;
+ width: 16px;
+ height: 16px;
}
.monaco-action-bar .action-label {
diff --git a/src/vs/base/browser/ui/actionbar/actionbar.ts b/src/vs/base/browser/ui/actionbar/actionbar.ts
index b6fcbb24295..d5486aefaee 100644
--- a/src/vs/base/browser/ui/actionbar/actionbar.ts
+++ b/src/vs/base/browser/ui/actionbar/actionbar.ts
@@ -529,7 +529,7 @@ export class ActionBar extends Disposable implements IActionRunner {
await this._actionRunner.run(action, context);
}
- dispose(): void {
+ override dispose(): void {
dispose(this.viewItems);
this.viewItems = [];
diff --git a/src/vs/base/browser/ui/checkbox/checkbox.ts b/src/vs/base/browser/ui/checkbox/checkbox.ts
index d9319424fc5..5a67a234770 100644
--- a/src/vs/base/browser/ui/checkbox/checkbox.ts
+++ b/src/vs/base/browser/ui/checkbox/checkbox.ts
@@ -44,7 +44,7 @@ export class CheckboxActionViewItem extends BaseActionViewItem {
protected checkbox: Checkbox | undefined;
protected readonly disposables = new DisposableStore();
- render(container: HTMLElement): void {
+ override render(container: HTMLElement): void {
this.element = container;
this.disposables.clear();
@@ -59,7 +59,7 @@ export class CheckboxActionViewItem extends BaseActionViewItem {
this.element.appendChild(this.checkbox.domNode);
}
- updateEnabled(): void {
+ override updateEnabled(): void {
if (this.checkbox) {
if (this.isEnabled()) {
this.checkbox.enable();
@@ -69,33 +69,33 @@ export class CheckboxActionViewItem extends BaseActionViewItem {
}
}
- updateChecked(): void {
+ override updateChecked(): void {
if (this.checkbox) {
this.checkbox.checked = this._action.checked;
}
}
- focus(): void {
+ override focus(): void {
if (this.checkbox) {
this.checkbox.domNode.tabIndex = 0;
this.checkbox.focus();
}
}
- blur(): void {
+ override blur(): void {
if (this.checkbox) {
this.checkbox.domNode.tabIndex = -1;
this.checkbox.domNode.blur();
}
}
- setFocusable(focusable: boolean): void {
+ override setFocusable(focusable: boolean): void {
if (this.checkbox) {
this.checkbox.domNode.tabIndex = focusable ? 0 : -1;
}
}
- dispose(): void {
+ override dispose(): void {
this.disposables.dispose();
super.dispose();
}
diff --git a/src/vs/base/browser/ui/contextview/contextview.ts b/src/vs/base/browser/ui/contextview/contextview.ts
index 9632da876a7..30efdd96b63 100644
--- a/src/vs/base/browser/ui/contextview/contextview.ts
+++ b/src/vs/base/browser/ui/contextview/contextview.ts
@@ -351,7 +351,7 @@ export class ContextView extends Disposable {
}
}
- dispose(): void {
+ override dispose(): void {
this.hide();
super.dispose();
diff --git a/src/vs/base/browser/ui/dialog/dialog.ts b/src/vs/base/browser/ui/dialog/dialog.ts
index a1646c4ccf3..35d0ddc2db7 100644
--- a/src/vs/base/browser/ui/dialog/dialog.ts
+++ b/src/vs/base/browser/ui/dialog/dialog.ts
@@ -417,7 +417,7 @@ export class Dialog extends Disposable {
this.applyStyles();
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
if (this.modalElement) {
diff --git a/src/vs/base/browser/ui/dropdown/dropdown.ts b/src/vs/base/browser/ui/dropdown/dropdown.ts
index ea6611aba29..059b85ecb19 100644
--- a/src/vs/base/browser/ui/dropdown/dropdown.ts
+++ b/src/vs/base/browser/ui/dropdown/dropdown.ts
@@ -125,7 +125,7 @@ export class BaseDropdown extends ActionRunner {
this.hide();
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this.hide();
@@ -159,7 +159,7 @@ export class Dropdown extends BaseDropdown {
this.contextViewProvider = options.contextViewProvider;
}
- show(): void {
+ override show(): void {
super.show();
this.element.classList.add('active');
@@ -187,7 +187,7 @@ export class Dropdown extends BaseDropdown {
this.element.classList.remove('active');
}
- hide(): void {
+ override hide(): void {
super.hide();
if (this.contextViewProvider) {
@@ -250,7 +250,7 @@ export class DropdownMenu extends BaseDropdown {
this._actions = actions;
}
- show(): void {
+ override show(): void {
super.show();
this.element.classList.add('active');
@@ -269,7 +269,7 @@ export class DropdownMenu extends BaseDropdown {
});
}
- hide(): void {
+ override hide(): void {
super.hide();
}
diff --git a/src/vs/base/browser/ui/dropdown/dropdownActionViewItem.ts b/src/vs/base/browser/ui/dropdown/dropdownActionViewItem.ts
index 67e2e9bb7c1..c7f9e2b890a 100644
--- a/src/vs/base/browser/ui/dropdown/dropdownActionViewItem.ts
+++ b/src/vs/base/browser/ui/dropdown/dropdownActionViewItem.ts
@@ -42,23 +42,26 @@ export class DropdownMenuActionViewItem extends BaseActionViewItem {
private _onDidChangeVisibility = this._register(new Emitter());
readonly onDidChangeVisibility = this._onDidChangeVisibility.event;
+ protected override readonly options: IDropdownMenuActionViewItemOptions;
+
constructor(
action: IAction,
menuActionsOrProvider: readonly IAction[] | IActionProvider,
contextMenuProvider: IContextMenuProvider,
- protected options: IDropdownMenuActionViewItemOptions = {}
+ options: IDropdownMenuActionViewItemOptions = Object.create(null)
) {
super(null, action, options);
this.menuActionsOrProvider = menuActionsOrProvider;
this.contextMenuProvider = contextMenuProvider;
+ this.options = options;
if (this.options.actionRunner) {
this.actionRunner = this.options.actionRunner;
}
}
- render(container: HTMLElement): void {
+ override render(container: HTMLElement): void {
this.actionItem = container;
const labelRenderer: ILabelRenderer = (el: HTMLElement): IDisposable | null => {
@@ -123,7 +126,7 @@ export class DropdownMenuActionViewItem extends BaseActionViewItem {
this.updateEnabled();
}
- setActionContext(newContext: unknown): void {
+ override setActionContext(newContext: unknown): void {
super.setActionContext(newContext);
if (this.dropdownMenu) {
@@ -141,7 +144,7 @@ export class DropdownMenuActionViewItem extends BaseActionViewItem {
}
}
- protected updateEnabled(): void {
+ protected override updateEnabled(): void {
const disabled = !this.getAction().enabled;
this.actionItem?.classList.toggle('disabled', disabled);
this.element?.classList.toggle('disabled', disabled);
@@ -166,7 +169,7 @@ export class ActionWithDropdownActionViewItem extends ActionViewItem {
super(context, action, options);
}
- render(container: HTMLElement): void {
+ override render(container: HTMLElement): void {
super.render(container);
if (this.element) {
this.element.classList.add('action-dropdown-item');
diff --git a/src/vs/base/browser/ui/findinput/replaceInput.ts b/src/vs/base/browser/ui/findinput/replaceInput.ts
index 1d6200fdb85..13ed1ade990 100644
--- a/src/vs/base/browser/ui/findinput/replaceInput.ts
+++ b/src/vs/base/browser/ui/findinput/replaceInput.ts
@@ -382,7 +382,7 @@ export class ReplaceInput extends Widget {
this.domNode.style.width = newWidth + 'px';
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
}
diff --git a/src/vs/base/browser/ui/grid/grid.ts b/src/vs/base/browser/ui/grid/grid.ts
index 75e139fc46f..38bc8c53d9e 100644
--- a/src/vs/base/browser/ui/grid/grid.ts
+++ b/src/vs/base/browser/ui/grid/grid.ts
@@ -551,7 +551,7 @@ export class SerializableGrid extends Grid {
};
}
- layout(width: number, height: number): void {
+ override layout(width: number, height: number): void {
super.layout(width, height);
if (this.initialLayoutContext) {
diff --git a/src/vs/base/browser/ui/inputbox/inputBox.ts b/src/vs/base/browser/ui/inputbox/inputBox.ts
index c22ee4169c4..62638456615 100644
--- a/src/vs/base/browser/ui/inputbox/inputBox.ts
+++ b/src/vs/base/browser/ui/inputbox/inputBox.ts
@@ -610,7 +610,7 @@ export class InputBox extends Widget {
}
}
- public dispose(): void {
+ public override dispose(): void {
this._hideMessage();
this.message = null;
diff --git a/src/vs/base/browser/ui/list/listWidget.ts b/src/vs/base/browser/ui/list/listWidget.ts
index 41cfef5cd14..dcffc73d99b 100644
--- a/src/vs/base/browser/ui/list/listWidget.ts
+++ b/src/vs/base/browser/ui/list/listWidget.ts
@@ -188,7 +188,7 @@ class SelectionTrait extends Trait {
super('selected');
}
- renderIndex(index: number, container: HTMLElement): void {
+ override renderIndex(index: number, container: HTMLElement): void {
super.renderIndex(index, container);
if (this.setAriaSelected) {
diff --git a/src/vs/base/browser/ui/menu/menu.ts b/src/vs/base/browser/ui/menu/menu.ts
index 36e09bb3a66..4bf8f108bed 100644
--- a/src/vs/base/browser/ui/menu/menu.ts
+++ b/src/vs/base/browser/ui/menu/menu.ts
@@ -264,7 +264,7 @@ export class Menu extends ActionBar {
}
}
- getContainer(): HTMLElement {
+ override getContainer(): HTMLElement {
return this.scrollableElement.getDomNode();
}
@@ -309,7 +309,7 @@ export class Menu extends ActionBar {
}
}
- protected updateFocus(fromRight?: boolean): void {
+ protected override updateFocus(fromRight?: boolean): void {
super.updateFocus(fromRight, true);
if (typeof this.focusedItem !== 'undefined') {
@@ -385,7 +385,7 @@ class BaseMenuActionViewItem extends BaseActionViewItem {
public container: HTMLElement | undefined;
- protected options: IMenuItemOptions;
+ protected override options: IMenuItemOptions;
protected item: HTMLElement | undefined;
private runOnceToEnableMouseUp: RunOnceScheduler;
@@ -465,7 +465,7 @@ class BaseMenuActionViewItem extends BaseActionViewItem {
this._register(this.runOnceToEnableMouseUp);
}
- render(container: HTMLElement): void {
+ override render(container: HTMLElement): void {
super.render(container);
if (!this.element) {
@@ -504,12 +504,12 @@ class BaseMenuActionViewItem extends BaseActionViewItem {
this.updateChecked();
}
- blur(): void {
+ override blur(): void {
super.blur();
this.applyStyle();
}
- focus(): void {
+ override focus(): void {
super.focus();
if (this.item) {
@@ -526,7 +526,7 @@ class BaseMenuActionViewItem extends BaseActionViewItem {
}
}
- updateLabel(): void {
+ override updateLabel(): void {
if (!this.label) {
return;
}
@@ -579,7 +579,7 @@ class BaseMenuActionViewItem extends BaseActionViewItem {
}
}
- updateTooltip(): void {
+ override updateTooltip(): void {
let title: string | null = null;
if (this.getAction().tooltip) {
@@ -598,7 +598,7 @@ class BaseMenuActionViewItem extends BaseActionViewItem {
}
}
- updateClass(): void {
+ override updateClass(): void {
if (this.cssClass && this.item) {
this.item.classList.remove(...this.cssClass.split(' '));
}
@@ -614,7 +614,7 @@ class BaseMenuActionViewItem extends BaseActionViewItem {
}
}
- updateEnabled(): void {
+ override updateEnabled(): void {
if (this.getAction().enabled) {
if (this.element) {
this.element.classList.remove('disabled');
@@ -639,7 +639,7 @@ class BaseMenuActionViewItem extends BaseActionViewItem {
}
}
- updateChecked(): void {
+ override updateChecked(): void {
if (!this.item) {
return;
}
@@ -724,7 +724,7 @@ class SubmenuMenuActionViewItem extends BaseMenuActionViewItem {
}, 750);
}
- render(container: HTMLElement): void {
+ override render(container: HTMLElement): void {
super.render(container);
if (!this.element) {
@@ -783,7 +783,7 @@ class SubmenuMenuActionViewItem extends BaseMenuActionViewItem {
}));
}
- updateEnabled(): void {
+ override updateEnabled(): void {
// override on submenu entry
// native menus do not observe enablement on sumbenus
// we mimic that behavior
@@ -794,7 +794,7 @@ class SubmenuMenuActionViewItem extends BaseMenuActionViewItem {
this.createSubmenu(selectFirst);
}
- onClick(e: EventLike): void {
+ override onClick(e: EventLike): void {
// stop clicking from trying to run an action
EventHelper.stop(e, true);
@@ -925,7 +925,7 @@ class SubmenuMenuActionViewItem extends BaseMenuActionViewItem {
}
}
- protected applyStyle(): void {
+ protected override applyStyle(): void {
super.applyStyle();
if (!this.menuStyle) {
@@ -944,7 +944,7 @@ class SubmenuMenuActionViewItem extends BaseMenuActionViewItem {
}
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this.hideScheduler.dispose();
diff --git a/src/vs/base/browser/ui/menu/menubar.ts b/src/vs/base/browser/ui/menu/menubar.ts
index 129c5663523..1667f808289 100644
--- a/src/vs/base/browser/ui/menu/menubar.ts
+++ b/src/vs/base/browser/ui/menu/menubar.ts
@@ -422,7 +422,7 @@ export class MenuBar extends Disposable {
}
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this.menuCache.forEach(menuBarMenu => {
diff --git a/src/vs/base/browser/ui/sash/sash.ts b/src/vs/base/browser/ui/sash/sash.ts
index b3d76b24de9..d9bd8266e44 100644
--- a/src/vs/base/browser/ui/sash/sash.ts
+++ b/src/vs/base/browser/ui/sash/sash.ts
@@ -484,7 +484,7 @@ export class Sash extends Disposable {
return undefined;
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this.el.remove();
}
diff --git a/src/vs/base/browser/ui/scrollbar/scrollableElement.ts b/src/vs/base/browser/ui/scrollbar/scrollableElement.ts
index fe7b7edc9b9..733e913ae6f 100644
--- a/src/vs/base/browser/ui/scrollbar/scrollableElement.ts
+++ b/src/vs/base/browser/ui/scrollbar/scrollableElement.ts
@@ -239,7 +239,7 @@ export abstract class AbstractScrollableElement extends Widget {
this._revealOnScroll = true;
}
- public dispose(): void {
+ public override dispose(): void {
this._mouseWheelToDispose = dispose(this._mouseWheelToDispose);
super.dispose();
}
diff --git a/src/vs/base/browser/ui/selectBox/selectBoxCustom.ts b/src/vs/base/browser/ui/selectBox/selectBoxCustom.ts
index 19e22c09854..9f60bfb43ef 100644
--- a/src/vs/base/browser/ui/selectBox/selectBoxCustom.ts
+++ b/src/vs/base/browser/ui/selectBox/selectBoxCustom.ts
@@ -1046,7 +1046,7 @@ export class SelectBoxList extends Disposable implements ISelectBoxDelegate, ILi
}
}
- public dispose(): void {
+ public override dispose(): void {
this.hideSelectDropDown(false);
super.dispose();
}
diff --git a/src/vs/base/browser/ui/splitview/paneview.css b/src/vs/base/browser/ui/splitview/paneview.css
index f7484d47102..7dc22bed9f2 100644
--- a/src/vs/base/browser/ui/splitview/paneview.css
+++ b/src/vs/base/browser/ui/splitview/paneview.css
@@ -52,6 +52,10 @@
margin-left: auto;
}
+.monaco-pane-view .pane > .pane-header > .actions .action-item {
+ margin-right: 4px;
+}
+
.monaco-pane-view .pane > .pane-header > .actions .action-label {
padding: 2px;
}
diff --git a/src/vs/base/browser/ui/splitview/paneview.ts b/src/vs/base/browser/ui/splitview/paneview.ts
index 15bdc774f69..06e8b064d49 100644
--- a/src/vs/base/browser/ui/splitview/paneview.ts
+++ b/src/vs/base/browser/ui/splitview/paneview.ts
@@ -568,7 +568,7 @@ export class PaneView extends Disposable {
}, 200);
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this.paneItems.forEach(i => i.disposable.dispose());
diff --git a/src/vs/base/browser/ui/splitview/splitview.ts b/src/vs/base/browser/ui/splitview/splitview.ts
index 40082a2270f..8434bc1c528 100644
--- a/src/vs/base/browser/ui/splitview/splitview.ts
+++ b/src/vs/base/browser/ui/splitview/splitview.ts
@@ -1022,7 +1022,7 @@ export class SplitView extends Disposable {
return undefined;
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this.viewItems.forEach(i => i.dispose());
diff --git a/src/vs/base/browser/ui/toolbar/toolbar.ts b/src/vs/base/browser/ui/toolbar/toolbar.ts
index 8d1c5b6b17a..013fc13a2b7 100644
--- a/src/vs/base/browser/ui/toolbar/toolbar.ts
+++ b/src/vs/base/browser/ui/toolbar/toolbar.ts
@@ -181,7 +181,7 @@ export class ToolBar extends Disposable {
this.actionBar.clear();
}
- dispose(): void {
+ override dispose(): void {
this.clear();
super.dispose();
}
@@ -202,7 +202,7 @@ class ToggleMenuAction extends Action {
this.toggleDropdownMenu = toggleDropdownMenu;
}
- async run(): Promise {
+ async override run(): Promise {
this.toggleDropdownMenu();
}
diff --git a/src/vs/base/browser/ui/tree/abstractTree.ts b/src/vs/base/browser/ui/tree/abstractTree.ts
index 2b466e54918..efc69fa4755 100644
--- a/src/vs/base/browser/ui/tree/abstractTree.ts
+++ b/src/vs/base/browser/ui/tree/abstractTree.ts
@@ -29,11 +29,11 @@ import { treeItemExpandedIcon, treeFilterOnTypeOnIcon, treeFilterOnTypeOffIcon,
class TreeElementsDragAndDropData extends ElementsDragAndDropData {
- set context(context: TContext | undefined) {
+ override set context(context: TContext | undefined) {
this.data.context = context;
}
- get context(): TContext | undefined {
+ override get context(): TContext | undefined {
return this.data.context;
}
@@ -1091,7 +1091,7 @@ class TreeNodeListMouseController extends MouseController<
super(list);
}
- protected onViewPointer(e: IListMouseEvent>): void {
+ protected override onViewPointer(e: IListMouseEvent>): void {
if (isInputElement(e.browserEvent.target as HTMLElement) || isMonacoEditor(e.browserEvent.target as HTMLElement)) {
return;
}
@@ -1141,7 +1141,7 @@ class TreeNodeListMouseController extends MouseController<
super.onViewPointer(e);
}
- protected onDoubleClick(e: IListMouseEvent>): void {
+ protected override onDoubleClick(e: IListMouseEvent>): void {
const onTwistie = (e.browserEvent.target as HTMLElement).classList.contains('monaco-tl-twistie');
if (onTwistie || !this.tree.expandOnDoubleClick) {
@@ -1175,11 +1175,11 @@ class TreeNodeList extends List>
super(user, container, virtualDelegate, renderers, options);
}
- protected createMouseController(options: ITreeNodeListOptions): MouseController> {
+ protected override createMouseController(options: ITreeNodeListOptions): MouseController> {
return new TreeNodeListMouseController(this, options.tree);
}
- splice(start: number, deleteCount: number, elements: ITreeNode[] = []): void {
+ override splice(start: number, deleteCount: number, elements: ITreeNode[] = []): void {
super.splice(start, deleteCount, elements);
if (elements.length === 0) {
@@ -1217,7 +1217,7 @@ class TreeNodeList extends List>
}
}
- setFocus(indexes: number[], browserEvent?: UIEvent, fromAPI = false): void {
+ override setFocus(indexes: number[], browserEvent?: UIEvent, fromAPI = false): void {
super.setFocus(indexes, browserEvent);
if (!fromAPI) {
@@ -1225,7 +1225,7 @@ class TreeNodeList extends List>
}
}
- setSelection(indexes: number[], browserEvent?: UIEvent, fromAPI = false): void {
+ override setSelection(indexes: number[], browserEvent?: UIEvent, fromAPI = false): void {
super.setSelection(indexes, browserEvent);
if (!fromAPI) {
@@ -1233,7 +1233,7 @@ class TreeNodeList extends List>
}
}
- setAnchor(index: number | undefined, fromAPI = false): void {
+ override setAnchor(index: number | undefined, fromAPI = false): void {
super.setAnchor(index);
if (!fromAPI) {
diff --git a/src/vs/base/browser/ui/tree/asyncDataTree.ts b/src/vs/base/browser/ui/tree/asyncDataTree.ts
index 731137df0a6..c9be6ae0d68 100644
--- a/src/vs/base/browser/ui/tree/asyncDataTree.ts
+++ b/src/vs/base/browser/ui/tree/asyncDataTree.ts
@@ -157,11 +157,11 @@ function asTreeContextMenuEvent(e: ITreeContextMenuEvent extends ElementsDragAndDropData {
- set context(context: TContext | undefined) {
+ override set context(context: TContext | undefined) {
this.data.context = context;
}
- get context(): TContext | undefined {
+ override get context(): TContext | undefined {
return this.data.context;
}
@@ -1148,7 +1148,7 @@ export class CompressibleAsyncDataTree extends As
this.filter = options.filter;
}
- protected createTree(
+ protected override createTree(
user: string,
container: HTMLElement,
delegate: IListVirtualDelegate,
@@ -1162,18 +1162,18 @@ export class CompressibleAsyncDataTree extends As
return new CompressibleObjectTree(user, container, objectTreeDelegate, objectTreeRenderers, objectTreeOptions);
}
- protected asTreeElement(node: IAsyncDataTreeNode, viewStateContext?: IAsyncDataTreeViewStateContext): ICompressedTreeElement> {
+ protected override asTreeElement(node: IAsyncDataTreeNode, viewStateContext?: IAsyncDataTreeViewStateContext): ICompressedTreeElement> {
return {
incompressible: this.compressionDelegate.isIncompressible(node.element as T),
...super.asTreeElement(node, viewStateContext)
};
}
- updateOptions(options: ICompressibleAsyncDataTreeOptionsUpdate = {}): void {
+ override updateOptions(options: ICompressibleAsyncDataTreeOptionsUpdate = {}): void {
this.tree.updateOptions(options);
}
- getViewState(): IAsyncDataTreeViewState {
+ override getViewState(): IAsyncDataTreeViewState {
if (!this.identityProvider) {
throw new TreeError(this.user, 'Can\'t get tree view state without an identity provider');
}
@@ -1201,7 +1201,7 @@ export class CompressibleAsyncDataTree extends As
return { focus, selection, expanded, scrollTop: this.scrollTop };
}
- protected render(node: IAsyncDataTreeNode, viewStateContext?: IAsyncDataTreeViewStateContext): void {
+ protected override render(node: IAsyncDataTreeNode, viewStateContext?: IAsyncDataTreeViewStateContext): void {
if (!this.identityProvider) {
return super.render(node, viewStateContext);
}
@@ -1277,7 +1277,7 @@ export class CompressibleAsyncDataTree extends As
// For compressed async data trees, `TreeVisibility.Recurse` doesn't currently work
// and we have to filter everything beforehand
// Related to #85193 and #85835
- protected processChildren(children: Iterable): Iterable {
+ protected override processChildren(children: Iterable): Iterable {
if (this.filter) {
children = Iterable.filter(children, e => {
const result = this.filter!.filter(e, TreeVisibility.Visible);
diff --git a/src/vs/base/browser/ui/tree/objectTree.ts b/src/vs/base/browser/ui/tree/objectTree.ts
index 21e65570132..0e417939857 100644
--- a/src/vs/base/browser/ui/tree/objectTree.ts
+++ b/src/vs/base/browser/ui/tree/objectTree.ts
@@ -40,7 +40,7 @@ export class ObjectTree, TFilterData = void> extends
protected model!: IObjectTreeModel;
- get onDidChangeCollapseState(): Event> { return this.model.onDidChangeCollapseState; }
+ override get onDidChangeCollapseState(): Event> { return this.model.onDidChangeCollapseState; }
constructor(
user: string,
@@ -208,15 +208,15 @@ export class CompressibleObjectTree, TFilterData = vo
super(user, container, delegate, compressibleRenderers, asObjectTreeOptions(compressedTreeNodeProvider, options));
}
- setChildren(element: T | null, children: Iterable> = Iterable.empty(), options?: IObjectTreeSetChildrenOptions): void {
+ override setChildren(element: T | null, children: Iterable> = Iterable.empty(), options?: IObjectTreeSetChildrenOptions): void {
this.model.setChildren(element, children, options);
}
- protected createModel(user: string, view: IList>, options: ICompressibleObjectTreeOptions): ITreeModel {
+ protected override createModel(user: string, view: IList>, options: ICompressibleObjectTreeOptions): ITreeModel {
return new CompressibleObjectTreeModel(user, view, options);
}
- updateOptions(optionsUpdate: ICompressibleObjectTreeOptionsUpdate = {}): void {
+ override updateOptions(optionsUpdate: ICompressibleObjectTreeOptionsUpdate = {}): void {
super.updateOptions(optionsUpdate);
if (typeof optionsUpdate.compressionEnabled !== 'undefined') {
diff --git a/src/vs/base/browser/ui/tree/treeDefaults.ts b/src/vs/base/browser/ui/tree/treeDefaults.ts
index 834ab449fdd..155d88b8804 100644
--- a/src/vs/base/browser/ui/tree/treeDefaults.ts
+++ b/src/vs/base/browser/ui/tree/treeDefaults.ts
@@ -13,7 +13,7 @@ export class CollapseAllAction extends Action {
super('vs.tree.collapse', nls.localize('collapse all', "Collapse All"), 'collapse-all', enabled);
}
- async run(): Promise {
+ async override run(): Promise {
this.viewer.collapseAll();
this.viewer.setSelection([]);
this.viewer.setFocus([]);
diff --git a/src/vs/base/common/async.ts b/src/vs/base/common/async.ts
index 03fb0d84bc4..7fdec199755 100644
--- a/src/vs/base/common/async.ts
+++ b/src/vs/base/common/async.ts
@@ -381,7 +381,7 @@ export class AutoOpenBarrier extends Barrier {
this._timeout = setTimeout(() => this.open(), autoOpenTimeMs);
}
- open(): void {
+ override open(): void {
clearTimeout(this._timeout);
super.open();
}
@@ -772,7 +772,7 @@ export class RunOnceWorker extends RunOnceScheduler {
}
}
- protected doRun(): void {
+ protected override doRun(): void {
const units = this.units;
this.units = [];
@@ -781,7 +781,7 @@ export class RunOnceWorker extends RunOnceScheduler {
}
}
- dispose(): void {
+ override dispose(): void {
this.units = [];
super.dispose();
diff --git a/src/vs/base/common/event.ts b/src/vs/base/common/event.ts
index add858d259c..8b4efda9d44 100644
--- a/src/vs/base/common/event.ts
+++ b/src/vs/base/common/event.ts
@@ -674,7 +674,7 @@ export class PauseableEmitter extends Emitter {
}
}
- fire(event: T): void {
+ override fire(event: T): void {
if (this._listeners) {
if (this._isPaused !== 0) {
this._eventQueue.push(event);
diff --git a/src/vs/base/common/map.ts b/src/vs/base/common/map.ts
index 0e08876e537..2ca5fe50f9a 100644
--- a/src/vs/base/common/map.ts
+++ b/src/vs/base/common/map.ts
@@ -1028,7 +1028,7 @@ export class LRUCache extends LinkedMap {
this.checkTrim();
}
- get(key: K, touch: Touch = Touch.AsNew): V | undefined {
+ override get(key: K, touch: Touch = Touch.AsNew): V | undefined {
return super.get(key, touch);
}
@@ -1036,7 +1036,7 @@ export class LRUCache extends LinkedMap {
return super.get(key, Touch.None);
}
- set(key: K, value: V): this {
+ override set(key: K, value: V): this {
super.set(key, value, Touch.AsNew);
this.checkTrim();
return this;
diff --git a/src/vs/base/common/network.ts b/src/vs/base/common/network.ts
index 31ee24034ec..cff5d95d3d6 100644
--- a/src/vs/base/common/network.ts
+++ b/src/vs/base/common/network.ts
@@ -73,11 +73,6 @@ export namespace Schemas {
*/
export const vscodeWebview = 'vscode-webview';
- /**
- * Scheme used for loading resources inside of webviews.
- */
- export const vscodeWebviewResource = 'vscode-webview-resource';
-
/**
* Scheme used for extension pages
*/
diff --git a/src/vs/base/common/scrollable.ts b/src/vs/base/common/scrollable.ts
index 73a8622890e..cb2f7860bf6 100644
--- a/src/vs/base/common/scrollable.ts
+++ b/src/vs/base/common/scrollable.ts
@@ -223,7 +223,7 @@ export class Scrollable extends Disposable {
this._smoothScrolling = null;
}
- public dispose(): void {
+ public override dispose(): void {
if (this._smoothScrolling) {
this._smoothScrolling.dispose();
this._smoothScrolling = null;
diff --git a/src/vs/base/common/uri.ts b/src/vs/base/common/uri.ts
index 55a39c0f372..c2b0b02eb86 100644
--- a/src/vs/base/common/uri.ts
+++ b/src/vs/base/common/uri.ts
@@ -418,14 +418,14 @@ class Uri extends URI {
_formatted: string | null = null;
_fsPath: string | null = null;
- get fsPath(): string {
+ override get fsPath(): string {
if (!this._fsPath) {
this._fsPath = uriToFsPath(this, false);
}
return this._fsPath;
}
- toString(skipEncoding: boolean = false): string {
+ override toString(skipEncoding: boolean = false): string {
if (!skipEncoding) {
if (!this._formatted) {
this._formatted = _asFormatted(this, false);
@@ -437,7 +437,7 @@ class Uri extends URI {
}
}
- toJSON(): UriComponents {
+ override toJSON(): UriComponents {
const res = {
$mid: 1
};
diff --git a/src/vs/base/node/processes.ts b/src/vs/base/node/processes.ts
index 0273af2365c..363dc229f62 100644
--- a/src/vs/base/node/processes.ts
+++ b/src/vs/base/node/processes.ts
@@ -379,7 +379,7 @@ export class LineProcess extends AbstractProcess {
this.stderrLineDecoder = stderrLineDecoder;
}
- protected handleClose(data: any, cc: ValueCallback, pp: ProgressCallback, ee: ErrorCallback): void {
+ protected override handleClose(data: any, cc: ValueCallback, pp: ProgressCallback, ee: ErrorCallback): void {
const stdoutLine = this.stdoutLineDecoder ? this.stdoutLineDecoder.end() : null;
if (stdoutLine) {
pp({ line: stdoutLine, source: Source.stdout });
diff --git a/src/vs/base/parts/ipc/common/ipc.mp.ts b/src/vs/base/parts/ipc/common/ipc.mp.ts
index 01c36e6351d..1022eb173c4 100644
--- a/src/vs/base/parts/ipc/common/ipc.mp.ts
+++ b/src/vs/base/parts/ipc/common/ipc.mp.ts
@@ -72,7 +72,7 @@ export class Client extends IPCClient implements IDisposable {
this.protocol = protocol;
}
- dispose(): void {
+ override dispose(): void {
this.protocol.disconnect();
}
}
diff --git a/src/vs/base/parts/ipc/common/ipc.net.ts b/src/vs/base/parts/ipc/common/ipc.net.ts
index 777d4379f43..aad4fb4e92a 100644
--- a/src/vs/base/parts/ipc/common/ipc.net.ts
+++ b/src/vs/base/parts/ipc/common/ipc.net.ts
@@ -252,7 +252,7 @@ class ProtocolReader extends Disposable {
return this._incomingData.read(this._incomingData.byteLength);
}
- public dispose(): void {
+ public override dispose(): void {
this._isDisposed = true;
super.dispose();
}
@@ -412,7 +412,7 @@ export class Client extends IPCClient {
super(protocol, id, ipcLogger);
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
const socket = this.protocol.getSocket();
this.protocol.sendDisconnect();
diff --git a/src/vs/base/parts/ipc/electron-sandbox/ipc.electron.ts b/src/vs/base/parts/ipc/electron-sandbox/ipc.electron.ts
index f2eaa813112..94ece5a43be 100644
--- a/src/vs/base/parts/ipc/electron-sandbox/ipc.electron.ts
+++ b/src/vs/base/parts/ipc/electron-sandbox/ipc.electron.ts
@@ -32,7 +32,7 @@ export class Client extends IPCClient implements IDisposable {
this.protocol = protocol;
}
- dispose(): void {
+ override dispose(): void {
this.protocol.disconnect();
}
}
diff --git a/src/vs/base/parts/ipc/node/ipc.net.ts b/src/vs/base/parts/ipc/node/ipc.net.ts
index 09024a5fed1..aa2512c4272 100644
--- a/src/vs/base/parts/ipc/node/ipc.net.ts
+++ b/src/vs/base/parts/ipc/node/ipc.net.ts
@@ -266,7 +266,7 @@ export class WebSocketNodeSocket extends Disposable implements ISocket {
this._register(this.socket.onClose(() => this._onClose.fire()));
}
- public dispose(): void {
+ public override dispose(): void {
if (this._zlibDeflateFlushWaitingCount > 0) {
// Wait for any outstanding writes to finish before disposing
this._register(this._onDidZlibFlush.event(() => {
@@ -581,7 +581,7 @@ export class Server extends IPCServer {
this.server = server;
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
if (this.server) {
this.server.close();
diff --git a/src/vs/base/parts/ipc/test/common/ipc.test.ts b/src/vs/base/parts/ipc/test/common/ipc.test.ts
index f9855abda48..38ce85a7a0c 100644
--- a/src/vs/base/parts/ipc/test/common/ipc.test.ts
+++ b/src/vs/base/parts/ipc/test/common/ipc.test.ts
@@ -66,7 +66,7 @@ class TestIPCClient extends IPCClient {
super(protocol, id);
}
- dispose(): void {
+ override dispose(): void {
this._onDidDisconnect.fire();
super.dispose();
}
diff --git a/src/vs/base/parts/quickinput/browser/media/quickInput.css b/src/vs/base/parts/quickinput/browser/media/quickInput.css
index db7cdb89b0a..341495c9ec4 100644
--- a/src/vs/base/parts/quickinput/browser/media/quickInput.css
+++ b/src/vs/base/parts/quickinput/browser/media/quickInput.css
@@ -20,6 +20,7 @@
display: flex;
margin-left: 4px;
flex: 1;
+ height: 24px;
}
.quick-input-left-action-bar.monaco-action-bar .actions-container {
@@ -35,12 +36,10 @@
display: flex;
margin-right: 4px;
flex: 1;
+ height: 24px;
}
.quick-input-titlebar .monaco-action-bar .action-label.codicon {
- margin: 0;
- width: 19px;
- height: 100%;
background-position: center;
background-repeat: no-repeat;
}
@@ -260,10 +259,8 @@
}
.quick-input-list .quick-input-list-entry-action-bar .action-label.codicon {
- margin: 0;
- height: 100%;
- padding: 0 2px;
- vertical-align: middle;
+ margin-right: 4px;
+ padding: 2px;
}
.quick-input-list .quick-input-list-entry-action-bar {
@@ -274,10 +271,6 @@
margin-right: 4px; /* separate from scrollbar */
}
-.quick-input-list .quick-input-list-entry-action-bar .action-label.codicon {
- margin-right: 4px; /* separate actions */
-}
-
.quick-input-list .quick-input-list-entry .quick-input-list-entry-action-bar .action-label.always-visible,
.quick-input-list .quick-input-list-entry:hover .quick-input-list-entry-action-bar .action-label,
.quick-input-list .monaco-list-row.focused .quick-input-list-entry-action-bar .action-label {
diff --git a/src/vs/base/parts/quickinput/browser/quickInput.ts b/src/vs/base/parts/quickinput/browser/quickInput.ts
index f84ec1479c3..52d7088de07 100644
--- a/src/vs/base/parts/quickinput/browser/quickInput.ts
+++ b/src/vs/base/parts/quickinput/browser/quickInput.ts
@@ -410,7 +410,7 @@ class QuickInput extends Disposable implements IQuickInput {
readonly onDispose = this.onDisposeEmitter.event;
- dispose(): void {
+ override dispose(): void {
this.hide();
this.onDisposeEmitter.fire();
@@ -694,7 +694,7 @@ class QuickPick extends QuickInput implements IQuickPi
}
}
- show() {
+ override show() {
if (!this.visible) {
this.visibleDisposables.add(
this.ui.inputBox.onDidChange(value => {
@@ -884,7 +884,7 @@ class QuickPick extends QuickInput implements IQuickPi
});
}
- protected update() {
+ protected override update() {
if (!this.visible) {
return;
}
@@ -1063,7 +1063,7 @@ class InputBox extends QuickInput implements IInputBox {
readonly onDidAccept = this.onDidAcceptEmitter.event;
- show() {
+ override show() {
if (!this.visible) {
this.visibleDisposables.add(
this.ui.inputBox.onDidChange(value => {
@@ -1079,7 +1079,7 @@ class InputBox extends QuickInput implements IInputBox {
super.show();
}
- protected update() {
+ protected override update() {
if (!this.visible) {
return;
}
@@ -1388,8 +1388,12 @@ export class QuickInputController extends Disposable {
const index = input.items.indexOf(event.item);
if (index !== -1) {
const items = input.items.slice();
- items.splice(index, 1);
+ const removed = items.splice(index, 1);
+ const activeItems = input.activeItems.filter((ai) => ai !== removed[0]);
input.items = items;
+ if (activeItems) {
+ input.activeItems = activeItems;
+ }
}
}
})),
diff --git a/src/vs/base/parts/storage/common/storage.ts b/src/vs/base/parts/storage/common/storage.ts
index a2c28e17070..ff52a44983b 100644
--- a/src/vs/base/parts/storage/common/storage.ts
+++ b/src/vs/base/parts/storage/common/storage.ts
@@ -320,7 +320,7 @@ export class Storage extends Disposable implements IStorage {
return new Promise(resolve => this.whenFlushedCallbacks.push(resolve));
}
- dispose(): void {
+ override dispose(): void {
this.flushDelayer.cancel(); // workaround https://github.com/microsoft/vscode/issues/116777
this.flushDelayer.dispose();
diff --git a/src/vs/base/parts/storage/test/node/storage.test.ts b/src/vs/base/parts/storage/test/node/storage.test.ts
index 990e531905e..fd963590dc4 100644
--- a/src/vs/base/parts/storage/test/node/storage.test.ts
+++ b/src/vs/base/parts/storage/test/node/storage.test.ts
@@ -111,7 +111,7 @@ flakySuite('Storage Library', function () {
class TestSQLiteStorageDatabase extends SQLiteStorageDatabase {
private readonly _onDidChangeItemsExternal = new Emitter();
- get onDidChangeItemsExternal(): Event { return this._onDidChangeItemsExternal.event; }
+ override get onDidChangeItemsExternal(): Event { return this._onDidChangeItemsExternal.event; }
fireDidChangeItemsExternal(event: IStorageItemsChangeEvent): void {
this._onDidChangeItemsExternal.fire(event);
diff --git a/src/vs/code/electron-main/app.ts b/src/vs/code/electron-main/app.ts
index 761d20f3786..a34f03523b7 100644
--- a/src/vs/code/electron-main/app.ts
+++ b/src/vs/code/electron-main/app.ts
@@ -93,6 +93,7 @@ import { ISignService } from 'vs/platform/sign/common/sign';
* even if the user starts many instances (e.g. from the command line).
*/
export class CodeApplication extends Disposable {
+
private windowsMainService: IWindowsMainService | undefined;
private nativeHostMainService: INativeHostMainService | undefined;
diff --git a/src/vs/code/electron-main/main.ts b/src/vs/code/electron-main/main.ts
index 9f66c45a146..a2687622d9d 100644
--- a/src/vs/code/electron-main/main.ts
+++ b/src/vs/code/electron-main/main.ts
@@ -8,6 +8,7 @@ import { app, dialog } from 'electron';
import { promises, unlinkSync } from 'fs';
import { localize } from 'vs/nls';
import { isWindows, IProcessEnvironment, isMacintosh } from 'vs/base/common/platform';
+import { mark } from 'vs/base/common/performance';
import product from 'vs/platform/product/common/product';
import { parseMainProcessArgv, addArg } from 'vs/platform/environment/node/argvHelper';
import { createWaitMarkerFile } from 'vs/platform/environment/node/wait';
@@ -107,7 +108,7 @@ class CodeMain {
// Create the main IPC server by trying to be the server
// If this throws an error it means we are not the first
// instance of VS Code running and so we would quit.
- const mainProcessNodeIpcServer = await this.doStartup(logService, environmentService, lifecycleMainService, instantiationService, productService, true);
+ const mainProcessNodeIpcServer = await this.claimInstance(logService, environmentService, lifecycleMainService, instantiationService, productService, true);
// Delay creation of spdlog for perf reasons (https://github.com/microsoft/vscode/issues/72906)
bufferLogService.logger = new SpdLogLogger('main', join(environmentService.logsPath, 'main.log'), true, bufferLogService.getLevel());
@@ -221,14 +222,16 @@ class CodeMain {
return Promise.all([environmentServiceInitialization, configurationServiceInitialization, stateServiceInitialization]);
}
- private async doStartup(logService: ILogService, environmentMainService: IEnvironmentMainService, lifecycleMainService: ILifecycleMainService, instantiationService: IInstantiationService, productService: IProductService, retry: boolean): Promise {
+ private async claimInstance(logService: ILogService, environmentMainService: IEnvironmentMainService, lifecycleMainService: ILifecycleMainService, instantiationService: IInstantiationService, productService: IProductService, retry: boolean): Promise {
// Try to setup a server for running. If that succeeds it means
// we are the first instance to startup. Otherwise it is likely
// that another instance is already running.
let mainProcessNodeIpcServer: NodeIPCServer;
try {
+ mark('code/willStartMainServer');
mainProcessNodeIpcServer = await nodeIPCServe(environmentMainService.mainIPCHandle);
+ mark('code/didStartMainServer');
once(lifecycleMainService.onWillShutdown)(() => mainProcessNodeIpcServer.dispose());
} catch (error) {
@@ -273,7 +276,7 @@ class CodeMain {
throw error;
}
- return this.doStartup(logService, environmentMainService, lifecycleMainService, instantiationService, productService, false);
+ return this.claimInstance(logService, environmentMainService, lifecycleMainService, instantiationService, productService, false);
}
// Tests from CLI require to be the only instance currently
diff --git a/src/vs/editor/browser/config/configuration.ts b/src/vs/editor/browser/config/configuration.ts
index 545d1321e7b..ddb077f57db 100644
--- a/src/vs/editor/browser/config/configuration.ts
+++ b/src/vs/editor/browser/config/configuration.ts
@@ -112,7 +112,7 @@ class CSSBasedConfiguration extends Disposable {
this._evictUntrustedReadingsTimeout = -1;
}
- public dispose(): void {
+ public override dispose(): void {
if (this._evictUntrustedReadingsTimeout !== -1) {
clearTimeout(this._evictUntrustedReadingsTimeout);
this._evictUntrustedReadingsTimeout = -1;
@@ -353,11 +353,11 @@ export class Configuration extends CommonEditorConfiguration {
this._recomputeOptions();
}
- public observeReferenceElement(dimension?: IDimension): void {
+ public override observeReferenceElement(dimension?: IDimension): void {
this._elementSizeObserver.observe(dimension);
}
- public updatePixelRatio(): void {
+ public override updatePixelRatio(): void {
this._recomputeOptions();
}
diff --git a/src/vs/editor/browser/config/elementSizeObserver.ts b/src/vs/editor/browser/config/elementSizeObserver.ts
index 06b42c6f9c6..cf42a075793 100644
--- a/src/vs/editor/browser/config/elementSizeObserver.ts
+++ b/src/vs/editor/browser/config/elementSizeObserver.ts
@@ -52,7 +52,7 @@ export class ElementSizeObserver extends Disposable {
this.measureReferenceDomElement(false, dimension);
}
- public dispose(): void {
+ public override dispose(): void {
this.stopObserving();
super.dispose();
}
diff --git a/src/vs/editor/browser/controller/mouseHandler.ts b/src/vs/editor/browser/controller/mouseHandler.ts
index 005fe7ad0ca..0b26756bc9f 100644
--- a/src/vs/editor/browser/controller/mouseHandler.ts
+++ b/src/vs/editor/browser/controller/mouseHandler.ts
@@ -133,13 +133,13 @@ export class MouseHandler extends ViewEventHandler {
this._context.addEventHandler(this);
}
- public dispose(): void {
+ public override dispose(): void {
this._context.removeEventHandler(this);
super.dispose();
}
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
if (e.hasChanged(EditorOption.layoutInfo)) {
// layout change
const height = this._context.configuration.options.get(EditorOption.layoutInfo).height;
@@ -150,14 +150,14 @@ export class MouseHandler extends ViewEventHandler {
}
return false;
}
- public onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
+ public override onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
this._mouseDownOperation.onCursorStateChanged(e);
return false;
}
- public onFocusChanged(e: viewEvents.ViewFocusChangedEvent): boolean {
+ public override onFocusChanged(e: viewEvents.ViewFocusChangedEvent): boolean {
return false;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
this._mouseDownOperation.onScrollChanged();
return false;
}
@@ -311,7 +311,7 @@ class MouseDownOperation extends Disposable {
this._lastMouseEvent = null;
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
diff --git a/src/vs/editor/browser/controller/mouseTarget.ts b/src/vs/editor/browser/controller/mouseTarget.ts
index bc0478ca458..d2af49b222f 100644
--- a/src/vs/editor/browser/controller/mouseTarget.ts
+++ b/src/vs/editor/browser/controller/mouseTarget.ts
@@ -422,7 +422,7 @@ class HitTestRequest extends BareHitTestRequest {
}
}
- public toString(): string {
+ public override toString(): string {
return `pos(${this.pos.x},${this.pos.y}), editorPos(${this.editorPos.x},${this.editorPos.y}), mouseVerticalOffset: ${this.mouseVerticalOffset}, mouseContentHorizontalOffset: ${this.mouseContentHorizontalOffset}\n\ttarget: ${this.target ? (this.target).outerHTML : null}`;
}
diff --git a/src/vs/editor/browser/controller/pointerHandler.ts b/src/vs/editor/browser/controller/pointerHandler.ts
index ee291a86fcb..5c4df88ff3b 100644
--- a/src/vs/editor/browser/controller/pointerHandler.ts
+++ b/src/vs/editor/browser/controller/pointerHandler.ts
@@ -87,7 +87,7 @@ export class PointerEventHandler extends MouseHandler {
}
}
- public _onMouseDown(e: EditorMouseEvent): void {
+ public override _onMouseDown(e: EditorMouseEvent): void {
if ((e.browserEvent as any).pointerType === 'touch') {
return;
}
diff --git a/src/vs/editor/browser/controller/textAreaHandler.ts b/src/vs/editor/browser/controller/textAreaHandler.ts
index be84e6511c0..fe41979c3a9 100644
--- a/src/vs/editor/browser/controller/textAreaHandler.ts
+++ b/src/vs/editor/browser/controller/textAreaHandler.ts
@@ -330,7 +330,7 @@ export class TextAreaHandler extends ViewPart {
}));
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
@@ -425,7 +425,7 @@ export class TextAreaHandler extends ViewPart {
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
const layoutInfo = options.get(EditorOption.layoutInfo);
@@ -454,34 +454,34 @@ export class TextAreaHandler extends ViewPart {
return true;
}
- public onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
+ public override onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
this._selections = e.selections.slice(0);
this._modelSelections = e.modelSelections.slice(0);
this._textAreaInput.writeScreenReaderContent('selection changed');
return true;
}
- public onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
+ public override onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
// true for inline decorations that can end up relayouting text
return true;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
return true;
}
- public onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
+ public override onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
return true;
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
return true;
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
this._scrollLeft = e.scrollLeft;
this._scrollTop = e.scrollTop;
return true;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return true;
}
diff --git a/src/vs/editor/browser/controller/textAreaInput.ts b/src/vs/editor/browser/controller/textAreaInput.ts
index 63722bb873a..37dbdb7f67b 100644
--- a/src/vs/editor/browser/controller/textAreaInput.ts
+++ b/src/vs/editor/browser/controller/textAreaInput.ts
@@ -524,7 +524,7 @@ export class TextAreaInput extends Disposable {
});
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
if (this._selectionChangeListener) {
this._selectionChangeListener.dispose();
diff --git a/src/vs/editor/browser/core/editorState.ts b/src/vs/editor/browser/core/editorState.ts
index 5d4c1ffb0e5..36014981103 100644
--- a/src/vs/editor/browser/core/editorState.ts
+++ b/src/vs/editor/browser/core/editorState.ts
@@ -119,7 +119,7 @@ export class EditorStateCancellationTokenSource extends EditorKeybindingCancella
}
}
- dispose() {
+ override dispose() {
this._listener.dispose();
super.dispose();
}
@@ -137,7 +137,7 @@ export class TextModelCancellationTokenSource extends CancellationTokenSource im
this._listener = model.onDidChangeContent(() => this.cancel());
}
- dispose() {
+ override dispose() {
this._listener.dispose();
super.dispose();
}
diff --git a/src/vs/editor/browser/core/keybindingCancellation.ts b/src/vs/editor/browser/core/keybindingCancellation.ts
index 206fbeac885..2fc418a32a4 100644
--- a/src/vs/editor/browser/core/keybindingCancellation.ts
+++ b/src/vs/editor/browser/core/keybindingCancellation.ts
@@ -81,7 +81,7 @@ export class EditorKeybindingCancellationTokenSource extends CancellationTokenSo
this._unregister = editor.invokeWithinContext(accessor => accessor.get(IEditorCancellationTokens).add(editor, this));
}
- dispose(): void {
+ override dispose(): void {
this._unregister();
super.dispose();
}
diff --git a/src/vs/editor/browser/services/bulkEditService.ts b/src/vs/editor/browser/services/bulkEditService.ts
index 5a49c330f4c..340cd1c28ea 100644
--- a/src/vs/editor/browser/services/bulkEditService.ts
+++ b/src/vs/editor/browser/services/bulkEditService.ts
@@ -47,7 +47,7 @@ export class ResourceTextEdit extends ResourceEdit {
readonly resource: URI,
readonly textEdit: TextEdit,
readonly versionId?: number,
- readonly metadata?: WorkspaceEditMetadata
+ metadata?: WorkspaceEditMetadata
) {
super(metadata);
}
@@ -58,7 +58,7 @@ export class ResourceFileEdit extends ResourceEdit {
readonly oldResource: URI | undefined,
readonly newResource: URI | undefined,
readonly options?: WorkspaceFileEditOptions,
- readonly metadata?: WorkspaceEditMetadata
+ metadata?: WorkspaceEditMetadata
) {
super(metadata);
}
diff --git a/src/vs/editor/browser/view/viewImpl.ts b/src/vs/editor/browser/view/viewImpl.ts
index 5589fd7f120..67823047296 100644
--- a/src/vs/editor/browser/view/viewImpl.ts
+++ b/src/vs/editor/browser/view/viewImpl.ts
@@ -300,32 +300,32 @@ export class View extends ViewEventHandler {
}
// --- begin event handlers
- public handleEvents(events: viewEvents.ViewEvent[]): void {
+ public override handleEvents(events: viewEvents.ViewEvent[]): void {
super.handleEvents(events);
this._scheduleRender();
}
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
this._configPixelRatio = this._context.configuration.options.get(EditorOption.pixelRatio);
this.domNode.setClassName(this._getEditorClassName());
this._applyLayout();
return false;
}
- public onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
+ public override onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
this._selections = e.selections;
return false;
}
- public onFocusChanged(e: viewEvents.ViewFocusChangedEvent): boolean {
+ public override onFocusChanged(e: viewEvents.ViewFocusChangedEvent): boolean {
this.domNode.setClassName(this._getEditorClassName());
return false;
}
- public onThemeChanged(e: viewEvents.ViewThemeChangedEvent): boolean {
+ public override onThemeChanged(e: viewEvents.ViewThemeChangedEvent): boolean {
this.domNode.setClassName(this._getEditorClassName());
return false;
}
// --- end event handlers
- public dispose(): void {
+ public override dispose(): void {
if (this._renderAnimationFrame !== null) {
this._renderAnimationFrame.dispose();
this._renderAnimationFrame = null;
diff --git a/src/vs/editor/browser/view/viewOverlays.ts b/src/vs/editor/browser/view/viewOverlays.ts
index 130f1f09fae..0088f3d8b46 100644
--- a/src/vs/editor/browser/view/viewOverlays.ts
+++ b/src/vs/editor/browser/view/viewOverlays.ts
@@ -36,7 +36,7 @@ export class ViewOverlays extends ViewPart implements IVisibleLinesHost onBrowserDesperateReveal(this.scrollbarDomNode.domNode, true, false)));
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
@@ -133,7 +133,7 @@ export class EditorScrollbar extends ViewPart {
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
if (
e.hasChanged(EditorOption.scrollbar)
|| e.hasChanged(EditorOption.mouseWheelScrollSensitivity)
@@ -157,10 +157,10 @@ export class EditorScrollbar extends ViewPart {
}
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return true;
}
- public onThemeChanged(e: viewEvents.ViewThemeChangedEvent): boolean {
+ public override onThemeChanged(e: viewEvents.ViewThemeChangedEvent): boolean {
this.scrollbar.updateClassName('editor-scrollable' + ' ' + getThemeTypeSelector(this._context.theme.type));
return true;
}
diff --git a/src/vs/editor/browser/viewParts/glyphMargin/glyphMargin.ts b/src/vs/editor/browser/viewParts/glyphMargin/glyphMargin.ts
index 0cdd9c61cc2..3ccec5f60e3 100644
--- a/src/vs/editor/browser/viewParts/glyphMargin/glyphMargin.ts
+++ b/src/vs/editor/browser/viewParts/glyphMargin/glyphMargin.ts
@@ -98,7 +98,7 @@ export class GlyphMarginOverlay extends DedupOverlay {
this._context.addEventHandler(this);
}
- public dispose(): void {
+ public override dispose(): void {
this._context.removeEventHandler(this);
this._renderResult = null;
super.dispose();
@@ -106,7 +106,7 @@ export class GlyphMarginOverlay extends DedupOverlay {
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
const layoutInfo = options.get(EditorOption.layoutInfo);
@@ -116,25 +116,25 @@ export class GlyphMarginOverlay extends DedupOverlay {
this._glyphMarginWidth = layoutInfo.glyphMarginWidth;
return true;
}
- public onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
+ public override onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
return true;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
return true;
}
- public onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
+ public override onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
return true;
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
return true;
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return e.scrollTopChanged;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return true;
}
diff --git a/src/vs/editor/browser/viewParts/indentGuides/indentGuides.ts b/src/vs/editor/browser/viewParts/indentGuides/indentGuides.ts
index f94c06247b4..de40ce80870 100644
--- a/src/vs/editor/browser/viewParts/indentGuides/indentGuides.ts
+++ b/src/vs/editor/browser/viewParts/indentGuides/indentGuides.ts
@@ -45,7 +45,7 @@ export class IndentGuidesOverlay extends DynamicViewOverlay {
this._context.addEventHandler(this);
}
- public dispose(): void {
+ public override dispose(): void {
this._context.removeEventHandler(this);
this._renderResult = null;
super.dispose();
@@ -53,7 +53,7 @@ export class IndentGuidesOverlay extends DynamicViewOverlay {
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
const wrappingInfo = options.get(EditorOption.wrappingInfo);
const fontInfo = options.get(EditorOption.fontInfo);
@@ -65,7 +65,7 @@ export class IndentGuidesOverlay extends DynamicViewOverlay {
this._maxIndentLeft = wrappingInfo.wrappingColumn === -1 ? -1 : (wrappingInfo.wrappingColumn * fontInfo.typicalHalfwidthCharacterWidth);
return true;
}
- public onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
+ public override onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
const selection = e.selections[0];
const newPrimaryLineNumber = selection.isEmpty() ? selection.positionLineNumber : 0;
@@ -76,29 +76,29 @@ export class IndentGuidesOverlay extends DynamicViewOverlay {
return false;
}
- public onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
+ public override onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
// true for inline decorations
return true;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
return true;
}
- public onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
+ public override onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
return true;
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
return true;
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return e.scrollTopChanged;// || e.scrollWidthChanged;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return true;
}
- public onLanguageConfigurationChanged(e: viewEvents.ViewLanguageConfigurationEvent): boolean {
+ public override onLanguageConfigurationChanged(e: viewEvents.ViewLanguageConfigurationEvent): boolean {
return true;
}
diff --git a/src/vs/editor/browser/viewParts/lineNumbers/lineNumbers.ts b/src/vs/editor/browser/viewParts/lineNumbers/lineNumbers.ts
index 0eceebf7cf6..1c3f5d416d6 100644
--- a/src/vs/editor/browser/viewParts/lineNumbers/lineNumbers.ts
+++ b/src/vs/editor/browser/viewParts/lineNumbers/lineNumbers.ts
@@ -54,7 +54,7 @@ export class LineNumbersOverlay extends DynamicViewOverlay {
this._lineNumbersWidth = layoutInfo.lineNumbersWidth;
}
- public dispose(): void {
+ public override dispose(): void {
this._context.removeEventHandler(this);
this._renderResult = null;
super.dispose();
@@ -62,11 +62,11 @@ export class LineNumbersOverlay extends DynamicViewOverlay {
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
this._readConfig();
return true;
}
- public onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
+ public override onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
const primaryViewPosition = e.selections[0].getPosition();
this._lastCursorModelPosition = this._context.model.coordinatesConverter.convertViewPositionToModelPosition(primaryViewPosition);
@@ -80,22 +80,22 @@ export class LineNumbersOverlay extends DynamicViewOverlay {
}
return shouldRender;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
return true;
}
- public onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
+ public override onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
return true;
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
return true;
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return e.scrollTopChanged;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return true;
}
diff --git a/src/vs/editor/browser/viewParts/lines/viewLine.ts b/src/vs/editor/browser/viewParts/lines/viewLine.ts
index 0c73f13abed..edec7f7a041 100644
--- a/src/vs/editor/browser/viewParts/lines/viewLine.ts
+++ b/src/vs/editor/browser/viewParts/lines/viewLine.ts
@@ -662,7 +662,7 @@ class RenderedViewLine implements IRenderedViewLine {
}
class WebKitRenderedViewLine extends RenderedViewLine {
- protected _readVisibleRangesForRange(domNode: FastDomNode, startColumn: number, endColumn: number, context: DomReadingContext): HorizontalRange[] | null {
+ protected override _readVisibleRangesForRange(domNode: FastDomNode, startColumn: number, endColumn: number, context: DomReadingContext): HorizontalRange[] | null {
const output = super._readVisibleRangesForRange(domNode, startColumn, endColumn, context);
if (!output || output.length === 0 || startColumn === endColumn || (startColumn === 1 && endColumn === this._characterMapping.length)) {
diff --git a/src/vs/editor/browser/viewParts/lines/viewLines.ts b/src/vs/editor/browser/viewParts/lines/viewLines.ts
index 6e92c558e11..8d9601ff720 100644
--- a/src/vs/editor/browser/viewParts/lines/viewLines.ts
+++ b/src/vs/editor/browser/viewParts/lines/viewLines.ts
@@ -152,7 +152,7 @@ export class ViewLines extends ViewPart implements IVisibleLinesHost,
this._horizontalRevealRequest = null;
}
- public dispose(): void {
+ public override dispose(): void {
this._asyncUpdateLineWidths.dispose();
this._asyncCheckMonospaceFontAssumptions.dispose();
super.dispose();
@@ -172,7 +172,7 @@ export class ViewLines extends ViewPart implements IVisibleLinesHost,
// ---- begin view event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
this._visibleLines.onConfigurationChanged(e);
if (e.hasChanged(EditorOption.wrappingInfo)) {
this._maxLineWidth = 0;
@@ -217,7 +217,7 @@ export class ViewLines extends ViewPart implements IVisibleLinesHost,
return false;
}
- public onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
+ public override onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
const rendStartLineNumber = this._visibleLines.getStartLineNumber();
const rendEndLineNumber = this._visibleLines.getEndLineNumber();
let r = false;
@@ -226,7 +226,7 @@ export class ViewLines extends ViewPart implements IVisibleLinesHost,
}
return r;
}
- public onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
+ public override onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
if (true/*e.inlineDecorationsChanged*/) {
const rendStartLineNumber = this._visibleLines.getStartLineNumber();
const rendEndLineNumber = this._visibleLines.getEndLineNumber();
@@ -236,21 +236,21 @@ export class ViewLines extends ViewPart implements IVisibleLinesHost,
}
return true;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
const shouldRender = this._visibleLines.onFlushed(e);
this._maxLineWidth = 0;
return shouldRender;
}
- public onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
+ public override onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
return this._visibleLines.onLinesChanged(e);
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
return this._visibleLines.onLinesDeleted(e);
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
return this._visibleLines.onLinesInserted(e);
}
- public onRevealRangeRequest(e: viewEvents.ViewRevealRangeRequestEvent): boolean {
+ public override onRevealRangeRequest(e: viewEvents.ViewRevealRangeRequestEvent): boolean {
// Using the future viewport here in order to handle multiple
// incoming reveal range requests that might all desire to be animated
const desiredScrollTop = this._computeScrollTopToRevealRange(this._context.viewLayout.getFutureViewport(), e.source, e.range, e.selections, e.verticalType);
@@ -286,7 +286,7 @@ export class ViewLines extends ViewPart implements IVisibleLinesHost,
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
if (this._horizontalRevealRequest && e.scrollLeftChanged) {
// cancel any outstanding horizontal reveal request if someone else scrolls horizontally.
this._horizontalRevealRequest = null;
@@ -303,14 +303,14 @@ export class ViewLines extends ViewPart implements IVisibleLinesHost,
return this._visibleLines.onScrollChanged(e) || true;
}
- public onTokensChanged(e: viewEvents.ViewTokensChangedEvent): boolean {
+ public override onTokensChanged(e: viewEvents.ViewTokensChangedEvent): boolean {
return this._visibleLines.onTokensChanged(e);
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
this._context.model.setMaxLineWidth(this._maxLineWidth);
return this._visibleLines.onZonesChanged(e);
}
- public onThemeChanged(e: viewEvents.ViewThemeChangedEvent): boolean {
+ public override onThemeChanged(e: viewEvents.ViewThemeChangedEvent): boolean {
return this._onOptionsMaybeChanged();
}
diff --git a/src/vs/editor/browser/viewParts/linesDecorations/linesDecorations.ts b/src/vs/editor/browser/viewParts/linesDecorations/linesDecorations.ts
index 6e40faf77b4..bdc2eed9738 100644
--- a/src/vs/editor/browser/viewParts/linesDecorations/linesDecorations.ts
+++ b/src/vs/editor/browser/viewParts/linesDecorations/linesDecorations.ts
@@ -30,7 +30,7 @@ export class LinesDecorationsOverlay extends DedupOverlay {
this._context.addEventHandler(this);
}
- public dispose(): void {
+ public override dispose(): void {
this._context.removeEventHandler(this);
this._renderResult = null;
super.dispose();
@@ -38,32 +38,32 @@ export class LinesDecorationsOverlay extends DedupOverlay {
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
const layoutInfo = options.get(EditorOption.layoutInfo);
this._decorationsLeft = layoutInfo.decorationsLeft;
this._decorationsWidth = layoutInfo.decorationsWidth;
return true;
}
- public onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
+ public override onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
return true;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
return true;
}
- public onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
+ public override onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
return true;
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
return true;
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return e.scrollTopChanged;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return true;
}
diff --git a/src/vs/editor/browser/viewParts/margin/margin.ts b/src/vs/editor/browser/viewParts/margin/margin.ts
index 8181e2f8aa8..11ba874cbff 100644
--- a/src/vs/editor/browser/viewParts/margin/margin.ts
+++ b/src/vs/editor/browser/viewParts/margin/margin.ts
@@ -45,7 +45,7 @@ export class Margin extends ViewPart {
this._domNode.appendChild(this._glyphMarginBackgroundDomNode);
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
@@ -55,7 +55,7 @@ export class Margin extends ViewPart {
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
const layoutInfo = options.get(EditorOption.layoutInfo);
@@ -66,7 +66,7 @@ export class Margin extends ViewPart {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return super.onScrollChanged(e) || e.scrollTopChanged;
}
diff --git a/src/vs/editor/browser/viewParts/marginDecorations/marginDecorations.ts b/src/vs/editor/browser/viewParts/marginDecorations/marginDecorations.ts
index 7f5d18d8621..89a154d5dcd 100644
--- a/src/vs/editor/browser/viewParts/marginDecorations/marginDecorations.ts
+++ b/src/vs/editor/browser/viewParts/marginDecorations/marginDecorations.ts
@@ -20,7 +20,7 @@ export class MarginViewLineDecorationsOverlay extends DedupOverlay {
this._context.addEventHandler(this);
}
- public dispose(): void {
+ public override dispose(): void {
this._context.removeEventHandler(this);
this._renderResult = null;
super.dispose();
@@ -28,28 +28,28 @@ export class MarginViewLineDecorationsOverlay extends DedupOverlay {
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
return true;
}
- public onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
+ public override onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
return true;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
return true;
}
- public onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
+ public override onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
return true;
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
return true;
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return e.scrollTopChanged;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return true;
}
diff --git a/src/vs/editor/browser/viewParts/minimap/minimap.ts b/src/vs/editor/browser/viewParts/minimap/minimap.ts
index 348315fa9a8..0ea038e6b89 100644
--- a/src/vs/editor/browser/viewParts/minimap/minimap.ts
+++ b/src/vs/editor/browser/viewParts/minimap/minimap.ts
@@ -759,7 +759,7 @@ export class Minimap extends ViewPart implements IMinimapModel {
this._actual = new InnerMinimap(context.theme, this);
}
- public dispose(): void {
+ public override dispose(): void {
this._actual.dispose();
super.dispose();
}
@@ -781,27 +781,27 @@ export class Minimap extends ViewPart implements IMinimapModel {
// ---- begin view event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
return this._onOptionsMaybeChanged();
}
- public onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
+ public override onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
this._selections = e.selections;
this._minimapSelections = null;
return this._actual.onSelectionChanged();
}
- public onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
+ public override onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
if (e.affectsMinimap) {
return this._actual.onDecorationsChanged();
}
return false;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
if (this._samplingState) {
this._shouldCheckSampling = true;
}
return this._actual.onFlushed();
}
- public onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
+ public override onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
if (this._samplingState) {
const minimapLineRange = this._samplingState.modelLineRangeToMinimapLineRange(e.fromLineNumber, e.toLineNumber);
if (minimapLineRange) {
@@ -813,7 +813,7 @@ export class Minimap extends ViewPart implements IMinimapModel {
return this._actual.onLinesChanged(e.fromLineNumber, e.toLineNumber);
}
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
if (this._samplingState) {
const [changeStartIndex, changeEndIndex] = this._samplingState.onLinesDeleted(e);
if (changeStartIndex <= changeEndIndex) {
@@ -825,7 +825,7 @@ export class Minimap extends ViewPart implements IMinimapModel {
return this._actual.onLinesDeleted(e.fromLineNumber, e.toLineNumber);
}
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
if (this._samplingState) {
this._samplingState.onLinesInserted(e);
this._shouldCheckSampling = true;
@@ -834,16 +834,16 @@ export class Minimap extends ViewPart implements IMinimapModel {
return this._actual.onLinesInserted(e.fromLineNumber, e.toLineNumber);
}
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return this._actual.onScrollChanged();
}
- public onThemeChanged(e: viewEvents.ViewThemeChangedEvent): boolean {
+ public override onThemeChanged(e: viewEvents.ViewThemeChangedEvent): boolean {
this._context.model.invalidateMinimapColorCache();
this._actual.onThemeChanged();
this._onOptionsMaybeChanged();
return true;
}
- public onTokensChanged(e: viewEvents.ViewTokensChangedEvent): boolean {
+ public override onTokensChanged(e: viewEvents.ViewTokensChangedEvent): boolean {
if (this._samplingState) {
let ranges: { fromLineNumber: number; toLineNumber: number; }[] = [];
for (const range of e.ranges) {
@@ -861,11 +861,11 @@ export class Minimap extends ViewPart implements IMinimapModel {
return this._actual.onTokensChanged(e.ranges);
}
}
- public onTokensColorsChanged(e: viewEvents.ViewTokensColorsChangedEvent): boolean {
+ public override onTokensColorsChanged(e: viewEvents.ViewTokensColorsChangedEvent): boolean {
this._onOptionsMaybeChanged();
return this._actual.onTokensColorsChanged();
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return this._actual.onZonesChanged();
}
@@ -1211,7 +1211,7 @@ class InnerMinimap extends Disposable {
this._model.setScrollTop(scrollTop);
}
- public dispose(): void {
+ public override dispose(): void {
this._mouseDownListener.dispose();
this._sliderMouseMoveMonitor.dispose();
this._sliderMouseDownListener.dispose();
diff --git a/src/vs/editor/browser/viewParts/overlayWidgets/overlayWidgets.ts b/src/vs/editor/browser/viewParts/overlayWidgets/overlayWidgets.ts
index b51be8aaba8..ea3881e8a8a 100644
--- a/src/vs/editor/browser/viewParts/overlayWidgets/overlayWidgets.ts
+++ b/src/vs/editor/browser/viewParts/overlayWidgets/overlayWidgets.ts
@@ -52,7 +52,7 @@ export class ViewOverlayWidgets extends ViewPart {
this._domNode.setClassName('overlayWidgets');
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
this._widgets = {};
}
@@ -63,7 +63,7 @@ export class ViewOverlayWidgets extends ViewPart {
// ---- begin view event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
const layoutInfo = options.get(EditorOption.layoutInfo);
diff --git a/src/vs/editor/browser/viewParts/overviewRuler/decorationsOverviewRuler.ts b/src/vs/editor/browser/viewParts/overviewRuler/decorationsOverviewRuler.ts
index 287011babea..02cc65c127a 100644
--- a/src/vs/editor/browser/viewParts/overviewRuler/decorationsOverviewRuler.ts
+++ b/src/vs/editor/browser/viewParts/overviewRuler/decorationsOverviewRuler.ts
@@ -237,7 +237,7 @@ export class DecorationsOverviewRuler extends ViewPart {
this._cursorPositions = [];
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
this._tokensColorTrackerListener.dispose();
}
@@ -267,10 +267,10 @@ export class DecorationsOverviewRuler extends ViewPart {
// ---- begin view event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
return this._updateSettings(false);
}
- public onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
+ public override onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
this._cursorPositions = [];
for (let i = 0, len = e.selections.length; i < len; i++) {
this._cursorPositions[i] = e.selections[i].getPosition();
@@ -278,22 +278,22 @@ export class DecorationsOverviewRuler extends ViewPart {
this._cursorPositions.sort(Position.compare);
return true;
}
- public onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
+ public override onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
if (e.affectsOverviewRuler) {
return true;
}
return false;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return e.scrollHeightChanged;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return true;
}
- public onThemeChanged(e: viewEvents.ViewThemeChangedEvent): boolean {
+ public override onThemeChanged(e: viewEvents.ViewThemeChangedEvent): boolean {
// invalidate color cache
this._context.model.invalidateOverviewRulerColorCache();
return this._updateSettings(false);
diff --git a/src/vs/editor/browser/viewParts/overviewRuler/overviewRuler.ts b/src/vs/editor/browser/viewParts/overviewRuler/overviewRuler.ts
index b75ba0ae422..d13dc67d8ac 100644
--- a/src/vs/editor/browser/viewParts/overviewRuler/overviewRuler.ts
+++ b/src/vs/editor/browser/viewParts/overviewRuler/overviewRuler.ts
@@ -39,14 +39,14 @@ export class OverviewRuler extends ViewEventHandler implements IOverviewRuler {
this._context.addEventHandler(this);
}
- public dispose(): void {
+ public override dispose(): void {
this._context.removeEventHandler(this);
super.dispose();
}
// ---- begin view event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
if (e.hasChanged(EditorOption.lineHeight)) {
@@ -65,18 +65,18 @@ export class OverviewRuler extends ViewEventHandler implements IOverviewRuler {
return true;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
this._render();
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
if (e.scrollHeightChanged) {
this._zoneManager.setOuterHeight(e.scrollHeight);
this._render();
}
return true;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
this._render();
return true;
}
diff --git a/src/vs/editor/browser/viewParts/rulers/rulers.ts b/src/vs/editor/browser/viewParts/rulers/rulers.ts
index 696e088de58..fef13c81a92 100644
--- a/src/vs/editor/browser/viewParts/rulers/rulers.ts
+++ b/src/vs/editor/browser/viewParts/rulers/rulers.ts
@@ -32,19 +32,19 @@ export class Rulers extends ViewPart {
this._typicalHalfwidthCharacterWidth = options.get(EditorOption.fontInfo).typicalHalfwidthCharacterWidth;
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
this._rulers = options.get(EditorOption.rulers);
this._typicalHalfwidthCharacterWidth = options.get(EditorOption.fontInfo).typicalHalfwidthCharacterWidth;
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return e.scrollHeightChanged;
}
diff --git a/src/vs/editor/browser/viewParts/scrollDecoration/scrollDecoration.ts b/src/vs/editor/browser/viewParts/scrollDecoration/scrollDecoration.ts
index 12e4df8b465..5e5d5b6d566 100644
--- a/src/vs/editor/browser/viewParts/scrollDecoration/scrollDecoration.ts
+++ b/src/vs/editor/browser/viewParts/scrollDecoration/scrollDecoration.ts
@@ -37,7 +37,7 @@ export class ScrollDecorationViewPart extends ViewPart {
this._domNode.setAttribute('aria-hidden', 'true');
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
@@ -67,7 +67,7 @@ export class ScrollDecorationViewPart extends ViewPart {
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
const scrollbar = options.get(EditorOption.scrollbar);
this._useShadows = scrollbar.useShadows;
@@ -75,7 +75,7 @@ export class ScrollDecorationViewPart extends ViewPart {
this._updateShouldShow();
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
this._scrollTop = e.scrollTop;
return this._updateShouldShow();
}
diff --git a/src/vs/editor/browser/viewParts/selections/selections.ts b/src/vs/editor/browser/viewParts/selections/selections.ts
index 383f55a38c0..2ecb8afc50f 100644
--- a/src/vs/editor/browser/viewParts/selections/selections.ts
+++ b/src/vs/editor/browser/viewParts/selections/selections.ts
@@ -86,7 +86,7 @@ export class SelectionsOverlay extends DynamicViewOverlay {
this._context.addEventHandler(this);
}
- public dispose(): void {
+ public override dispose(): void {
this._context.removeEventHandler(this);
this._renderResult = null;
super.dispose();
@@ -94,37 +94,37 @@ export class SelectionsOverlay extends DynamicViewOverlay {
// --- begin event handlers
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
this._lineHeight = options.get(EditorOption.lineHeight);
this._roundedSelection = options.get(EditorOption.roundedSelection);
this._typicalHalfwidthCharacterWidth = options.get(EditorOption.fontInfo).typicalHalfwidthCharacterWidth;
return true;
}
- public onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
+ public override onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
this._selections = e.selections.slice(0);
return true;
}
- public onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
+ public override onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
// true for inline decorations that can end up relayouting text
return true;//e.inlineDecorationsChanged;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
return true;
}
- public onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
+ public override onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
return true;
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
return true;
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return e.scrollTopChanged;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return true;
}
diff --git a/src/vs/editor/browser/viewParts/viewCursors/viewCursors.ts b/src/vs/editor/browser/viewParts/viewCursors/viewCursors.ts
index ec5077ea0c7..7454608e4f4 100644
--- a/src/vs/editor/browser/viewParts/viewCursors/viewCursors.ts
+++ b/src/vs/editor/browser/viewParts/viewCursors/viewCursors.ts
@@ -74,7 +74,7 @@ export class ViewCursors extends ViewPart {
this._updateBlinking();
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
this._startCursorBlinkAnimation.dispose();
this._cursorFlatBlinkInterval.dispose();
@@ -85,17 +85,17 @@ export class ViewCursors extends ViewPart {
}
// --- begin event handlers
- public onCompositionStart(e: viewEvents.ViewCompositionStartEvent): boolean {
+ public override onCompositionStart(e: viewEvents.ViewCompositionStartEvent): boolean {
this._isComposingInput = true;
this._updateBlinking();
return true;
}
- public onCompositionEnd(e: viewEvents.ViewCompositionEndEvent): boolean {
+ public override onCompositionEnd(e: viewEvents.ViewCompositionEndEvent): boolean {
this._isComposingInput = false;
this._updateBlinking();
return true;
}
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
this._readOnly = options.get(EditorOption.readOnly);
@@ -138,7 +138,7 @@ export class ViewCursors extends ViewPart {
}
}
- public onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
+ public override onCursorStateChanged(e: viewEvents.ViewCursorStateChangedEvent): boolean {
const positions: Position[] = [];
for (let i = 0, len = e.selections.length; i < len; i++) {
positions[i] = e.selections[i].getPosition();
@@ -154,31 +154,31 @@ export class ViewCursors extends ViewPart {
return true;
}
- public onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
+ public override onDecorationsChanged(e: viewEvents.ViewDecorationsChangedEvent): boolean {
// true for inline decorations that can end up relayouting text
return true;
}
- public onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
+ public override onFlushed(e: viewEvents.ViewFlushedEvent): boolean {
return true;
}
- public onFocusChanged(e: viewEvents.ViewFocusChangedEvent): boolean {
+ public override onFocusChanged(e: viewEvents.ViewFocusChangedEvent): boolean {
this._editorHasFocus = e.isFocused;
this._updateBlinking();
return false;
}
- public onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
+ public override onLinesChanged(e: viewEvents.ViewLinesChangedEvent): boolean {
return true;
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
return true;
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return true;
}
- public onTokensChanged(e: viewEvents.ViewTokensChangedEvent): boolean {
+ public override onTokensChanged(e: viewEvents.ViewTokensChangedEvent): boolean {
const shouldRender = (position: Position) => {
for (let i = 0, len = e.ranges.length; i < len; i++) {
if (e.ranges[i].fromLineNumber <= position.lineNumber && position.lineNumber <= e.ranges[i].toLineNumber) {
@@ -197,7 +197,7 @@ export class ViewCursors extends ViewPart {
}
return false;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return true;
}
diff --git a/src/vs/editor/browser/viewParts/viewZones/viewZones.ts b/src/vs/editor/browser/viewParts/viewZones/viewZones.ts
index a73026a3f9b..7d6ae188521 100644
--- a/src/vs/editor/browser/viewParts/viewZones/viewZones.ts
+++ b/src/vs/editor/browser/viewParts/viewZones/viewZones.ts
@@ -66,7 +66,7 @@ export class ViewZones extends ViewPart {
this._zones = {};
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
this._zones = {};
}
@@ -97,7 +97,7 @@ export class ViewZones extends ViewPart {
return hadAChange;
}
- public onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
+ public override onConfigurationChanged(e: viewEvents.ViewConfigurationChangedEvent): boolean {
const options = this._context.configuration.options;
const layoutInfo = options.get(EditorOption.layoutInfo);
@@ -112,23 +112,23 @@ export class ViewZones extends ViewPart {
return true;
}
- public onLineMappingChanged(e: viewEvents.ViewLineMappingChangedEvent): boolean {
+ public override onLineMappingChanged(e: viewEvents.ViewLineMappingChangedEvent): boolean {
return this._recomputeWhitespacesProps();
}
- public onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
+ public override onLinesDeleted(e: viewEvents.ViewLinesDeletedEvent): boolean {
return true;
}
- public onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
+ public override onScrollChanged(e: viewEvents.ViewScrollChangedEvent): boolean {
return e.scrollTopChanged || e.scrollWidthChanged;
}
- public onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
+ public override onZonesChanged(e: viewEvents.ViewZonesChangedEvent): boolean {
return true;
}
- public onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
+ public override onLinesInserted(e: viewEvents.ViewLinesInsertedEvent): boolean {
return true;
}
diff --git a/src/vs/editor/browser/widget/codeEditorWidget.ts b/src/vs/editor/browser/widget/codeEditorWidget.ts
index 244447104d6..9d89b72db24 100644
--- a/src/vs/editor/browser/widget/codeEditorWidget.ts
+++ b/src/vs/editor/browser/widget/codeEditorWidget.ts
@@ -346,7 +346,7 @@ export class CodeEditorWidget extends Disposable implements editorBrowser.ICodeE
return editorCommon.EditorType.ICodeEditor;
}
- public dispose(): void {
+ public override dispose(): void {
this._codeEditorService.removeCodeEditor(this);
this._focusTracker.dispose();
@@ -1906,7 +1906,7 @@ export class EditorModeContext extends Disposable {
update();
}
- dispose() {
+ override dispose() {
super.dispose();
}
diff --git a/src/vs/editor/browser/widget/diffEditorWidget.ts b/src/vs/editor/browser/widget/diffEditorWidget.ts
index 6256e77b6ec..19170cfe828 100644
--- a/src/vs/editor/browser/widget/diffEditorWidget.ts
+++ b/src/vs/editor/browser/widget/diffEditorWidget.ts
@@ -619,7 +619,7 @@ export class DiffEditorWidget extends Disposable implements editorBrowser.IDiffE
return instantiationService.createInstance(CodeEditorWidget, container, options, editorWidgetOptions);
}
- public dispose(): void {
+ public override dispose(): void {
this._codeEditorService.removeDiffEditor(this);
if (this._beginUpdateDecorationsTimeout !== -1) {
@@ -2239,7 +2239,7 @@ class InlineViewZonesComputer extends ViewZonesComputer {
this._lineBreaksComputer = this._modifiedEditor._getViewModel()!.createLineBreaksComputer();
}
- public getViewZones(): IEditorsZones {
+ public override getViewZones(): IEditorsZones {
const result = super.getViewZones();
this._finalize(result);
return result;
diff --git a/src/vs/editor/browser/widget/diffNavigator.ts b/src/vs/editor/browser/widget/diffNavigator.ts
index f341549a1dc..a69b401c677 100644
--- a/src/vs/editor/browser/widget/diffNavigator.ts
+++ b/src/vs/editor/browser/widget/diffNavigator.ts
@@ -221,7 +221,7 @@ export class DiffNavigator extends Disposable implements IDiffNavigator {
this._move(false, scrollType);
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this.ranges = [];
this.disposed = true;
diff --git a/src/vs/editor/browser/widget/embeddedCodeEditorWidget.ts b/src/vs/editor/browser/widget/embeddedCodeEditorWidget.ts
index 3836a4ec057..911e593563a 100644
--- a/src/vs/editor/browser/widget/embeddedCodeEditorWidget.ts
+++ b/src/vs/editor/browser/widget/embeddedCodeEditorWidget.ts
@@ -57,7 +57,7 @@ export class EmbeddedCodeEditorWidget extends CodeEditorWidget {
super.updateOptions(this._overwriteOptions);
}
- updateOptions(newOptions: IEditorOptions): void {
+ override updateOptions(newOptions: IEditorOptions): void {
objects.mixin(this._overwriteOptions, newOptions, true);
super.updateOptions(this._overwriteOptions);
}
@@ -102,7 +102,7 @@ export class EmbeddedDiffEditorWidget extends DiffEditorWidget {
super.updateOptions(this._overwriteOptions);
}
- updateOptions(newOptions: IEditorOptions): void {
+ override updateOptions(newOptions: IEditorOptions): void {
objects.mixin(this._overwriteOptions, newOptions, true);
super.updateOptions(this._overwriteOptions);
}
diff --git a/src/vs/editor/common/config/editorOptions.ts b/src/vs/editor/common/config/editorOptions.ts
index 73ff201a517..6ebd6492b19 100644
--- a/src/vs/editor/common/config/editorOptions.ts
+++ b/src/vs/editor/common/config/editorOptions.ts
@@ -906,7 +906,7 @@ class EditorBooleanOption extends SimpleEditorOption extends SimpleEditorOption extends SimpleEditorOption extends SimpleEditorOption extends
this._allowedValues = allowedValues;
}
- public validate(input: any): V {
+ public override validate(input: any): V {
return stringSet(input, this.defaultValue, this._allowedValues);
}
}
@@ -1112,7 +1112,7 @@ class EditorAccessibilitySupport extends BaseEditorOption {
);
}
- public validate(input: any): number {
+ public override validate(input: any): number {
let r = EditorFloatOption.float(input, this.defaultValue);
if (r === 0) {
return EDITOR_FONT_DEFAULTS.fontSize;
}
return EditorFloatOption.clamp(r, 6, 100);
}
- public compute(env: IEnvironmentalOptions, options: IComputedEditorOptions, value: number): number {
+ public override compute(env: IEnvironmentalOptions, options: IComputedEditorOptions, value: number): number {
// The final fontSize respects the editor zoom level.
// So take the result from env.fontInfo
return env.fontInfo.fontSize;
@@ -2470,7 +2470,7 @@ class EditorLineHeight extends EditorIntOption {
);
}
- public compute(env: IEnvironmentalOptions, options: IComputedEditorOptions, value: number): number {
+ public override compute(env: IEnvironmentalOptions, options: IComputedEditorOptions, value: number): number {
// The lineHeight is computed from the fontSize if it is 0.
// Moreover, the final lineHeight respects the editor zoom level.
// So take the result from env.fontInfo
@@ -2766,7 +2766,7 @@ export type ValidQuickSuggestionsOptions = boolean | Readonly {
- public readonly defaultValue: Readonly>;
+ public override readonly defaultValue: Readonly>;
constructor() {
const defaults: ValidQuickSuggestionsOptions = {
diff --git a/src/vs/editor/common/controller/cursor.ts b/src/vs/editor/common/controller/cursor.ts
index 36c9691b295..41f2a9b17d4 100644
--- a/src/vs/editor/common/controller/cursor.ts
+++ b/src/vs/editor/common/controller/cursor.ts
@@ -156,7 +156,7 @@ export class Cursor extends Disposable {
this._prevEditOperationType = EditOperationType.Other;
}
- public dispose(): void {
+ public override dispose(): void {
this._cursors.dispose();
this._autoClosedActions = dispose(this._autoClosedActions);
super.dispose();
diff --git a/src/vs/editor/common/controller/cursorTypeOperations.ts b/src/vs/editor/common/controller/cursorTypeOperations.ts
index ff4c4c567ef..4acc0ae72be 100644
--- a/src/vs/editor/common/controller/cursorTypeOperations.ts
+++ b/src/vs/editor/common/controller/cursorTypeOperations.ts
@@ -957,7 +957,7 @@ export class TypeWithAutoClosingCommand extends ReplaceCommandWithOffsetCursorSt
this.enclosingRange = null;
}
- public computeCursorState(model: ITextModel, helper: ICursorStateComputerData): Selection {
+ public override computeCursorState(model: ITextModel, helper: ICursorStateComputerData): Selection {
let inverseEditOperations = helper.getInverseEditOperations();
let range = inverseEditOperations[0].range;
this.closeCharacterRange = new Range(range.startLineNumber, range.endColumn - this._closeCharacter.length, range.endLineNumber, range.endColumn);
diff --git a/src/vs/editor/common/core/selection.ts b/src/vs/editor/common/core/selection.ts
index 8f192adcb93..37ec04cd12d 100644
--- a/src/vs/editor/common/core/selection.ts
+++ b/src/vs/editor/common/core/selection.ts
@@ -76,7 +76,7 @@ export class Selection extends Range {
/**
* Transform to a human-readable representation.
*/
- public toString(): string {
+ public override toString(): string {
return '[' + this.selectionStartLineNumber + ',' + this.selectionStartColumn + ' -> ' + this.positionLineNumber + ',' + this.positionColumn + ']';
}
@@ -114,7 +114,7 @@ export class Selection extends Range {
/**
* Create a new selection with a different `positionLineNumber` and `positionColumn`.
*/
- public setEndPosition(endLineNumber: number, endColumn: number): Selection {
+ public override setEndPosition(endLineNumber: number, endColumn: number): Selection {
if (this.getDirection() === SelectionDirection.LTR) {
return new Selection(this.startLineNumber, this.startColumn, endLineNumber, endColumn);
}
@@ -131,7 +131,7 @@ export class Selection extends Range {
/**
* Create a new selection with a different `selectionStartLineNumber` and `selectionStartColumn`.
*/
- public setStartPosition(startLineNumber: number, startColumn: number): Selection {
+ public override setStartPosition(startLineNumber: number, startColumn: number): Selection {
if (this.getDirection() === SelectionDirection.LTR) {
return new Selection(startLineNumber, startColumn, this.endLineNumber, this.endColumn);
}
diff --git a/src/vs/editor/common/model/textModel.ts b/src/vs/editor/common/model/textModel.ts
index 6e01f85c7c5..5abb19f3ce0 100644
--- a/src/vs/editor/common/model/textModel.ts
+++ b/src/vs/editor/common/model/textModel.ts
@@ -384,7 +384,7 @@ export class TextModel extends Disposable implements model.ITextModel {
this._tokenization = new TextModelTokenization(this);
}
- public dispose(): void {
+ public override dispose(): void {
this._isDisposing = true;
this._onWillDispose.fire();
this._languageRegistryListener.dispose();
diff --git a/src/vs/editor/common/model/textModelTokens.ts b/src/vs/editor/common/model/textModelTokens.ts
index 3cf7bc9a292..5696219cc4a 100644
--- a/src/vs/editor/common/model/textModelTokens.ts
+++ b/src/vs/editor/common/model/textModelTokens.ts
@@ -246,7 +246,7 @@ export class TextModelTokenization extends Disposable {
this._resetTokenizationState();
}
- public dispose(): void {
+ public override dispose(): void {
this._isDisposed = true;
super.dispose();
}
diff --git a/src/vs/editor/common/services/editorWorkerServiceImpl.ts b/src/vs/editor/common/services/editorWorkerServiceImpl.ts
index dc6a0f706a6..2763dd1280d 100644
--- a/src/vs/editor/common/services/editorWorkerServiceImpl.ts
+++ b/src/vs/editor/common/services/editorWorkerServiceImpl.ts
@@ -77,7 +77,7 @@ export class EditorWorkerServiceImpl extends Disposable implements IEditorWorker
this._register(modes.CompletionProviderRegistry.register('*', new WordBasedCompletionItemProvider(this._workerManager, configurationService, this._modelService)));
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
@@ -225,7 +225,7 @@ class WorkerManager extends Disposable {
this._register(this._modelService.onModelRemoved(_ => this._checkStopEmptyWorker()));
}
- public dispose(): void {
+ public override dispose(): void {
if (this._editorWorkerClient) {
this._editorWorkerClient.dispose();
this._editorWorkerClient = null;
@@ -292,7 +292,7 @@ class EditorModelManager extends Disposable {
}
}
- public dispose(): void {
+ public override dispose(): void {
for (let modelUrl in this._syncedModels) {
dispose(this._syncedModels[modelUrl]);
}
@@ -523,7 +523,7 @@ export class EditorWorkerClient extends Disposable {
});
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this._disposed = true;
}
diff --git a/src/vs/editor/common/services/markerDecorationsServiceImpl.ts b/src/vs/editor/common/services/markerDecorationsServiceImpl.ts
index a80302ddbcc..939470195c9 100644
--- a/src/vs/editor/common/services/markerDecorationsServiceImpl.ts
+++ b/src/vs/editor/common/services/markerDecorationsServiceImpl.ts
@@ -81,7 +81,7 @@ export class MarkerDecorationsService extends Disposable implements IMarkerDecor
this._register(this._markerService.onMarkerChanged(this._handleMarkerChange, this));
}
- dispose() {
+ override dispose() {
super.dispose();
this._markerDecorations.forEach(value => value.dispose());
this._markerDecorations.clear();
diff --git a/src/vs/editor/common/services/modelServiceImpl.ts b/src/vs/editor/common/services/modelServiceImpl.ts
index af7000ffd9c..3357e282ea1 100644
--- a/src/vs/editor/common/services/modelServiceImpl.ts
+++ b/src/vs/editor/common/services/modelServiceImpl.ts
@@ -774,7 +774,7 @@ export class ModelSemanticColoring extends Disposable {
this._fetchDocumentSemanticTokens.schedule(0);
}
- public dispose(): void {
+ public override dispose(): void {
if (this._currentDocumentResponse) {
this._currentDocumentResponse.dispose();
this._currentDocumentResponse = null;
diff --git a/src/vs/editor/common/services/webWorker.ts b/src/vs/editor/common/services/webWorker.ts
index 55dfe84af39..2e3e7e67148 100644
--- a/src/vs/editor/common/services/webWorker.ts
+++ b/src/vs/editor/common/services/webWorker.ts
@@ -76,7 +76,7 @@ class MonacoWebWorkerImpl extends EditorWorkerClient implements MonacoWebWork
}
// foreign host request
- public fhr(method: string, args: any[]): Promise {
+ public override fhr(method: string, args: any[]): Promise {
if (!this._foreignModuleHost || typeof this._foreignModuleHost[method] !== 'function') {
return Promise.reject(new Error('Missing method ' + method + ' or missing main thread foreign host.'));
}
diff --git a/src/vs/editor/common/viewLayout/viewLayout.ts b/src/vs/editor/common/viewLayout/viewLayout.ts
index 9d5583ee0da..2f9d18b13ba 100644
--- a/src/vs/editor/common/viewLayout/viewLayout.ts
+++ b/src/vs/editor/common/viewLayout/viewLayout.ts
@@ -178,7 +178,7 @@ export class ViewLayout extends Disposable implements IViewLayout {
this._updateHeight();
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
diff --git a/src/vs/editor/common/viewModel/monospaceLineBreaksComputer.ts b/src/vs/editor/common/viewModel/monospaceLineBreaksComputer.ts
index 9640d007634..57370e069f7 100644
--- a/src/vs/editor/common/viewModel/monospaceLineBreaksComputer.ts
+++ b/src/vs/editor/common/viewModel/monospaceLineBreaksComputer.ts
@@ -32,7 +32,7 @@ class WrappingCharacterClassifier extends CharacterClassifier {
}
}
- public get(charCode: number): CharacterClass {
+ public override get(charCode: number): CharacterClass {
if (charCode >= 0 && charCode < 256) {
return this._asciiMap[charCode];
} else {
diff --git a/src/vs/editor/common/viewModel/viewModelImpl.ts b/src/vs/editor/common/viewModel/viewModelImpl.ts
index 82dc9853efb..eb7353176b5 100644
--- a/src/vs/editor/common/viewModel/viewModelImpl.ts
+++ b/src/vs/editor/common/viewModel/viewModelImpl.ts
@@ -142,7 +142,7 @@ export class ViewModel extends Disposable implements IViewModel {
this._updateConfigurationViewLineCountNow();
}
- public dispose(): void {
+ public override dispose(): void {
// First remove listeners, as disposing the lines might end up sending
// model decoration changed events ... and we no longer care about them ...
super.dispose();
diff --git a/src/vs/editor/contrib/codeAction/codeActionModel.ts b/src/vs/editor/contrib/codeAction/codeActionModel.ts
index d6e380ddc78..9f230ac4da9 100644
--- a/src/vs/editor/contrib/codeAction/codeActionModel.ts
+++ b/src/vs/editor/contrib/codeAction/codeActionModel.ts
@@ -207,7 +207,7 @@ export class CodeActionModel extends Disposable {
this._update();
}
- dispose(): void {
+ override dispose(): void {
if (this.#isDisposed) {
return;
}
diff --git a/src/vs/editor/contrib/codeAction/codeActionUi.ts b/src/vs/editor/contrib/codeAction/codeActionUi.ts
index 099789d9504..7a7abf66191 100644
--- a/src/vs/editor/contrib/codeAction/codeActionUi.ts
+++ b/src/vs/editor/contrib/codeAction/codeActionUi.ts
@@ -52,7 +52,7 @@ export class CodeActionUi extends Disposable {
});
}
- dispose() {
+ override dispose() {
this.#disposed = true;
super.dispose();
}
diff --git a/src/vs/editor/contrib/codeAction/lightBulbWidget.ts b/src/vs/editor/contrib/codeAction/lightBulbWidget.ts
index 8eac58026d9..7c0fedd446f 100644
--- a/src/vs/editor/contrib/codeAction/lightBulbWidget.ts
+++ b/src/vs/editor/contrib/codeAction/lightBulbWidget.ts
@@ -126,7 +126,7 @@ export class LightBulbWidget extends Disposable implements IContentWidget {
this._register(this._keybindingService.onDidUpdateKeybindings(this._updateLightBulbTitleAndIcon, this));
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this._editor.removeContentWidget(this);
}
diff --git a/src/vs/editor/contrib/colorPicker/colorContributions.ts b/src/vs/editor/contrib/colorPicker/colorContributions.ts
index 90f91a846c1..d7eefbf410f 100644
--- a/src/vs/editor/contrib/colorPicker/colorContributions.ts
+++ b/src/vs/editor/contrib/colorPicker/colorContributions.ts
@@ -26,7 +26,7 @@ export class ColorContribution extends Disposable implements IEditorContribution
this._register(_editor.onMouseDown((e) => this.onMouseDown(e)));
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
}
diff --git a/src/vs/editor/contrib/colorPicker/colorDetector.ts b/src/vs/editor/contrib/colorPicker/colorDetector.ts
index ccd09304a0e..4579e3c2daf 100644
--- a/src/vs/editor/contrib/colorPicker/colorDetector.ts
+++ b/src/vs/editor/contrib/colorPicker/colorDetector.ts
@@ -92,7 +92,7 @@ export class ColorDetector extends Disposable implements IEditorContribution {
return editor.getContribution(this.ID);
}
- dispose(): void {
+ override dispose(): void {
this.stop();
this.removeAllDecorations();
super.dispose();
diff --git a/src/vs/editor/contrib/dnd/dnd.ts b/src/vs/editor/contrib/dnd/dnd.ts
index d535bb90696..1f15af6828a 100644
--- a/src/vs/editor/contrib/dnd/dnd.ts
+++ b/src/vs/editor/contrib/dnd/dnd.ts
@@ -232,7 +232,7 @@ export class DragAndDropController extends Disposable implements IEditorContribu
target.type === MouseTargetType.GUTTER_LINE_DECORATIONS;
}
- public dispose(): void {
+ public override dispose(): void {
this._removeDecoration();
this._dragSelection = null;
this._mouseDown = false;
diff --git a/src/vs/editor/contrib/find/findController.ts b/src/vs/editor/contrib/find/findController.ts
index 8d4e6649a23..4da47b56237 100644
--- a/src/vs/editor/contrib/find/findController.ts
+++ b/src/vs/editor/contrib/find/findController.ts
@@ -139,7 +139,7 @@ export class CommonFindController extends Disposable implements IEditorContribut
}));
}
- public dispose(): void {
+ public override dispose(): void {
this.disposeModel();
super.dispose();
}
@@ -422,7 +422,7 @@ export class FindController extends CommonFindController implements IFindControl
this._findOptionsWidget = null;
}
- protected async _start(opts: IFindStartOptions): Promise {
+ protected async override _start(opts: IFindStartOptions): Promise {
if (!this._widget) {
this._createFindWidget();
}
@@ -459,7 +459,7 @@ export class FindController extends CommonFindController implements IFindControl
}
}
- public highlightFindOptions(ignoreWhenVisible: boolean = false): void {
+ public override highlightFindOptions(ignoreWhenVisible: boolean = false): void {
if (!this._widget) {
this._createFindWidget();
}
diff --git a/src/vs/editor/contrib/find/findOptionsWidget.ts b/src/vs/editor/contrib/find/findOptionsWidget.ts
index 660eed85579..72b9bdf9f2d 100644
--- a/src/vs/editor/contrib/find/findOptionsWidget.ts
+++ b/src/vs/editor/contrib/find/findOptionsWidget.ts
@@ -128,7 +128,7 @@ export class FindOptionsWidget extends Widget implements IOverlayWidget {
return ` (${kb.getLabel()})`;
}
- public dispose(): void {
+ public override dispose(): void {
this._editor.removeOverlayWidget(this);
super.dispose();
}
diff --git a/src/vs/editor/contrib/find/test/findController.test.ts b/src/vs/editor/contrib/find/test/findController.test.ts
index 9daca6b12da..6bf8752fe34 100644
--- a/src/vs/editor/contrib/find/test/findController.test.ts
+++ b/src/vs/editor/contrib/find/test/findController.test.ts
@@ -41,7 +41,7 @@ export class TestFindController extends CommonFindController {
this.hasFocus = false;
}
- protected async _start(opts: IFindStartOptions): Promise {
+ protected async override _start(opts: IFindStartOptions): Promise {
await super._start(opts);
if (opts.shouldFocus !== FindStartFocusAction.NoFocusChange) {
diff --git a/src/vs/editor/contrib/folding/folding.ts b/src/vs/editor/contrib/folding/folding.ts
index 139348a448a..c6396a556db 100644
--- a/src/vs/editor/contrib/folding/folding.ts
+++ b/src/vs/editor/contrib/folding/folding.ts
@@ -475,7 +475,7 @@ abstract class FoldingAction extends EditorAction {
abstract invoke(foldingController: FoldingController, foldingModel: FoldingModel, editor: ICodeEditor, args: T): void;
- public runEditorCommand(accessor: ServicesAccessor, editor: ICodeEditor, args: T): void | Promise {
+ public override runEditorCommand(accessor: ServicesAccessor, editor: ICodeEditor, args: T): void | Promise {
let foldingController = FoldingController.get(editor);
if (!foldingController) {
return;
diff --git a/src/vs/editor/contrib/gotoError/gotoErrorWidget.ts b/src/vs/editor/contrib/gotoError/gotoErrorWidget.ts
index 8096baa177b..5a78e184f6a 100644
--- a/src/vs/editor/contrib/gotoError/gotoErrorWidget.ts
+++ b/src/vs/editor/contrib/gotoError/gotoErrorWidget.ts
@@ -280,14 +280,14 @@ export class MarkerNavigationWidget extends PeekViewWidget {
}); // style() will trigger _applyStyles
}
- protected _applyStyles(): void {
+ protected override _applyStyles(): void {
if (this._parentContainer) {
this._parentContainer.style.backgroundColor = this._backgroundColor ? this._backgroundColor.toString() : '';
}
super._applyStyles();
}
- dispose(): void {
+ override dispose(): void {
this._callOnDispose.dispose();
super.dispose();
}
@@ -296,7 +296,7 @@ export class MarkerNavigationWidget extends PeekViewWidget {
this._parentContainer.focus();
}
- protected _fillHead(container: HTMLElement): void {
+ protected override _fillHead(container: HTMLElement): void {
super._fillHead(container);
this._disposables.add(this._actionbarWidget!.actionRunner.onBeforeRun(e => this.editor.focus()));
@@ -308,7 +308,7 @@ export class MarkerNavigationWidget extends PeekViewWidget {
menu.dispose();
}
- protected _fillTitleIcon(container: HTMLElement): void {
+ protected override _fillTitleIcon(container: HTMLElement): void {
this._icon = dom.append(container, dom.$(''));
}
@@ -325,7 +325,7 @@ export class MarkerNavigationWidget extends PeekViewWidget {
this._disposables.add(this._message);
}
- show(): void {
+ override show(): void {
throw new Error('call showAtMarker');
}
@@ -369,18 +369,18 @@ export class MarkerNavigationWidget extends PeekViewWidget {
this._relayout();
}
- protected _doLayoutBody(heightInPixel: number, widthInPixel: number): void {
+ protected override _doLayoutBody(heightInPixel: number, widthInPixel: number): void {
super._doLayoutBody(heightInPixel, widthInPixel);
this._heightInPixel = heightInPixel;
this._message.layout(heightInPixel, widthInPixel);
this._container.style.height = `${heightInPixel}px`;
}
- public _onWidth(widthInPixel: number): void {
+ public override _onWidth(widthInPixel: number): void {
this._message.layout(this._heightInPixel, widthInPixel);
}
- protected _relayout(): void {
+ protected override _relayout(): void {
super._relayout(this.computeRequiredHeight());
}
diff --git a/src/vs/editor/contrib/gotoSymbol/goToCommands.ts b/src/vs/editor/contrib/gotoSymbol/goToCommands.ts
index 83503350528..50ff4809f47 100644
--- a/src/vs/editor/contrib/gotoSymbol/goToCommands.ts
+++ b/src/vs/editor/contrib/gotoSymbol/goToCommands.ts
@@ -379,7 +379,7 @@ registerEditorAction(class GoToDeclarationAction extends DeclarationAction {
});
}
- protected _getNoResultFoundMessage(info: IWordAtPosition | null): string {
+ protected override _getNoResultFoundMessage(info: IWordAtPosition | null): string {
return info && info.word
? nls.localize('decl.noResultWord', "No declaration found for '{0}'", info.word)
: nls.localize('decl.generic.noResults', "No declaration found");
@@ -744,7 +744,7 @@ CommandsRegistry.registerCommand({
return editor.invokeWithinContext(accessor => {
const command = new class extends GenericGoToLocationAction {
- _getNoResultFoundMessage(info: IWordAtPosition | null) {
+ override _getNoResultFoundMessage(info: IWordAtPosition | null) {
return noResultsMessage || super._getNoResultFoundMessage(info);
}
}({
diff --git a/src/vs/editor/contrib/gotoSymbol/peek/referencesWidget.ts b/src/vs/editor/contrib/gotoSymbol/peek/referencesWidget.ts
index 09472b107c3..9f93856f7da 100644
--- a/src/vs/editor/contrib/gotoSymbol/peek/referencesWidget.ts
+++ b/src/vs/editor/contrib/gotoSymbol/peek/referencesWidget.ts
@@ -231,7 +231,7 @@ export class ReferenceWidget extends peekView.PeekViewWidget {
this.create();
}
- dispose(): void {
+ override dispose(): void {
this.setModel(undefined);
this._callOnDispose.dispose();
this._disposeOnNewModel.dispose();
@@ -254,7 +254,7 @@ export class ReferenceWidget extends peekView.PeekViewWidget {
});
}
- show(where: IRange) {
+ override show(where: IRange) {
this.editor.revealRangeInCenterIfOutsideViewport(where, ScrollType.Smooth);
super.show(where, this.layoutData.heightInLines || 18);
}
@@ -271,7 +271,7 @@ export class ReferenceWidget extends peekView.PeekViewWidget {
return this._preview.hasTextFocus();
}
- protected _onTitleClick(e: IMouseEvent): void {
+ protected override _onTitleClick(e: IMouseEvent): void {
if (this._preview && this._preview.getModel()) {
this._onDidSelectReference.fire({
element: this._getFocusedReference(),
@@ -398,13 +398,13 @@ export class ReferenceWidget extends peekView.PeekViewWidget {
dom.hide(this._treeContainer);
}
- protected _onWidth(width: number) {
+ protected override _onWidth(width: number) {
if (this._dim) {
this._doLayoutBody(this._dim.height, width);
}
}
- protected _doLayoutBody(heightInPixel: number, widthInPixel: number): void {
+ protected override _doLayoutBody(heightInPixel: number, widthInPixel: number): void {
super._doLayoutBody(heightInPixel, widthInPixel);
this._dim = new dom.Dimension(widthInPixel, heightInPixel);
this.layoutData.heightInLines = this._viewZone ? this._viewZone.heightInLines : this.layoutData.heightInLines;
diff --git a/src/vs/editor/contrib/hover/hoverWidgets.ts b/src/vs/editor/contrib/hover/hoverWidgets.ts
index ecc374f22b2..446be6a8b37 100644
--- a/src/vs/editor/contrib/hover/hoverWidgets.ts
+++ b/src/vs/editor/contrib/hover/hoverWidgets.ts
@@ -83,7 +83,7 @@ export class GlyphHoverWidget extends Widget implements IOverlayWidget {
return null;
}
- public dispose(): void {
+ public override dispose(): void {
this._editor.removeOverlayWidget(this);
super.dispose();
}
diff --git a/src/vs/editor/contrib/hover/modesContentHover.ts b/src/vs/editor/contrib/hover/modesContentHover.ts
index 7b401019e06..ade944741b1 100644
--- a/src/vs/editor/contrib/hover/modesContentHover.ts
+++ b/src/vs/editor/contrib/hover/modesContentHover.ts
@@ -292,7 +292,7 @@ export class ModesContentHoverWidget extends Widget implements IContentWidget, I
}));
}
- public dispose(): void {
+ public override dispose(): void {
this._hoverOperation.cancel();
this._editor.removeContentWidget(this);
super.dispose();
diff --git a/src/vs/editor/contrib/hover/modesGlyphHover.ts b/src/vs/editor/contrib/hover/modesGlyphHover.ts
index 54d0c58c732..db0337c0652 100644
--- a/src/vs/editor/contrib/hover/modesGlyphHover.ts
+++ b/src/vs/editor/contrib/hover/modesGlyphHover.ts
@@ -117,7 +117,7 @@ export class ModesGlyphHoverWidget extends GlyphHoverWidget {
}
- public dispose(): void {
+ public override dispose(): void {
this._hoverOperation.cancel();
super.dispose();
}
@@ -147,7 +147,7 @@ export class ModesGlyphHoverWidget extends GlyphHoverWidget {
this._hoverOperation.start(HoverStartMode.Delayed);
}
- public hide(): void {
+ public override hide(): void {
this._lastLineNumber = -1;
this._hoverOperation.cancel();
super.hide();
diff --git a/src/vs/editor/contrib/linkedEditing/linkedEditing.ts b/src/vs/editor/contrib/linkedEditing/linkedEditing.ts
index 441460812f4..8a586c1b13a 100644
--- a/src/vs/editor/contrib/linkedEditing/linkedEditing.ts
+++ b/src/vs/editor/contrib/linkedEditing/linkedEditing.ts
@@ -230,7 +230,7 @@ export class LinkedEditingContribution extends Disposable implements IEditorCont
}
}
- public dispose(): void {
+ public override dispose(): void {
this.clearRanges();
super.dispose();
}
@@ -378,7 +378,7 @@ export class LinkedEditingAction extends EditorAction {
});
}
- runCommand(accessor: ServicesAccessor, args: [URI, IPosition]): void | Promise {
+ override runCommand(accessor: ServicesAccessor, args: [URI, IPosition]): void | Promise {
const editorService = accessor.get(ICodeEditorService);
const [uri, pos] = Array.isArray(args) && args || [undefined, undefined];
diff --git a/src/vs/editor/contrib/multicursor/multicursor.ts b/src/vs/editor/contrib/multicursor/multicursor.ts
index 250a689c71d..cfbd4b2d208 100644
--- a/src/vs/editor/contrib/multicursor/multicursor.ts
+++ b/src/vs/editor/contrib/multicursor/multicursor.ts
@@ -463,7 +463,7 @@ export class MultiCursorSelectionController extends Disposable implements IEdito
this._session = null;
}
- public dispose(): void {
+ public override dispose(): void {
this._endSession();
super.dispose();
}
@@ -1060,7 +1060,7 @@ export class SelectionHighlighter extends Disposable implements IEditorContribut
className: 'selectionHighlight',
});
- public dispose(): void {
+ public override dispose(): void {
this._setState(null);
super.dispose();
}
diff --git a/src/vs/editor/contrib/parameterHints/parameterHintsModel.ts b/src/vs/editor/contrib/parameterHints/parameterHintsModel.ts
index 2c01d06321b..8c4d7d6fcb2 100644
--- a/src/vs/editor/contrib/parameterHints/parameterHintsModel.ts
+++ b/src/vs/editor/contrib/parameterHints/parameterHintsModel.ts
@@ -305,7 +305,7 @@ export class ParameterHintsModel extends Disposable {
}
}
- dispose(): void {
+ override dispose(): void {
this.cancel(true);
super.dispose();
}
diff --git a/src/vs/editor/contrib/peekView/peekView.ts b/src/vs/editor/contrib/peekView/peekView.ts
index 91e8bc41b42..6dbcde25dac 100644
--- a/src/vs/editor/contrib/peekView/peekView.ts
+++ b/src/vs/editor/contrib/peekView/peekView.ts
@@ -124,7 +124,7 @@ export abstract class PeekViewWidget extends ZoneWidget {
objects.mixin(this.options, defaultOptions, false);
}
- dispose(): void {
+ override dispose(): void {
if (!this.disposed) {
this.disposed = true; // prevent consumers who dispose on onDidClose from looping
super.dispose();
@@ -132,7 +132,7 @@ export abstract class PeekViewWidget extends ZoneWidget {
}
}
- style(styles: IPeekViewStyles): void {
+ override style(styles: IPeekViewStyles): void {
let options = this.options;
if (styles.headerBackgroundColor) {
options.headerBackgroundColor = styles.headerBackgroundColor;
@@ -146,7 +146,7 @@ export abstract class PeekViewWidget extends ZoneWidget {
super.style(styles);
}
- protected _applyStyles(): void {
+ protected override _applyStyles(): void {
super._applyStyles();
let options = this.options;
if (this._headElement && options.headerBackgroundColor) {
@@ -241,7 +241,7 @@ export abstract class PeekViewWidget extends ZoneWidget {
protected abstract _fillBody(container: HTMLElement): void;
- protected _doLayout(heightInPixel: number, widthInPixel: number): void {
+ protected override _doLayout(heightInPixel: number, widthInPixel: number): void {
if (!this._isShowing && heightInPixel < 0) {
// Looks like the view zone got folded away!
diff --git a/src/vs/editor/contrib/quickAccess/gotoSymbolQuickAccess.ts b/src/vs/editor/contrib/quickAccess/gotoSymbolQuickAccess.ts
index 96c0a9be6e9..c2c0e0d4368 100644
--- a/src/vs/editor/contrib/quickAccess/gotoSymbolQuickAccess.ts
+++ b/src/vs/editor/contrib/quickAccess/gotoSymbolQuickAccess.ts
@@ -35,10 +35,13 @@ export abstract class AbstractGotoSymbolQuickAccessProvider extends AbstractEdit
static SCOPE_PREFIX = ':';
static PREFIX_BY_CATEGORY = `${AbstractGotoSymbolQuickAccessProvider.PREFIX}${AbstractGotoSymbolQuickAccessProvider.SCOPE_PREFIX}`;
- constructor(protected options: IGotoSymbolQuickAccessProviderOptions = Object.create(null)) {
+ protected override readonly options: IGotoSymbolQuickAccessProviderOptions;
+
+ constructor(options: IGotoSymbolQuickAccessProviderOptions = Object.create(null)) {
super(options);
- options.canAcceptInBackground = true;
+ this.options = options;
+ this.options.canAcceptInBackground = true;
}
protected provideWithoutTextEditor(picker: IQuickPick): IDisposable {
diff --git a/src/vs/editor/contrib/rename/rename.ts b/src/vs/editor/contrib/rename/rename.ts
index 415473087a8..3eafd33c8a0 100644
--- a/src/vs/editor/contrib/rename/rename.ts
+++ b/src/vs/editor/contrib/rename/rename.ts
@@ -281,7 +281,7 @@ export class RenameAction extends EditorAction {
});
}
- runCommand(accessor: ServicesAccessor, args: [URI, IPosition]): void | Promise {
+ override runCommand(accessor: ServicesAccessor, args: [URI, IPosition]): void | Promise {
const editorService = accessor.get(ICodeEditorService);
const [uri, pos] = Array.isArray(args) && args || [undefined, undefined];
diff --git a/src/vs/editor/contrib/snippet/snippetParser.ts b/src/vs/editor/contrib/snippet/snippetParser.ts
index b6bd2eb37d3..8986345e970 100644
--- a/src/vs/editor/contrib/snippet/snippetParser.ts
+++ b/src/vs/editor/contrib/snippet/snippetParser.ts
@@ -200,13 +200,13 @@ export class Text extends Marker {
constructor(public value: string) {
super();
}
- toString() {
+ override toString() {
return this.value;
}
toTextmateString(): string {
return Text.escape(this.value);
}
- len(): number {
+ override len(): number {
return this.value.length;
}
clone(): Text {
@@ -279,7 +279,7 @@ export class Choice extends Marker {
readonly options: Text[] = [];
- appendChild(marker: Marker): this {
+ override appendChild(marker: Marker): this {
if (marker instanceof Text) {
marker.parent = this;
this.options.push(marker);
@@ -287,7 +287,7 @@ export class Choice extends Marker {
return this;
}
- toString() {
+ override toString() {
return this.options[0].value;
}
@@ -297,7 +297,7 @@ export class Choice extends Marker {
.join(',');
}
- len(): number {
+ override len(): number {
return this.options[0].len();
}
@@ -341,7 +341,7 @@ export class Transform extends Marker {
return ret;
}
- toString(): string {
+ override toString(): string {
return '';
}
@@ -555,12 +555,12 @@ export class TextmateSnippet extends Marker {
return this;
}
- appendChild(child: Marker) {
+ override appendChild(child: Marker) {
this._placeholders = undefined;
return super.appendChild(child);
}
- replace(child: Marker, others: Marker[]): void {
+ override replace(child: Marker, others: Marker[]): void {
this._placeholders = undefined;
return super.replace(child, others);
}
diff --git a/src/vs/editor/contrib/snippet/test/snippetVariables.test.ts b/src/vs/editor/contrib/snippet/test/snippetVariables.test.ts
index dff2a64beff..e370cd9ed15 100644
--- a/src/vs/editor/contrib/snippet/test/snippetVariables.test.ts
+++ b/src/vs/editor/contrib/snippet/test/snippetVariables.test.ts
@@ -21,7 +21,7 @@ import { toWorkspaceFolders } from 'vs/platform/workspaces/common/workspaces';
suite('Snippet Variables Resolver', function () {
const labelService = new class extends mock() {
- getUriLabel(uri: URI) {
+ override getUriLabel(uri: URI) {
return uri.fsPath;
}
};
@@ -92,7 +92,7 @@ suite('Snippet Variables Resolver', function () {
test('Path delimiters in code snippet variables aren\'t specific to remote OS #76840', function () {
const labelService = new class extends mock() {
- getUriLabel(uri: URI) {
+ override getUriLabel(uri: URI) {
return uri.fsPath.replace(/\/|\\/g, '|');
}
};
@@ -349,7 +349,7 @@ suite('Snippet Variables Resolver', function () {
// Mock a label service (only coded for file uris)
const workspaceLabelService = ((rootPath: string): ILabelService => {
const labelService = new class extends mock() {
- getUriLabel(uri: URI, options: { relative?: boolean } = {}) {
+ override getUriLabel(uri: URI, options: { relative?: boolean } = {}) {
const rootFsPath = URI.file(rootPath).fsPath + sep;
const fsPath = uri.fsPath;
if (options.relative && rootPath && fsPath.startsWith(rootFsPath)) {
diff --git a/src/vs/editor/contrib/suggest/suggestMemory.ts b/src/vs/editor/contrib/suggest/suggestMemory.ts
index dfb293635a2..639725c3742 100644
--- a/src/vs/editor/contrib/suggest/suggestMemory.ts
+++ b/src/vs/editor/contrib/suggest/suggestMemory.ts
@@ -90,7 +90,7 @@ export class LRUMemory extends Memory {
});
}
- select(model: ITextModel, pos: IPosition, items: CompletionItem[]): number {
+ override select(model: ITextModel, pos: IPosition, items: CompletionItem[]): number {
if (items.length === 0) {
return 0;
@@ -166,7 +166,7 @@ export class PrefixMemory extends Memory {
});
}
- select(model: ITextModel, pos: IPosition, items: CompletionItem[]): number {
+ override select(model: ITextModel, pos: IPosition, items: CompletionItem[]): number {
let { word } = model.getWordUntilPosition(pos);
if (!word) {
return super.select(model, pos, items);
diff --git a/src/vs/editor/contrib/suggest/suggestWidgetStatus.ts b/src/vs/editor/contrib/suggest/suggestWidgetStatus.ts
index 5739f38f198..ad7f97aee27 100644
--- a/src/vs/editor/contrib/suggest/suggestWidgetStatus.ts
+++ b/src/vs/editor/contrib/suggest/suggestWidgetStatus.ts
@@ -17,7 +17,7 @@ import { IInstantiationService } from 'vs/platform/instantiation/common/instanti
class StatusBarViewItem extends MenuEntryActionViewItem {
- updateLabel() {
+ override updateLabel() {
const kb = this._keybindingService.lookupKeybinding(this._action.id);
if (!kb) {
return super.updateLabel();
diff --git a/src/vs/editor/contrib/suggest/test/suggestController.test.ts b/src/vs/editor/contrib/suggest/test/suggestController.test.ts
index 11a7019886b..492e1cd391d 100644
--- a/src/vs/editor/contrib/suggest/test/suggestController.test.ts
+++ b/src/vs/editor/contrib/suggest/test/suggestController.test.ts
@@ -45,19 +45,19 @@ suite('SuggestController', function () {
[IStorageService, new InMemoryStorageService()],
[IKeybindingService, new MockKeybindingService()],
[IEditorWorkerService, new class extends mock() {
- computeWordRanges() {
+ override computeWordRanges() {
return Promise.resolve({});
}
}],
[ISuggestMemoryService, new class extends mock() {
- memorize(): void { }
- select(): number { return 0; }
+ override memorize(): void { }
+ override select(): number { return 0; }
}],
[IMenuService, new class extends mock() {
- createMenu() {
+ override createMenu() {
return new class extends mock() {
- onDidChange = Event.None;
- dispose() { }
+ override onDidChange = Event.None;
+ override dispose() { }
};
}
}]
diff --git a/src/vs/editor/contrib/suggest/test/suggestModel.test.ts b/src/vs/editor/contrib/suggest/test/suggestModel.test.ts
index 6a4afa2d48b..a17fa48850b 100644
--- a/src/vs/editor/contrib/suggest/test/suggestModel.test.ts
+++ b/src/vs/editor/contrib/suggest/test/suggestModel.test.ts
@@ -194,12 +194,12 @@ suite('SuggestModel - TriggerAndCancelOracle', function () {
const oracle = new SuggestModel(
editor,
new class extends mock() {
- computeWordRanges() {
+ override computeWordRanges() {
return Promise.resolve({});
}
},
new class extends mock() {
- readText() {
+ override readText() {
return Promise.resolve('CLIPPY');
}
},
@@ -680,7 +680,7 @@ suite('SuggestModel - TriggerAndCancelOracle', function () {
return withOracle(async (sugget, editor) => {
class TestCtrl extends SuggestController {
- _insertSuggestion(item: ISelectedSuggestion, flags: number = 0) {
+ override _insertSuggestion(item: ISelectedSuggestion, flags: number = 0) {
super._insertSuggestion(item, flags);
}
}
diff --git a/src/vs/editor/contrib/suggest/test/wordDistance.test.ts b/src/vs/editor/contrib/suggest/test/wordDistance.test.ts
index 05ccef17d53..c7f7cbf70c9 100644
--- a/src/vs/editor/contrib/suggest/test/wordDistance.test.ts
+++ b/src/vs/editor/contrib/suggest/test/wordDistance.test.ts
@@ -54,8 +54,8 @@ suite('suggest, word distance', function () {
editor.setPosition({ lineNumber: 2, column: 2 });
let modelService = new class extends mock() {
- onModelRemoved = Event.None;
- getModel(uri: URI) {
+ override onModelRemoved = Event.None;
+ override getModel(uri: URI) {
return uri.toString() === model.uri.toString() ? model : null;
}
};
@@ -74,7 +74,7 @@ suite('suggest, word distance', function () {
});
model.onDidChangeContent(e => this._worker.acceptModelChanged(model.uri.toString(), e));
}
- computeWordRanges(resource: URI, range: IRange): Promise<{ [word: string]: IRange[] } | null> {
+ override computeWordRanges(resource: URI, range: IRange): Promise<{ [word: string]: IRange[] } | null> {
return this._worker.computeWordRanges(resource.toString(), range, DEFAULT_WORD_REGEXP.source, DEFAULT_WORD_REGEXP.flags);
}
};
diff --git a/src/vs/editor/contrib/wordHighlighter/wordHighlighter.ts b/src/vs/editor/contrib/wordHighlighter/wordHighlighter.ts
index 35e8b64ab00..8e2d8a09078 100644
--- a/src/vs/editor/contrib/wordHighlighter/wordHighlighter.ts
+++ b/src/vs/editor/contrib/wordHighlighter/wordHighlighter.ts
@@ -140,7 +140,7 @@ class TextualOccurenceAtPositionRequest extends OccurenceAtPositionRequest {
});
}
- public isValid(model: ITextModel, selection: Selection, decorationIds: string[]): boolean {
+ public override isValid(model: ITextModel, selection: Selection, decorationIds: string[]): boolean {
const currentSelectionIsEmpty = selection.isEmpty();
if (this._selectionIsEmpty !== currentSelectionIsEmpty) {
return false;
@@ -528,7 +528,7 @@ class WordHighlighterContribution extends Disposable implements IEditorContribut
}
}
- public dispose(): void {
+ public override dispose(): void {
if (this.wordHighlighter) {
this.wordHighlighter.dispose();
this.wordHighlighter = null;
diff --git a/src/vs/editor/contrib/wordOperations/wordOperations.ts b/src/vs/editor/contrib/wordOperations/wordOperations.ts
index d617c47bc68..4dd26431815 100644
--- a/src/vs/editor/contrib/wordOperations/wordOperations.ts
+++ b/src/vs/editor/contrib/wordOperations/wordOperations.ts
@@ -187,7 +187,7 @@ export class CursorWordAccessibilityLeft extends WordLeftCommand {
});
}
- protected _move(_: WordCharacterClassifier, model: ITextModel, position: Position, wordNavigationType: WordNavigationType): Position {
+ protected override _move(_: WordCharacterClassifier, model: ITextModel, position: Position, wordNavigationType: WordNavigationType): Position {
return super._move(getMapForWordSeparators(EditorOptions.wordSeparators.defaultValue), model, position, wordNavigationType);
}
}
@@ -202,7 +202,7 @@ export class CursorWordAccessibilityLeftSelect extends WordLeftCommand {
});
}
- protected _move(_: WordCharacterClassifier, model: ITextModel, position: Position, wordNavigationType: WordNavigationType): Position {
+ protected override _move(_: WordCharacterClassifier, model: ITextModel, position: Position, wordNavigationType: WordNavigationType): Position {
return super._move(getMapForWordSeparators(EditorOptions.wordSeparators.defaultValue), model, position, wordNavigationType);
}
}
@@ -295,7 +295,7 @@ export class CursorWordAccessibilityRight extends WordRightCommand {
});
}
- protected _move(_: WordCharacterClassifier, model: ITextModel, position: Position, wordNavigationType: WordNavigationType): Position {
+ protected override _move(_: WordCharacterClassifier, model: ITextModel, position: Position, wordNavigationType: WordNavigationType): Position {
return super._move(getMapForWordSeparators(EditorOptions.wordSeparators.defaultValue), model, position, wordNavigationType);
}
}
@@ -310,7 +310,7 @@ export class CursorWordAccessibilityRightSelect extends WordRightCommand {
});
}
- protected _move(_: WordCharacterClassifier, model: ITextModel, position: Position, wordNavigationType: WordNavigationType): Position {
+ protected override _move(_: WordCharacterClassifier, model: ITextModel, position: Position, wordNavigationType: WordNavigationType): Position {
return super._move(getMapForWordSeparators(EditorOptions.wordSeparators.defaultValue), model, position, wordNavigationType);
}
}
diff --git a/src/vs/editor/standalone/browser/accessibilityHelp/accessibilityHelp.ts b/src/vs/editor/standalone/browser/accessibilityHelp/accessibilityHelp.ts
index 9da4eb8dacb..cc247cbbd2b 100644
--- a/src/vs/editor/standalone/browser/accessibilityHelp/accessibilityHelp.ts
+++ b/src/vs/editor/standalone/browser/accessibilityHelp/accessibilityHelp.ts
@@ -177,7 +177,7 @@ class AccessibilityHelpWidget extends Widget implements IOverlayWidget {
this._editor.addOverlayWidget(this);
}
- public dispose(): void {
+ public override dispose(): void {
this._editor.removeOverlayWidget(this);
super.dispose();
}
diff --git a/src/vs/editor/standalone/browser/iPadShowKeyboard/iPadShowKeyboard.ts b/src/vs/editor/standalone/browser/iPadShowKeyboard/iPadShowKeyboard.ts
index e67d6364b57..39c480d0dd9 100644
--- a/src/vs/editor/standalone/browser/iPadShowKeyboard/iPadShowKeyboard.ts
+++ b/src/vs/editor/standalone/browser/iPadShowKeyboard/iPadShowKeyboard.ts
@@ -44,7 +44,7 @@ export class IPadShowKeyboard extends Disposable implements IEditorContribution
}
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
if (this.widget) {
this.widget.dispose();
@@ -77,7 +77,7 @@ class ShowKeyboardWidget extends Disposable implements IOverlayWidget {
this.editor.addOverlayWidget(this);
}
- public dispose(): void {
+ public override dispose(): void {
this.editor.removeOverlayWidget(this);
super.dispose();
}
diff --git a/src/vs/editor/standalone/browser/inspectTokens/inspectTokens.ts b/src/vs/editor/standalone/browser/inspectTokens/inspectTokens.ts
index 88b54c86776..df5c80779c3 100644
--- a/src/vs/editor/standalone/browser/inspectTokens/inspectTokens.ts
+++ b/src/vs/editor/standalone/browser/inspectTokens/inspectTokens.ts
@@ -53,7 +53,7 @@ class InspectTokensController extends Disposable implements IEditorContribution
this._register(this._editor.onKeyUp((e) => e.keyCode === KeyCode.Escape && this.stop()));
}
- public dispose(): void {
+ public override dispose(): void {
this.stop();
super.dispose();
}
@@ -171,7 +171,7 @@ class InspectTokensWidget extends Disposable implements IContentWidget {
this._editor.addContentWidget(this);
}
- public dispose(): void {
+ public override dispose(): void {
this._editor.removeContentWidget(this);
super.dispose();
}
diff --git a/src/vs/editor/standalone/browser/quickInput/standaloneQuickInputServiceImpl.ts b/src/vs/editor/standalone/browser/quickInput/standaloneQuickInputServiceImpl.ts
index ae133a70df0..4c4bd06b51a 100644
--- a/src/vs/editor/standalone/browser/quickInput/standaloneQuickInputServiceImpl.ts
+++ b/src/vs/editor/standalone/browser/quickInput/standaloneQuickInputServiceImpl.ts
@@ -45,7 +45,7 @@ export class EditorScopedQuickInputServiceImpl extends QuickInputService {
};
}
- protected createController(): QuickInputController {
+ protected override createController(): QuickInputController {
return super.createController(this.host);
}
}
diff --git a/src/vs/editor/standalone/browser/standaloneCodeEditor.ts b/src/vs/editor/standalone/browser/standaloneCodeEditor.ts
index f315ffb0ed1..89dff41d626 100644
--- a/src/vs/editor/standalone/browser/standaloneCodeEditor.ts
+++ b/src/vs/editor/standalone/browser/standaloneCodeEditor.ts
@@ -427,11 +427,11 @@ export class StandaloneEditor extends StandaloneCodeEditor implements IStandalon
}
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
- public updateOptions(newOptions: Readonly): void {
+ public override updateOptions(newOptions: Readonly): void {
updateConfigurationService(this._configurationService, newOptions, false);
if (typeof newOptions.theme === 'string') {
this._standaloneThemeService.setTheme(newOptions.theme);
@@ -442,14 +442,14 @@ export class StandaloneEditor extends StandaloneCodeEditor implements IStandalon
super.updateOptions(newOptions);
}
- _attachModel(model: ITextModel | null): void {
+ override _attachModel(model: ITextModel | null): void {
super._attachModel(model);
if (this._modelData) {
this._contextViewService.setContainer(this._modelData.view.domNode.domNode);
}
}
- _postDetachModelCleanup(detachedModel: ITextModel): void {
+ override _postDetachModelCleanup(detachedModel: ITextModel): void {
super._postDetachModelCleanup(detachedModel);
if (detachedModel && this._ownsModel) {
detachedModel.dispose();
@@ -503,11 +503,11 @@ export class StandaloneDiffEditor extends DiffEditorWidget implements IStandalon
this._contextViewService.setContainer(this._containerDomElement);
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
- public updateOptions(newOptions: Readonly): void {
+ public override updateOptions(newOptions: Readonly): void {
updateConfigurationService(this._configurationService, newOptions, true);
if (typeof newOptions.theme === 'string') {
this._standaloneThemeService.setTheme(newOptions.theme);
@@ -518,15 +518,15 @@ export class StandaloneDiffEditor extends DiffEditorWidget implements IStandalon
super.updateOptions(newOptions);
}
- protected _createInnerEditor(instantiationService: IInstantiationService, container: HTMLElement, options: Readonly): CodeEditorWidget {
+ protected override _createInnerEditor(instantiationService: IInstantiationService, container: HTMLElement, options: Readonly): CodeEditorWidget {
return instantiationService.createInstance(StandaloneCodeEditor, container, options);
}
- public getOriginalEditor(): IStandaloneCodeEditor {
+ public override getOriginalEditor(): IStandaloneCodeEditor {
return super.getOriginalEditor();
}
- public getModifiedEditor(): IStandaloneCodeEditor {
+ public override getModifiedEditor(): IStandaloneCodeEditor {
return super.getModifiedEditor();
}
diff --git a/src/vs/editor/standalone/test/browser/standaloneLanguages.test.ts b/src/vs/editor/standalone/test/browser/standaloneLanguages.test.ts
index c4755792c8b..bacfb056976 100644
--- a/src/vs/editor/standalone/test/browser/standaloneLanguages.test.ts
+++ b/src/vs/editor/standalone/test/browser/standaloneLanguages.test.ts
@@ -25,7 +25,7 @@ suite('TokenizationSupport2Adapter', () => {
constructor() {
super(null!, null!);
}
- public match(languageId: LanguageId, token: string): number {
+ public override match(languageId: LanguageId, token: string): number {
return (
((this.counter++) << MetadataConsts.FOREGROUND_OFFSET)
| (languageId << MetadataConsts.LANGUAGEID_OFFSET)
diff --git a/src/vs/editor/test/browser/services/decorationRenderOptions.test.ts b/src/vs/editor/test/browser/services/decorationRenderOptions.test.ts
index 8f0e568cc5f..55be31487ce 100644
--- a/src/vs/editor/test/browser/services/decorationRenderOptions.test.ts
+++ b/src/vs/editor/test/browser/services/decorationRenderOptions.test.ts
@@ -32,11 +32,11 @@ class TestGlobalStyleSheet extends GlobalStyleSheet {
super(null!);
}
- public insertRule(rule: string, index?: number): void {
+ public override insertRule(rule: string, index?: number): void {
this.rules.unshift(rule);
}
- public removeRulesContainingSelector(ruleName: string): void {
+ public override removeRulesContainingSelector(ruleName: string): void {
for (let i = 0; i < this.rules.length; i++) {
if (this.rules[i].indexOf(ruleName) >= 0) {
this.rules.splice(i, 1);
diff --git a/src/vs/editor/test/browser/testCodeEditor.ts b/src/vs/editor/test/browser/testCodeEditor.ts
index 94741e65678..a516f3bed10 100644
--- a/src/vs/editor/test/browser/testCodeEditor.ts
+++ b/src/vs/editor/test/browser/testCodeEditor.ts
@@ -36,10 +36,10 @@ export interface ITestCodeEditor extends IActiveCodeEditor {
export class TestCodeEditor extends CodeEditorWidget implements ICodeEditor {
//#region testing overrides
- protected _createConfiguration(options: Readonly): IConfiguration {
+ protected override _createConfiguration(options: Readonly): IConfiguration {
return new TestConfiguration(options);
}
- protected _createView(viewModel: ViewModel): [View, boolean] {
+ protected override _createView(viewModel: ViewModel): [View, boolean] {
// Never create a view
return [null! as View, false];
}
@@ -47,7 +47,7 @@ export class TestCodeEditor extends CodeEditorWidget implements ICodeEditor {
public setHasTextFocus(hasTextFocus: boolean): void {
this._hasTextFocus = hasTextFocus;
}
- public hasTextFocus(): boolean {
+ public override hasTextFocus(): boolean {
return this._hasTextFocus;
}
//#endregion
@@ -64,7 +64,7 @@ export class TestCodeEditor extends CodeEditorWidget implements ICodeEditor {
}
class TestCodeEditorWithAutoModelDisposal extends TestCodeEditor {
- public dispose() {
+ public override dispose() {
super.dispose();
if (this._modelData) {
this._modelData.model.dispose();
diff --git a/src/vs/editor/test/common/config/commonEditorConfig.test.ts b/src/vs/editor/test/common/config/commonEditorConfig.test.ts
index 1faca7a1767..465996a78a1 100644
--- a/src/vs/editor/test/common/config/commonEditorConfig.test.ts
+++ b/src/vs/editor/test/common/config/commonEditorConfig.test.ts
@@ -53,7 +53,7 @@ suite('Common Editor Config', () => {
});
class TestWrappingConfiguration extends TestConfiguration {
- protected _getEnvConfiguration(): IEnvConfiguration {
+ protected override _getEnvConfiguration(): IEnvConfiguration {
return {
extraEditorClassName: '',
outerWidth: 1000,
diff --git a/src/vs/editor/test/common/services/textResourceConfigurationService.test.ts b/src/vs/editor/test/common/services/textResourceConfigurationService.test.ts
index a66a305526d..dff36f24346 100644
--- a/src/vs/editor/test/common/services/textResourceConfigurationService.test.ts
+++ b/src/vs/editor/test/common/services/textResourceConfigurationService.test.ts
@@ -18,10 +18,10 @@ suite('TextResourceConfigurationService - Update', () => {
let configurationValue: IConfigurationValue = {};
let updateArgs: any[];
let configurationService = new class extends TestConfigurationService {
- inspect() {
+ override inspect() {
return configurationValue;
}
- updateValue() {
+ override updateValue() {
updateArgs = [...arguments];
return Promise.resolve();
}
diff --git a/src/vs/editor/test/common/viewModel/viewModelImpl.test.ts b/src/vs/editor/test/common/viewModel/viewModelImpl.test.ts
index d556f3da7a5..0e9f99aba50 100644
--- a/src/vs/editor/test/common/viewModel/viewModelImpl.test.ts
+++ b/src/vs/editor/test/common/viewModel/viewModelImpl.test.ts
@@ -66,7 +66,7 @@ suite('ViewModel', () => {
viewLineCount.push(viewModel.getLineCount());
viewModel.addViewEventHandler(new class extends ViewEventHandler {
- handleEvents(events: ViewEvent[]): void {
+ override handleEvents(events: ViewEvent[]): void {
// Access the view model
viewLineCount.push(viewModel.getLineCount());
}
diff --git a/src/vs/platform/actions/browser/menuEntryActionViewItem.ts b/src/vs/platform/actions/browser/menuEntryActionViewItem.ts
index eed8299d735..19fe5f4821c 100644
--- a/src/vs/platform/actions/browser/menuEntryActionViewItem.ts
+++ b/src/vs/platform/actions/browser/menuEntryActionViewItem.ts
@@ -129,7 +129,7 @@ export class MenuEntryActionViewItem extends ActionViewItem {
return this._wantsAltCommand && (this._action).alt || this._action;
}
- onClick(event: MouseEvent): void {
+ override onClick(event: MouseEvent): void {
event.preventDefault();
event.stopPropagation();
@@ -138,7 +138,7 @@ export class MenuEntryActionViewItem extends ActionViewItem {
.catch(err => this._notificationService.error(err));
}
- render(container: HTMLElement): void {
+ override render(container: HTMLElement): void {
super.render(container);
container.classList.add('menu-entry');
@@ -176,13 +176,13 @@ export class MenuEntryActionViewItem extends ActionViewItem {
}));
}
- updateLabel(): void {
+ override updateLabel(): void {
if (this.options.label && this.label) {
this.label.textContent = this._commandAction.label;
}
}
- updateTooltip(): void {
+ override updateTooltip(): void {
if (this.label) {
const keybinding = this._keybindingService.lookupKeybinding(this._commandAction.id);
const keybindingLabel = keybinding && keybinding.getLabel();
@@ -204,7 +204,7 @@ export class MenuEntryActionViewItem extends ActionViewItem {
}
}
- updateClass(): void {
+ override updateClass(): void {
if (this.options.icon) {
if (this._commandAction !== this._action) {
if (this._action.alt) {
@@ -268,7 +268,7 @@ export class SubmenuEntryActionViewItem extends DropdownMenuActionViewItem {
});
}
- render(container: HTMLElement): void {
+ override render(container: HTMLElement): void {
super.render(container);
if (this.element) {
container.classList.add('menu-entry');
diff --git a/src/vs/platform/actions/common/actions.ts b/src/vs/platform/actions/common/actions.ts
index 972dbb84f9e..b6e6b834815 100644
--- a/src/vs/platform/actions/common/actions.ts
+++ b/src/vs/platform/actions/common/actions.ts
@@ -321,7 +321,7 @@ export class ExecuteCommandAction extends Action {
super(id, label);
}
- run(...args: any[]): Promise {
+ override run(...args: any[]): Promise {
return this._commandService.executeCommand(this.id, ...args);
}
}
@@ -337,7 +337,7 @@ export class SubmenuItemAction extends SubmenuAction {
super(`submenuitem.${item.submenu.id}`, typeof item.title === 'string' ? item.title : item.title.value, [], 'submenu');
}
- get actions(): readonly IAction[] {
+ override get actions(): readonly IAction[] {
const result: IAction[] = [];
const menu = this._menuService.createMenu(this.item.submenu, this._contextKeyService);
const groups = menu.getActions(this._options);
diff --git a/src/vs/platform/actions/test/common/menuService.test.ts b/src/vs/platform/actions/test/common/menuService.test.ts
index ee7369e0610..ca7d9eb31ad 100644
--- a/src/vs/platform/actions/test/common/menuService.test.ts
+++ b/src/vs/platform/actions/test/common/menuService.test.ts
@@ -13,7 +13,7 @@ import { MockContextKeyService } from 'vs/platform/keybinding/test/common/mockKe
// --- service instances
const contextKeyService = new class extends MockContextKeyService {
- contextMatchesRules() {
+ override contextMatchesRules() {
return true;
}
};
diff --git a/src/vs/platform/backup/test/electron-main/backupMainService.test.ts b/src/vs/platform/backup/test/electron-main/backupMainService.test.ts
index 561f0069f5a..9968de89189 100644
--- a/src/vs/platform/backup/test/electron-main/backupMainService.test.ts
+++ b/src/vs/platform/backup/test/electron-main/backupMainService.test.ts
@@ -123,7 +123,7 @@ flakySuite('BackupMainService', () => {
return path.join(this.backupHome, id);
}
- getFolderHash(folderUri: URI): string {
+ override getFolderHash(folderUri: URI): string {
return super.getFolderHash(folderUri);
}
};
diff --git a/src/vs/platform/configuration/common/configurationModels.ts b/src/vs/platform/configuration/common/configurationModels.ts
index 5993f0fddde..3988fd347ec 100644
--- a/src/vs/platform/configuration/common/configurationModels.ts
+++ b/src/vs/platform/configuration/common/configurationModels.ts
@@ -214,42 +214,53 @@ export class DefaultConfigurationModel extends ConfigurationModel {
}
}
+export interface ConfigurationParseOptions {
+ scopes: ConfigurationScope[] | undefined;
+ excludeUnsafeConfigurations?: boolean;
+}
+
export class ConfigurationModelParser {
private _raw: any = null;
private _configurationModel: ConfigurationModel | null = null;
+ private _excludedUnsafeConfigurations: string[] = [];
private _parseErrors: any[] = [];
- constructor(protected readonly _name: string, private _scopes?: ConfigurationScope[]) { }
+ constructor(protected readonly _name: string) { }
get configurationModel(): ConfigurationModel {
return this._configurationModel || new ConfigurationModel();
}
+ get excludedUnsafeConfigurations(): string[] {
+ return this._excludedUnsafeConfigurations;
+ }
+
get errors(): any[] {
return this._parseErrors;
}
- public parseContent(content: string | null | undefined): void {
+ public parse(content: string | null | undefined, options?: ConfigurationParseOptions): void {
if (!types.isUndefinedOrNull(content)) {
const raw = this.doParseContent(content);
- this.parseRaw(raw);
+ this.parseRaw(raw, options);
}
}
- public parseRaw(raw: any): void {
- this._raw = raw;
- const configurationModel = this.doParseRaw(raw);
- this._configurationModel = new ConfigurationModel(configurationModel.contents, configurationModel.keys, configurationModel.overrides);
- }
-
- public parse(): void {
+ public reparse(options: ConfigurationParseOptions): void {
if (this._raw) {
- this.parseRaw(this._raw);
+ this.parseRaw(this._raw, options);
}
}
- protected doParseContent(content: string): any {
+ public parseRaw(raw: any, options?: ConfigurationParseOptions): void {
+ this._raw = raw;
+ const { contents, keys, overrides, unsafeConfigurations } = this.doParseRaw(raw, options);
+ this._configurationModel = new ConfigurationModel(contents, keys, overrides);
+ this._excludedUnsafeConfigurations = unsafeConfigurations || [];
+ }
+
+ private doParseContent(content: string): any {
let raw: any = {};
let currentProperty: string | null = null;
let currentParent: any = [];
@@ -306,42 +317,49 @@ export class ConfigurationModelParser {
return raw;
}
- protected doParseRaw(raw: any): IConfigurationModel {
- if (this._scopes) {
- const configurationProperties = Registry.as(Extensions.Configuration).getConfigurationProperties();
- raw = this.filterByScope(raw, configurationProperties, true, this._scopes);
- }
+ protected doParseRaw(raw: any, options?: ConfigurationParseOptions): IConfigurationModel & { unsafeConfigurations?: string[] } {
+ const configurationProperties = Registry.as(Extensions.Configuration).getConfigurationProperties();
+ const filtered = this.filter(raw, configurationProperties, true, options);
+ raw = filtered.raw;
const contents = toValuesTree(raw, message => console.error(`Conflict in settings file ${this._name}: ${message}`));
const keys = Object.keys(raw);
const overrides: IOverrides[] = toOverrides(raw, message => console.error(`Conflict in settings file ${this._name}: ${message}`));
- return { contents, keys, overrides };
+ return { contents, keys, overrides, unsafeConfigurations: filtered.unsafeConfigurations };
}
- private filterByScope(properties: any, configurationProperties: { [qualifiedKey: string]: IConfigurationPropertySchema }, filterOverriddenProperties: boolean, scopes: ConfigurationScope[]): {} {
- const result: any = {};
+ private filter(properties: any, configurationProperties: { [qualifiedKey: string]: IConfigurationPropertySchema | undefined }, filterOverriddenProperties: boolean, options?: ConfigurationParseOptions): { raw: {}, unsafeConfigurations: string[] } {
+ if (!options?.scopes && !options?.excludeUnsafeConfigurations) {
+ return { raw: properties, unsafeConfigurations: [] };
+ }
+ const raw: any = {};
+ const unsafeConfigurations: string[] = [];
for (let key in properties) {
if (OVERRIDE_PROPERTY_PATTERN.test(key) && filterOverriddenProperties) {
- result[key] = this.filterByScope(properties[key], configurationProperties, false, scopes);
+ const result = this.filter(properties[key], configurationProperties, false, options);
+ raw[key] = result.raw;
+ unsafeConfigurations.push(...result.unsafeConfigurations);
} else {
- const scope = this.getScope(key, configurationProperties);
+ const propertySchema = configurationProperties[key];
+ const scope = propertySchema ? typeof propertySchema.scope !== 'undefined' ? propertySchema.scope : ConfigurationScope.WINDOW : undefined;
// Load unregistered configurations always.
- if (scope === undefined || scopes.indexOf(scope) !== -1) {
- result[key] = properties[key];
+ if (scope === undefined || options.scopes === undefined || options.scopes.includes(scope)) {
+ if (options.excludeUnsafeConfigurations && propertySchema?.requiresTrustedWorkspace) {
+ unsafeConfigurations.push(key);
+ } else {
+ raw[key] = properties[key];
+ }
}
}
}
- return result;
+ return { raw, unsafeConfigurations };
}
- private getScope(key: string, configurationProperties: { [qualifiedKey: string]: IConfigurationPropertySchema }): ConfigurationScope | undefined {
- const propertySchema = configurationProperties[key];
- return propertySchema ? typeof propertySchema.scope !== 'undefined' ? propertySchema.scope : ConfigurationScope.WINDOW : undefined;
- }
}
export class UserSettings extends Disposable {
private readonly parser: ConfigurationModelParser;
+ private readonly parseOptions: ConfigurationParseOptions;
protected readonly _onDidChange: Emitter = this._register(new Emitter());
readonly onDidChange: Event = this._onDidChange.event;
@@ -352,7 +370,8 @@ export class UserSettings extends Disposable {
private readonly fileService: IFileService
) {
super();
- this.parser = new ConfigurationModelParser(this.userSettingsResource.toString(), this.scopes);
+ this.parser = new ConfigurationModelParser(this.userSettingsResource.toString());
+ this.parseOptions = { scopes: this.scopes };
this._register(this.fileService.watch(extUri.dirname(this.userSettingsResource)));
// Also listen to the resource incase the resource is a symlink - https://github.com/microsoft/vscode/issues/118134
this._register(this.fileService.watch(this.userSettingsResource));
@@ -362,15 +381,15 @@ export class UserSettings extends Disposable {
async loadConfiguration(): Promise {
try {
const content = await this.fileService.readFile(this.userSettingsResource);
- this.parser.parseContent(content.value.toString() || '{}');
+ this.parser.parse(content.value.toString() || '{}', this.parseOptions);
return this.parser.configurationModel;
} catch (e) {
return new ConfigurationModel();
}
}
- reprocess(): ConfigurationModel {
- this.parser.parse();
+ reparse(): ConfigurationModel {
+ this.parser.reparse(this.parseOptions);
return this.parser.configurationModel;
}
}
@@ -802,5 +821,4 @@ export class AllKeysConfigurationChangeEvent extends ConfigurationChangeEvent {
constructor(configuration: Configuration, workspace: Workspace, public source: ConfigurationTarget, public sourceConfig: any) {
super({ keys: configuration.allKeys(), overrides: [] }, undefined, configuration, workspace);
}
-
}
diff --git a/src/vs/platform/configuration/common/configurationRegistry.ts b/src/vs/platform/configuration/common/configurationRegistry.ts
index 7504cb721b3..d4a53e68587 100644
--- a/src/vs/platform/configuration/common/configurationRegistry.ts
+++ b/src/vs/platform/configuration/common/configurationRegistry.ts
@@ -110,6 +110,7 @@ export const enum ConfigurationScope {
export interface IConfigurationPropertySchema extends IJSONSchema {
scope?: ConfigurationScope;
+ requiresTrustedWorkspace?: boolean;
included?: boolean;
tags?: string[];
/**
@@ -136,6 +137,7 @@ export interface IConfigurationNode {
properties?: { [path: string]: IConfigurationPropertySchema; };
allOf?: IConfigurationNode[];
scope?: ConfigurationScope;
+ requiresTrustedWorkspace?: boolean;
extensionInfo?: IConfigurationExtensionInfo;
}
@@ -297,8 +299,9 @@ class ConfigurationRegistry implements IConfigurationRegistry {
this.updateOverridePropertyPatternKey();
}
- private validateAndRegisterProperties(configuration: IConfigurationNode, validate: boolean = true, scope: ConfigurationScope = ConfigurationScope.WINDOW): string[] {
+ private validateAndRegisterProperties(configuration: IConfigurationNode, validate: boolean = true, scope: ConfigurationScope = ConfigurationScope.WINDOW, requiresTrustedWorkspace?: boolean): string[] {
scope = types.isUndefinedOrNull(configuration.scope) ? scope : configuration.scope;
+ requiresTrustedWorkspace = types.isUndefinedOrNull(configuration.requiresTrustedWorkspace) ? types.isUndefined(requiresTrustedWorkspace) ? false : requiresTrustedWorkspace : configuration.requiresTrustedWorkspace;
let propertyKeys: string[] = [];
let properties = configuration.properties;
if (properties) {
@@ -318,6 +321,7 @@ class ConfigurationRegistry implements IConfigurationRegistry {
property.scope = undefined; // No scope for overridable properties `[${identifier}]`
} else {
property.scope = types.isUndefinedOrNull(property.scope) ? scope : property.scope;
+ property.requiresTrustedWorkspace = types.isUndefinedOrNull(property.requiresTrustedWorkspace) ? requiresTrustedWorkspace : property.requiresTrustedWorkspace;
}
// Add to properties maps
@@ -341,7 +345,7 @@ class ConfigurationRegistry implements IConfigurationRegistry {
let subNodes = configuration.allOf;
if (subNodes) {
for (let node of subNodes) {
- propertyKeys.push(...this.validateAndRegisterProperties(node, validate, scope));
+ propertyKeys.push(...this.validateAndRegisterProperties(node, validate, scope, requiresTrustedWorkspace));
}
}
return propertyKeys;
diff --git a/src/vs/platform/configuration/test/common/configurationModels.test.ts b/src/vs/platform/configuration/test/common/configurationModels.test.ts
index 97b4a40cc91..544fd32e573 100644
--- a/src/vs/platform/configuration/test/common/configurationModels.test.ts
+++ b/src/vs/platform/configuration/test/common/configurationModels.test.ts
@@ -242,10 +242,10 @@ suite('CustomConfigurationModel', () => {
test('simple merge using models', () => {
let base = new ConfigurationModelParser('base');
- base.parseContent(JSON.stringify({ 'a': 1, 'b': 2 }));
+ base.parse(JSON.stringify({ 'a': 1, 'b': 2 }));
let add = new ConfigurationModelParser('add');
- add.parseContent(JSON.stringify({ 'a': 3, 'c': 4 }));
+ add.parse(JSON.stringify({ 'a': 3, 'c': 4 }));
let result = base.configurationModel.merge(add.configurationModel);
assert.deepStrictEqual(result.contents, { 'a': 3, 'b': 2, 'c': 4 });
@@ -253,14 +253,14 @@ suite('CustomConfigurationModel', () => {
test('simple merge with an undefined contents', () => {
let base = new ConfigurationModelParser('base');
- base.parseContent(JSON.stringify({ 'a': 1, 'b': 2 }));
+ base.parse(JSON.stringify({ 'a': 1, 'b': 2 }));
let add = new ConfigurationModelParser('add');
let result = base.configurationModel.merge(add.configurationModel);
assert.deepStrictEqual(result.contents, { 'a': 1, 'b': 2 });
base = new ConfigurationModelParser('base');
add = new ConfigurationModelParser('add');
- add.parseContent(JSON.stringify({ 'a': 1, 'b': 2 }));
+ add.parse(JSON.stringify({ 'a': 1, 'b': 2 }));
result = base.configurationModel.merge(add.configurationModel);
assert.deepStrictEqual(result.contents, { 'a': 1, 'b': 2 });
@@ -272,25 +272,25 @@ suite('CustomConfigurationModel', () => {
test('Recursive merge using config models', () => {
let base = new ConfigurationModelParser('base');
- base.parseContent(JSON.stringify({ 'a': { 'b': 1 } }));
+ base.parse(JSON.stringify({ 'a': { 'b': 1 } }));
let add = new ConfigurationModelParser('add');
- add.parseContent(JSON.stringify({ 'a': { 'b': 2 } }));
+ add.parse(JSON.stringify({ 'a': { 'b': 2 } }));
let result = base.configurationModel.merge(add.configurationModel);
assert.deepStrictEqual(result.contents, { 'a': { 'b': 2 } });
});
test('Test contents while getting an existing property', () => {
let testObject = new ConfigurationModelParser('test');
- testObject.parseContent(JSON.stringify({ 'a': 1 }));
+ testObject.parse(JSON.stringify({ 'a': 1 }));
assert.deepStrictEqual(testObject.configurationModel.getValue('a'), 1);
- testObject.parseContent(JSON.stringify({ 'a': { 'b': 1 } }));
+ testObject.parse(JSON.stringify({ 'a': { 'b': 1 } }));
assert.deepStrictEqual(testObject.configurationModel.getValue('a'), { 'b': 1 });
});
test('Test contents are undefined for non existing properties', () => {
const testObject = new ConfigurationModelParser('test');
- testObject.parseContent(JSON.stringify({
+ testObject.parse(JSON.stringify({
awesome: true
}));
@@ -305,7 +305,7 @@ suite('CustomConfigurationModel', () => {
test('Test configWithOverrides gives all content merged with overrides', () => {
const testObject = new ConfigurationModelParser('test');
- testObject.parseContent(JSON.stringify({ 'a': 1, 'c': 1, '[b]': { 'a': 2 } }));
+ testObject.parse(JSON.stringify({ 'a': 1, 'c': 1, '[b]': { 'a': 2 } }));
assert.deepStrictEqual(testObject.configurationModel.override('b').contents, { 'a': 2, 'c': 1, '[b]': { 'a': 2 } });
});
@@ -318,17 +318,17 @@ suite('CustomConfigurationModel', () => {
test('Test update with empty data', () => {
const testObject = new ConfigurationModelParser('test');
- testObject.parseContent('');
+ testObject.parse('');
assert.deepStrictEqual(testObject.configurationModel.contents, Object.create(null));
assert.deepStrictEqual(testObject.configurationModel.keys, []);
- testObject.parseContent(null!);
+ testObject.parse(null!);
assert.deepStrictEqual(testObject.configurationModel.contents, Object.create(null));
assert.deepStrictEqual(testObject.configurationModel.keys, []);
- testObject.parseContent(undefined!);
+ testObject.parse(undefined!);
assert.deepStrictEqual(testObject.configurationModel.contents, Object.create(null));
assert.deepStrictEqual(testObject.configurationModel.keys, []);
@@ -380,7 +380,7 @@ suite('Configuration', () => {
test('Test update value', () => {
const parser = new ConfigurationModelParser('test');
- parser.parseContent(JSON.stringify({ 'a': 1 }));
+ parser.parse(JSON.stringify({ 'a': 1 }));
const testObject: Configuration = new Configuration(parser.configurationModel, new ConfigurationModel());
testObject.updateValue('a', 2);
@@ -390,7 +390,7 @@ suite('Configuration', () => {
test('Test update value after inspect', () => {
const parser = new ConfigurationModelParser('test');
- parser.parseContent(JSON.stringify({ 'a': 1 }));
+ parser.parse(JSON.stringify({ 'a': 1 }));
const testObject: Configuration = new Configuration(parser.configurationModel, new ConfigurationModel());
testObject.inspect('a', {}, undefined);
@@ -503,7 +503,7 @@ suite('Configuration', () => {
function parseConfigurationModel(content: any): ConfigurationModel {
const parser = new ConfigurationModelParser('test');
- parser.parseContent(JSON.stringify(content));
+ parser.parse(JSON.stringify(content));
return parser.configurationModel;
}
@@ -951,6 +951,6 @@ suite('AllKeysConfigurationChangeEvent', () => {
function toConfigurationModel(obj: any): ConfigurationModel {
const parser = new ConfigurationModelParser('test');
- parser.parseContent(JSON.stringify(obj));
+ parser.parse(JSON.stringify(obj));
return parser.configurationModel;
}
diff --git a/src/vs/platform/contextkey/browser/contextKeyService.ts b/src/vs/platform/contextkey/browser/contextKeyService.ts
index e40016315b3..34cf6f57ddd 100644
--- a/src/vs/platform/contextkey/browser/contextKeyService.ts
+++ b/src/vs/platform/contextkey/browser/contextKeyService.ts
@@ -75,19 +75,19 @@ class NullContext extends Context {
super(-1, null);
}
- public setValue(key: string, value: any): boolean {
+ public override setValue(key: string, value: any): boolean {
return false;
}
- public removeValue(key: string): boolean {
+ public override removeValue(key: string): boolean {
return false;
}
- public getValue(key: string): T | undefined {
+ public override getValue(key: string): T | undefined {
return undefined;
}
- collectAllValues(): { [key: string]: any; } {
+ override collectAllValues(): { [key: string]: any; } {
return Object.create(null);
}
}
@@ -137,7 +137,7 @@ class ConfigAwareContextValuesContainer extends Context {
this._listener.dispose();
}
- getValue(key: string): any {
+ override getValue(key: string): any {
if (key.indexOf(ConfigAwareContextValuesContainer._keyPrefix) !== 0) {
return super.getValue(key);
@@ -168,15 +168,15 @@ class ConfigAwareContextValuesContainer extends Context {
return value;
}
- setValue(key: string, value: any): boolean {
+ override setValue(key: string, value: any): boolean {
return super.setValue(key, value);
}
- removeValue(key: string): boolean {
+ override removeValue(key: string): boolean {
return super.removeValue(key);
}
- collectAllValues(): { [key: string]: any; } {
+ override collectAllValues(): { [key: string]: any; } {
const result: { [key: string]: any } = Object.create(null);
this._values.forEach((value, index) => result[index] = value);
return { ...result, ...super.collectAllValues() };
diff --git a/src/vs/platform/debug/electron-main/extensionHostDebugIpc.ts b/src/vs/platform/debug/electron-main/extensionHostDebugIpc.ts
index 08ff2ccf628..c814220f558 100644
--- a/src/vs/platform/debug/electron-main/extensionHostDebugIpc.ts
+++ b/src/vs/platform/debug/electron-main/extensionHostDebugIpc.ts
@@ -16,7 +16,7 @@ export class ElectronExtensionHostDebugBroadcastChannel extends Extens
super();
}
- call(ctx: TContext, command: string, arg?: any): Promise {
+ override call(ctx: TContext, command: string, arg?: any): Promise {
if (command === 'openExtensionDevelopmentHostWindow') {
return this.openExtensionDevelopmentHostWindow(arg[0], arg[1], arg[2]);
} else {
diff --git a/src/vs/platform/extensionManagement/electron-sandbox/extensionTipsService.ts b/src/vs/platform/extensionManagement/electron-sandbox/extensionTipsService.ts
index 51066d5bedd..0fbf6a632fe 100644
--- a/src/vs/platform/extensionManagement/electron-sandbox/extensionTipsService.ts
+++ b/src/vs/platform/extensionManagement/electron-sandbox/extensionTipsService.ts
@@ -40,7 +40,7 @@ const lastPromptedMediumImpExeTimeStorageKey = 'extensionTips/lastPromptedMedium
export class ExtensionTipsService extends BaseExtensionTipsService {
- _serviceBrand: any;
+ override _serviceBrand: any;
private readonly highImportanceExecutableTips: Map = new Map();
private readonly mediumImportanceExecutableTips: Map = new Map();
@@ -101,13 +101,13 @@ export class ExtensionTipsService extends BaseExtensionTipsService {
});
}
- async getImportantExecutableBasedTips(): Promise {
+ async override getImportantExecutableBasedTips(): Promise {
const highImportanceExeTips = await this.getValidExecutableBasedExtensionTips(this.highImportanceExecutableTips);
const mediumImportanceExeTips = await this.getValidExecutableBasedExtensionTips(this.mediumImportanceExecutableTips);
return [...highImportanceExeTips, ...mediumImportanceExeTips];
}
- getOtherExecutableBasedTips(): Promise {
+ override getOtherExecutableBasedTips(): Promise {
return this.getValidExecutableBasedExtensionTips(this.allOtherExecutableTips);
}
diff --git a/src/vs/platform/files/common/fileService.ts b/src/vs/platform/files/common/fileService.ts
index 6bdbbe9cc5c..e1f24089f39 100644
--- a/src/vs/platform/files/common/fileService.ts
+++ b/src/vs/platform/files/common/fileService.ts
@@ -1000,7 +1000,7 @@ export class FileService extends Disposable implements IFileService {
].join();
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
this.activeWatchers.forEach(watcher => dispose(watcher.disposable));
diff --git a/src/vs/platform/files/electron-browser/diskFileSystemProvider.ts b/src/vs/platform/files/electron-browser/diskFileSystemProvider.ts
index a2db35c4e04..df5e06afc34 100644
--- a/src/vs/platform/files/electron-browser/diskFileSystemProvider.ts
+++ b/src/vs/platform/files/electron-browser/diskFileSystemProvider.ts
@@ -21,7 +21,7 @@ export class DiskFileSystemProvider extends NodeDiskFileSystemProvider {
super(logService, options);
}
- get capabilities(): FileSystemProviderCapabilities {
+ override get capabilities(): FileSystemProviderCapabilities {
if (!this._capabilities) {
this._capabilities = super.capabilities | FileSystemProviderCapabilities.Trash;
}
@@ -29,7 +29,7 @@ export class DiskFileSystemProvider extends NodeDiskFileSystemProvider {
return this._capabilities;
}
- protected async doDelete(filePath: string, opts: FileDeleteOptions): Promise {
+ protected async override doDelete(filePath: string, opts: FileDeleteOptions): Promise {
if (!opts.useTrash) {
return super.doDelete(filePath, opts);
}
diff --git a/src/vs/platform/files/node/diskFileSystemProvider.ts b/src/vs/platform/files/node/diskFileSystemProvider.ts
index ce598c20387..ff111367d43 100644
--- a/src/vs/platform/files/node/diskFileSystemProvider.ts
+++ b/src/vs/platform/files/node/diskFileSystemProvider.ts
@@ -730,7 +730,7 @@ export class DiskFileSystemProvider extends Disposable implements
//#endregion
- dispose(): void {
+ override dispose(): void {
super.dispose();
dispose(this.recursiveWatcher);
diff --git a/src/vs/platform/files/node/watcher/nodejs/watcherService.ts b/src/vs/platform/files/node/watcher/nodejs/watcherService.ts
index b645715d160..0d03d317fc0 100644
--- a/src/vs/platform/files/node/watcher/nodejs/watcherService.ts
+++ b/src/vs/platform/files/node/watcher/nodejs/watcherService.ts
@@ -124,7 +124,7 @@ export class FileWatcher extends Disposable {
}
}
- dispose(): void {
+ override dispose(): void {
this.isDisposed = true;
super.dispose();
diff --git a/src/vs/platform/files/node/watcher/nsfw/watcherService.ts b/src/vs/platform/files/node/watcher/nsfw/watcherService.ts
index 32dead9e7fd..eebfe67f502 100644
--- a/src/vs/platform/files/node/watcher/nsfw/watcherService.ts
+++ b/src/vs/platform/files/node/watcher/nsfw/watcherService.ts
@@ -91,7 +91,7 @@ export class FileWatcher extends Disposable {
}
}
- dispose(): void {
+ override dispose(): void {
this.isDisposed = true;
super.dispose();
diff --git a/src/vs/platform/files/node/watcher/unix/watcherService.ts b/src/vs/platform/files/node/watcher/unix/watcherService.ts
index 1ee26f965ee..da00c8a1658 100644
--- a/src/vs/platform/files/node/watcher/unix/watcherService.ts
+++ b/src/vs/platform/files/node/watcher/unix/watcherService.ts
@@ -92,7 +92,7 @@ export class FileWatcher extends Disposable {
}
}
- dispose(): void {
+ override dispose(): void {
this.isDisposed = true;
super.dispose();
diff --git a/src/vs/platform/files/test/browser/fileService.test.ts b/src/vs/platform/files/test/browser/fileService.test.ts
index ff4126f2aaa..7fa3f3fdb0f 100644
--- a/src/vs/platform/files/test/browser/fileService.test.ts
+++ b/src/vs/platform/files/test/browser/fileService.test.ts
@@ -141,7 +141,7 @@ suite('File Service', () => {
const service = new FileService(new NullLogService());
const provider = new class extends NullFileSystemProvider {
- async stat(resource: URI): Promise {
+ async override stat(resource: URI): Promise {
return {
mtime: Date.now(),
ctime: Date.now(),
@@ -150,7 +150,7 @@ suite('File Service', () => {
};
}
- readFile(resource: URI): Promise {
+ override readFile(resource: URI): Promise {
if (async) {
return timeout(5).then(() => { throw new Error('failed'); });
}
@@ -158,7 +158,7 @@ suite('File Service', () => {
throw new Error('failed');
}
- open(resource: URI, opts: FileOpenOptions): Promise {
+ override open(resource: URI, opts: FileOpenOptions): Promise {
if (async) {
return timeout(5).then(() => { throw new Error('failed'); });
}
diff --git a/src/vs/platform/files/test/electron-browser/diskFileService.test.ts b/src/vs/platform/files/test/electron-browser/diskFileService.test.ts
index 58574258ee2..9b6c2cc4c0b 100644
--- a/src/vs/platform/files/test/electron-browser/diskFileService.test.ts
+++ b/src/vs/platform/files/test/electron-browser/diskFileService.test.ts
@@ -58,7 +58,7 @@ export class TestDiskFileSystemProvider extends DiskFileSystemProvider {
private smallStatSize: boolean = false;
private _testCapabilities!: FileSystemProviderCapabilities;
- get capabilities(): FileSystemProviderCapabilities {
+ override get capabilities(): FileSystemProviderCapabilities {
if (!this._testCapabilities) {
this._testCapabilities =
FileSystemProviderCapabilities.FileReadWrite |
@@ -76,7 +76,7 @@ export class TestDiskFileSystemProvider extends DiskFileSystemProvider {
return this._testCapabilities;
}
- set capabilities(capabilities: FileSystemProviderCapabilities) {
+ override set capabilities(capabilities: FileSystemProviderCapabilities) {
this._testCapabilities = capabilities;
}
@@ -88,7 +88,7 @@ export class TestDiskFileSystemProvider extends DiskFileSystemProvider {
this.smallStatSize = enabled;
}
- async stat(resource: URI): Promise {
+ async override stat(resource: URI): Promise {
const res = await super.stat(resource);
if (this.invalidStatSize) {
@@ -100,7 +100,7 @@ export class TestDiskFileSystemProvider extends DiskFileSystemProvider {
return res;
}
- async read(fd: number, pos: number, data: Uint8Array, offset: number, length: number): Promise {
+ async override read(fd: number, pos: number, data: Uint8Array, offset: number, length: number): Promise {
const bytesRead = await super.read(fd, pos, data, offset, length);
this.totalBytesRead += bytesRead;
@@ -108,7 +108,7 @@ export class TestDiskFileSystemProvider extends DiskFileSystemProvider {
return bytesRead;
}
- async readFile(resource: URI): Promise {
+ async override readFile(resource: URI): Promise {
const res = await super.readFile(resource);
this.totalBytesRead += res.byteLength;
diff --git a/src/vs/platform/instantiation/common/instantiationService.ts b/src/vs/platform/instantiation/common/instantiationService.ts
index 00f0cb9d95e..5e9af069237 100644
--- a/src/vs/platform/instantiation/common/instantiationService.ts
+++ b/src/vs/platform/instantiation/common/instantiationService.ts
@@ -267,8 +267,8 @@ class Trace {
private static readonly _None = new class extends Trace {
constructor() { super(-1, null); }
- stop() { }
- branch() { return this; }
+ override stop() { }
+ override branch() { return this; }
};
static traceInvocation(ctor: any): Trace {
diff --git a/src/vs/platform/issue/electron-main/issueMainService.ts b/src/vs/platform/issue/electron-main/issueMainService.ts
index 86417c0922d..caf76f32f3b 100644
--- a/src/vs/platform/issue/electron-main/issueMainService.ts
+++ b/src/vs/platform/issue/electron-main/issueMainService.ts
@@ -239,7 +239,7 @@ export class IssueMainService implements ICommonIssueService {
const processExplorerWindowConfigUrl = processExplorerDisposables.add(this.protocolMainService.createIPCObjectUrl());
const position = this.getWindowPosition(this.processExplorerParentWindow, 800, 500);
- this.processExplorerWindow = this.createBrowserWindow(position, processExplorerWindowConfigUrl, data.styles.backgroundColor, localize('issueReporter', "Issue Reporter"), data.zoomLevel);
+ this.processExplorerWindow = this.createBrowserWindow(position, processExplorerWindowConfigUrl, data.styles.backgroundColor, localize('processExplorer', "Process Explorer"), data.zoomLevel);
// Store into config object URL
processExplorerWindowConfigUrl.update({
diff --git a/src/vs/platform/keybinding/common/abstractKeybindingService.ts b/src/vs/platform/keybinding/common/abstractKeybindingService.ts
index 23716ee9a1e..689431ec33e 100644
--- a/src/vs/platform/keybinding/common/abstractKeybindingService.ts
+++ b/src/vs/platform/keybinding/common/abstractKeybindingService.ts
@@ -61,7 +61,7 @@ export abstract class AbstractKeybindingService extends Disposable implements IK
this._logging = false;
}
- public dispose(): void {
+ public override dispose(): void {
super.dispose();
}
diff --git a/src/vs/platform/keybinding/test/common/mockKeybindingService.ts b/src/vs/platform/keybinding/test/common/mockKeybindingService.ts
index 2fe7bbabca9..4f022e9dddf 100644
--- a/src/vs/platform/keybinding/test/common/mockKeybindingService.ts
+++ b/src/vs/platform/keybinding/test/common/mockKeybindingService.ts
@@ -78,7 +78,7 @@ export class MockScopableContextKeyService extends MockContextKeyService {
/**
* Don't implement this for all tests since we rarely depend on this behavior and it isn't implemented fully
*/
- public createScoped(domNote: HTMLElement): IContextKeyService {
+ public override createScoped(domNote: HTMLElement): IContextKeyService {
return new MockContextKeyService();
}
}
diff --git a/src/vs/platform/list/browser/listService.ts b/src/vs/platform/list/browser/listService.ts
index d1b0fa73811..073308a7844 100644
--- a/src/vs/platform/list/browser/listService.ts
+++ b/src/vs/platform/list/browser/listService.ts
@@ -298,7 +298,7 @@ export class WorkbenchList extends List {
this.disposables.add(this.navigator);
}
- updateOptions(options: IWorkbenchListOptionsUpdate): void {
+ override updateOptions(options: IWorkbenchListOptionsUpdate): void {
super.updateOptions(options);
if (options.overrideStyles) {
@@ -315,7 +315,7 @@ export class WorkbenchList extends List {
return this._useAltAsMultipleSelectionModifier;
}
- dispose(): void {
+ override dispose(): void {
this._styler?.dispose();
super.dispose();
}
@@ -410,7 +410,7 @@ export class WorkbenchPagedList extends PagedList {
this.disposables.add(this.navigator);
}
- updateOptions(options: IWorkbenchListOptionsUpdate): void {
+ override updateOptions(options: IWorkbenchListOptionsUpdate): void {
super.updateOptions(options);
if (options.overrideStyles) {
@@ -427,7 +427,7 @@ export class WorkbenchPagedList extends PagedList {
return this._useAltAsMultipleSelectionModifier;
}
- dispose(): void {
+ override dispose(): void {
this._styler?.dispose();
this.disposables.dispose();
super.dispose();
@@ -547,7 +547,7 @@ export class WorkbenchTable extends Table {
this.disposables.add(this.navigator);
}
- updateOptions(options: IWorkbenchTableOptionsUpdate): void {
+ override updateOptions(options: IWorkbenchTableOptionsUpdate): void {
super.updateOptions(options);
if (options.overrideStyles) {
@@ -564,7 +564,7 @@ export class WorkbenchTable extends Table {
return this._useAltAsMultipleSelectionModifier;
}
- dispose(): void {
+ override dispose(): void {
this._styler?.dispose();
this.disposables.dispose();
super.dispose();
@@ -829,7 +829,7 @@ export class WorkbenchCompressibleObjectTree, TFilter
this.disposables.add(this.internals);
}
- updateOptions(options: IWorkbenchCompressibleObjectTreeOptionsUpdate = {}): void {
+ override updateOptions(options: IWorkbenchCompressibleObjectTreeOptionsUpdate = {}): void {
super.updateOptions(options);
if (options.overrideStyles) {
@@ -875,7 +875,7 @@ export class WorkbenchDataTree extends DataTree extends Async
this.disposables.add(this.internals);
}
- updateOptions(options: IWorkbenchAsyncDataTreeOptionsUpdate = {}): void {
+ override updateOptions(options: IWorkbenchAsyncDataTreeOptionsUpdate = {}): void {
super.updateOptions(options);
if (options.overrideStyles) {
diff --git a/src/vs/platform/log/common/bufferLog.ts b/src/vs/platform/log/common/bufferLog.ts
index ea9020d9ba7..6cedce2d542 100644
--- a/src/vs/platform/log/common/bufferLog.ts
+++ b/src/vs/platform/log/common/bufferLog.ts
@@ -82,7 +82,7 @@ export class BufferLogService extends AbstractLogger implements ILogService {
this._log(LogLevel.Critical, message, ...args);
}
- dispose(): void {
+ override dispose(): void {
if (this._logger) {
this._logger.dispose();
}
diff --git a/src/vs/platform/log/common/log.ts b/src/vs/platform/log/common/log.ts
index fe4e01b90bf..1e3b749705a 100644
--- a/src/vs/platform/log/common/log.ts
+++ b/src/vs/platform/log/common/log.ts
@@ -250,7 +250,7 @@ export class ConsoleMainLogger extends AbstractLogger implements ILogger {
}
}
- dispose(): void {
+ override dispose(): void {
// noop
}
@@ -303,7 +303,7 @@ export class ConsoleLogger extends AbstractLogger implements ILogger {
}
}
- dispose(): void {
+ override dispose(): void {
// noop
}
@@ -363,7 +363,7 @@ export class AdapterLogger extends AbstractLogger implements ILogger {
return toErrorMessage(msg, this.getLevel() <= LogLevel.Trace);
}
- dispose(): void {
+ override dispose(): void {
// noop
}
@@ -382,7 +382,7 @@ export class MultiplexLogService extends AbstractLogger implements ILogService {
}
}
- setLevel(level: LogLevel): void {
+ override setLevel(level: LogLevel): void {
for (const logService of this.logServices) {
logService.setLevel(level);
}
@@ -431,7 +431,7 @@ export class MultiplexLogService extends AbstractLogger implements ILogService {
}
}
- dispose(): void {
+ override dispose(): void {
for (const logService of this.logServices) {
logService.dispose();
}
@@ -517,7 +517,7 @@ export abstract class AbstractLoggerService extends Disposable implements ILogge
return logger;
}
- dispose(): void {
+ override dispose(): void {
this.logLevelChangeableLoggers.splice(0, this.logLevelChangeableLoggers.length);
this.loggers.forEach(logger => logger.dispose());
this.loggers.clear();
diff --git a/src/vs/platform/log/common/logIpc.ts b/src/vs/platform/log/common/logIpc.ts
index 49d356e9b5e..789b3e02a24 100644
--- a/src/vs/platform/log/common/logIpc.ts
+++ b/src/vs/platform/log/common/logIpc.ts
@@ -115,8 +115,6 @@ export class LoggerChannel implements IServerChannel {
export class LoggerChannelClient extends AbstractLoggerService implements ILoggerService {
- declare readonly _serviceBrand: undefined;
-
constructor(logLevel: LogLevel, onDidChangeLogLevel: Event, private readonly channel: IChannel) {
super(logLevel, onDidChangeLogLevel);
}
@@ -170,14 +168,13 @@ class Logger extends AbstractMessageLogger {
}
export class FollowerLogService extends LogService implements ILogService {
- declare readonly _serviceBrand: undefined;
constructor(private parent: LogLevelChannelClient, logService: ILogService) {
super(logService);
this._register(parent.onDidChangeLogLevel(level => logService.setLevel(level)));
}
- setLevel(level: LogLevel): void {
+ override setLevel(level: LogLevel): void {
super.setLevel(level);
this.parent.setLevel(level);
diff --git a/src/vs/platform/log/node/spdlogLog.ts b/src/vs/platform/log/node/spdlogLog.ts
index 5b036c47ce2..b97064e29f7 100644
--- a/src/vs/platform/log/node/spdlogLog.ts
+++ b/src/vs/platform/log/node/spdlogLog.ts
@@ -95,7 +95,7 @@ export class SpdLogLogger extends AbstractMessageLogger implements ILogger {
}
}
- flush(): void {
+ override flush(): void {
if (this._logger) {
this._logger.flush();
} else {
@@ -103,7 +103,7 @@ export class SpdLogLogger extends AbstractMessageLogger implements ILogger {
}
}
- dispose(): void {
+ override dispose(): void {
if (this._logger) {
this.disposeLogger();
} else {
diff --git a/src/vs/platform/quickinput/browser/commandsQuickAccess.ts b/src/vs/platform/quickinput/browser/commandsQuickAccess.ts
index a1d27c5aca2..96210a56e38 100644
--- a/src/vs/platform/quickinput/browser/commandsQuickAccess.ts
+++ b/src/vs/platform/quickinput/browser/commandsQuickAccess.ts
@@ -39,8 +39,10 @@ export abstract class AbstractCommandsQuickAccessProvider extends PickerQuickAcc
private readonly commandsHistory = this._register(this.instantiationService.createInstance(CommandsHistory));
+ protected override readonly options: ICommandsQuickAccessOptions;
+
constructor(
- protected options: ICommandsQuickAccessOptions,
+ options: ICommandsQuickAccessOptions,
@IInstantiationService private readonly instantiationService: IInstantiationService,
@IKeybindingService private readonly keybindingService: IKeybindingService,
@ICommandService private readonly commandService: ICommandService,
@@ -48,6 +50,8 @@ export abstract class AbstractCommandsQuickAccessProvider extends PickerQuickAcc
@INotificationService private readonly notificationService: INotificationService
) {
super(AbstractCommandsQuickAccessProvider.PREFIX, options);
+
+ this.options = options;
}
protected async getPicks(filter: string, disposables: DisposableStore, token: CancellationToken): Promise> {
diff --git a/src/vs/platform/quickinput/browser/quickInput.ts b/src/vs/platform/quickinput/browser/quickInput.ts
index d696697f9d0..097074264ef 100644
--- a/src/vs/platform/quickinput/browser/quickInput.ts
+++ b/src/vs/platform/quickinput/browser/quickInput.ts
@@ -166,7 +166,7 @@ export class QuickInputService extends Themable implements IQuickInputService {
return this.controller.cancel();
}
- protected updateStyles() {
+ protected override updateStyles() {
this.controller.applyStyles(this.computeStyles());
}
diff --git a/src/vs/platform/remote/node/tunnelService.ts b/src/vs/platform/remote/node/tunnelService.ts
index 0f0bb25118a..a43e2a2487a 100644
--- a/src/vs/platform/remote/node/tunnelService.ts
+++ b/src/vs/platform/remote/node/tunnelService.ts
@@ -58,7 +58,7 @@ class NodeRemoteTunnel extends Disposable implements RemoteTunnel {
this.tunnelRemoteHost = tunnelRemoteHost;
}
- public async dispose(): Promise {
+ public async override dispose(): Promise {
super.dispose();
this._server.removeListener('listening', this._listeningListener);
this._server.removeListener('connection', this._connectionListener);
diff --git a/src/vs/platform/request/electron-main/requestMainService.ts b/src/vs/platform/request/electron-main/requestMainService.ts
index d08f82f4510..ea709437ebb 100644
--- a/src/vs/platform/request/electron-main/requestMainService.ts
+++ b/src/vs/platform/request/electron-main/requestMainService.ts
@@ -14,7 +14,7 @@ function getRawRequest(options: IRequestOptions): IRawRequestFunction {
export class RequestMainService extends NodeRequestService {
- request(options: IRequestOptions, token: CancellationToken): Promise {
+ override request(options: IRequestOptions, token: CancellationToken): Promise {
return super.request({ ...(options || {}), getRawRequest }, token);
}
}
diff --git a/src/vs/platform/storage/browser/storageService.ts b/src/vs/platform/storage/browser/storageService.ts
index 5cfb899602e..c1aae2e7d2e 100644
--- a/src/vs/platform/storage/browser/storageService.ts
+++ b/src/vs/platform/storage/browser/storageService.ts
@@ -94,7 +94,7 @@ export class BrowserStorageService extends AbstractStorageService {
throw new Error('Migrating storage is currently unsupported in Web');
}
- protected shouldFlushWhenIdle(): boolean {
+ protected override shouldFlushWhenIdle(): boolean {
// this flush() will potentially cause new state to be stored
// since new state will only be created while the document
// has focus, one optimization is to not run this when the
diff --git a/src/vs/platform/storage/electron-main/storageMain.ts b/src/vs/platform/storage/electron-main/storageMain.ts
index c9a430dccef..234fb50c997 100644
--- a/src/vs/platform/storage/electron-main/storageMain.ts
+++ b/src/vs/platform/storage/electron-main/storageMain.ts
@@ -206,7 +206,7 @@ export class GlobalStorageMain extends BaseStorageMain implements IStorageMain {
}));
}
- protected async doInit(storage: IStorage): Promise {
+ protected async override doInit(storage: IStorage): Promise {
await super.doInit(storage);
// Apply global telemetry values as part of the initialization
diff --git a/src/vs/platform/storage/test/electron-main/storageMainService.test.ts b/src/vs/platform/storage/test/electron-main/storageMainService.test.ts
index 897a2367af3..0f55ca5c3b2 100644
--- a/src/vs/platform/storage/test/electron-main/storageMainService.test.ts
+++ b/src/vs/platform/storage/test/electron-main/storageMainService.test.ts
@@ -26,7 +26,7 @@ suite('StorageMainService', function () {
class TestStorageMainService extends StorageMainService {
- protected getStorageOptions(): IStorageMainOptions {
+ protected override getStorageOptions(): IStorageMainOptions {
return {
useInMemoryStorage: true
};
diff --git a/src/vs/platform/telemetry/browser/errorTelemetry.ts b/src/vs/platform/telemetry/browser/errorTelemetry.ts
index 6ef338fd841..48bb12478c4 100644
--- a/src/vs/platform/telemetry/browser/errorTelemetry.ts
+++ b/src/vs/platform/telemetry/browser/errorTelemetry.ts
@@ -8,7 +8,7 @@ import { globals } from 'vs/base/common/platform';
import BaseErrorTelemetry, { ErrorEvent } from 'vs/platform/telemetry/common/errorTelemetry';
export default class ErrorTelemetry extends BaseErrorTelemetry {
- protected installErrorListeners(): void {
+ protected override installErrorListeners(): void {
let oldOnError: Function;
let that = this;
if (typeof globals.onerror === 'function') {
diff --git a/src/vs/platform/telemetry/node/errorTelemetry.ts b/src/vs/platform/telemetry/node/errorTelemetry.ts
index 562eb44b9c9..ba1c2a18d46 100644
--- a/src/vs/platform/telemetry/node/errorTelemetry.ts
+++ b/src/vs/platform/telemetry/node/errorTelemetry.ts
@@ -7,7 +7,7 @@ import { onUnexpectedError, setUnexpectedErrorHandler } from 'vs/base/common/err
import BaseErrorTelemetry from 'vs/platform/telemetry/common/errorTelemetry';
export default class ErrorTelemetry extends BaseErrorTelemetry {
- protected installErrorListeners(): void {
+ protected override installErrorListeners(): void {
setUnexpectedErrorHandler(err => console.error(err));
// Print a console message when rejection isn't handled within N seconds. For details:
diff --git a/src/vs/platform/telemetry/test/browser/telemetryService.test.ts b/src/vs/platform/telemetry/test/browser/telemetryService.test.ts
index 936526af0b5..9b8969ffa14 100644
--- a/src/vs/platform/telemetry/test/browser/telemetryService.test.ts
+++ b/src/vs/platform/telemetry/test/browser/telemetryService.test.ts
@@ -217,7 +217,7 @@ suite('TelemetryService', () => {
return Promise.all(this.promises);
}
- publicLog(eventName: string, data?: ITelemetryData, anonymizeFilePaths?: boolean): Promise {
+ override publicLog(eventName: string, data?: ITelemetryData, anonymizeFilePaths?: boolean): Promise {
let p = super.publicLog(eventName, data, anonymizeFilePaths);
this.promises.push(p);
return p;
@@ -773,8 +773,8 @@ suite('TelemetryService', () => {
let service = new TelemetryService({
appender: testAppender
}, new class extends TestConfigurationService {
- onDidChangeConfiguration = emitter.event;
- getValue() {
+ override onDidChangeConfiguration = emitter.event;
+ override getValue() {
return {
enableTelemetry: enableTelemetry
} as any;
diff --git a/src/vs/platform/telemetry/test/common/telemetryLogAppender.test.ts b/src/vs/platform/telemetry/test/common/telemetryLogAppender.test.ts
index ff2be7d5cd5..5ed128e01f3 100644
--- a/src/vs/platform/telemetry/test/common/telemetryLogAppender.test.ts
+++ b/src/vs/platform/telemetry/test/common/telemetryLogAppender.test.ts
@@ -53,7 +53,7 @@ class TestTelemetryLogger extends AbstractLogger implements ILogger {
}
}
- dispose(): void { }
+ override dispose(): void { }
flush(): void { }
}
diff --git a/src/vs/platform/telemetry/test/electron-browser/appInsightsAppender.test.ts b/src/vs/platform/telemetry/test/electron-browser/appInsightsAppender.test.ts
index 07789a3aefc..77c201e4bd2 100644
--- a/src/vs/platform/telemetry/test/electron-browser/appInsightsAppender.test.ts
+++ b/src/vs/platform/telemetry/test/electron-browser/appInsightsAppender.test.ts
@@ -7,8 +7,8 @@ import { AppInsightsAppender } from 'vs/platform/telemetry/node/appInsightsAppen
import { TelemetryClient, Contracts } from 'applicationinsights';
class AppInsightsMock extends TelemetryClient {
- public config: any;
- public channel: any;
+ public override config: any;
+ public override channel: any;
public events: Contracts.EventTelemetry[] = [];
public IsTrackingPageView: boolean = false;
public exceptions: any[] = [];
@@ -17,11 +17,11 @@ class AppInsightsMock extends TelemetryClient {
super('testKey');
}
- public trackEvent(event: any) {
+ public override trackEvent(event: any) {
this.events.push(event);
}
- public flush(options: any): void {
+ public override flush(options: any): void {
// called on dispose
}
}
diff --git a/src/vs/platform/terminal/node/ptyHostService.ts b/src/vs/platform/terminal/node/ptyHostService.ts
index a37d86d3377..507586b21aa 100644
--- a/src/vs/platform/terminal/node/ptyHostService.ts
+++ b/src/vs/platform/terminal/node/ptyHostService.ts
@@ -137,7 +137,7 @@ export class PtyHostService extends Disposable implements IPtyService {
return [client, proxy];
}
- dispose() {
+ override dispose() {
this._isDisposed = true;
super.dispose();
}
diff --git a/src/vs/platform/terminal/node/terminalProcess.ts b/src/vs/platform/terminal/node/terminalProcess.ts
index bd875a24807..a35c7305d30 100644
--- a/src/vs/platform/terminal/node/terminalProcess.ts
+++ b/src/vs/platform/terminal/node/terminalProcess.ts
@@ -232,7 +232,7 @@ export class TerminalProcess extends Disposable implements ITerminalChildProcess
this._sendProcessId(ptyProcess.pid);
}
- public dispose(): void {
+ public override dispose(): void {
this._isDisposed = true;
if (this._titleInterval) {
clearInterval(this._titleInterval);
@@ -494,7 +494,7 @@ class DelayedResizer extends Disposable {
});
}
- dispose(): void {
+ override dispose(): void {
super.dispose();
clearTimeout(this._timeout);
}
diff --git a/src/vs/platform/terminal/node/windowsShellHelper.ts b/src/vs/platform/terminal/node/windowsShellHelper.ts
index 829afecba27..1fed411ed95 100644
--- a/src/vs/platform/terminal/node/windowsShellHelper.ts
+++ b/src/vs/platform/terminal/node/windowsShellHelper.ts
@@ -112,7 +112,7 @@ export class WindowsShellHelper extends Disposable implements IWindowsShellHelpe
return this.traverseTree(tree.children[favouriteChild]);
}
- public dispose(): void {
+ public override dispose(): void {
this._isDisposed = true;
super.dispose();
}
diff --git a/src/vs/platform/undoRedo/test/common/undoRedoService.test.ts b/src/vs/platform/undoRedo/test/common/undoRedoService.test.ts
index 49f4944544b..e4c8cc957df 100644
--- a/src/vs/platform/undoRedo/test/common/undoRedoService.test.ts
+++ b/src/vs/platform/undoRedo/test/common/undoRedoService.test.ts
@@ -133,7 +133,7 @@ suite('UndoRedoService', () => {
const resource1 = URI.file('test1.txt');
const resource2 = URI.file('test2.txt');
const service = createUndoRedoService(new class extends mock() {
- async show() {
+ async override show() {
return {
choice: 0 // confirm!
};
diff --git a/src/vs/platform/update/electron-main/updateService.darwin.ts b/src/vs/platform/update/electron-main/updateService.darwin.ts
index b876e8814dd..8a057282a37 100644
--- a/src/vs/platform/update/electron-main/updateService.darwin.ts
+++ b/src/vs/platform/update/electron-main/updateService.darwin.ts
@@ -19,8 +19,6 @@ import { IProductService } from 'vs/platform/product/common/productService';
export class DarwinUpdateService extends AbstractUpdateService {
- declare readonly _serviceBrand: undefined;
-
private readonly disposables = new DisposableStore();
@memoize private get onRawError(): Event { return Event.fromNodeEventEmitter(electron.autoUpdater, 'error', (_, message) => message); }
@@ -40,7 +38,7 @@ export class DarwinUpdateService extends AbstractUpdateService {
super(lifecycleMainService, configurationService, environmentMainService, requestService, logService, productService);
}
- initialize(): void {
+ override initialize(): void {
super.initialize();
this.onRawError(this.onError, this, this.disposables);
this.onRawUpdateAvailable(this.onUpdateAvailable, this, this.disposables);
@@ -110,7 +108,7 @@ export class DarwinUpdateService extends AbstractUpdateService {
this.setState(State.Idle(UpdateType.Archive));
}
- protected doQuitAndInstall(): void {
+ protected override doQuitAndInstall(): void {
this.logService.trace('update#quitAndInstall(): running raw#quitAndInstall()');
electron.autoUpdater.quitAndInstall();
}
diff --git a/src/vs/platform/update/electron-main/updateService.linux.ts b/src/vs/platform/update/electron-main/updateService.linux.ts
index 7387636bfe7..27d5d4d4f22 100644
--- a/src/vs/platform/update/electron-main/updateService.linux.ts
+++ b/src/vs/platform/update/electron-main/updateService.linux.ts
@@ -17,8 +17,6 @@ import { INativeHostMainService } from 'vs/platform/native/electron-main/nativeH
export class LinuxUpdateService extends AbstractUpdateService {
- declare readonly _serviceBrand: undefined;
-
constructor(
@ILifecycleMainService lifecycleMainService: ILifecycleMainService,
@IConfigurationService configurationService: IConfigurationService,
@@ -62,7 +60,7 @@ export class LinuxUpdateService extends AbstractUpdateService {
});
}
- protected async doDownloadUpdate(state: AvailableForDownload): Promise {
+ protected async override doDownloadUpdate(state: AvailableForDownload): Promise {
// Use the download URL if available as we don't currently detect the package type that was
// installed and the website download page is more useful than the tarball generally.
if (this.productService.downloadUrl && this.productService.downloadUrl.length > 0) {
diff --git a/src/vs/platform/update/electron-main/updateService.snap.ts b/src/vs/platform/update/electron-main/updateService.snap.ts
index e0d6010d06a..505adb5dd38 100644
--- a/src/vs/platform/update/electron-main/updateService.snap.ts
+++ b/src/vs/platform/update/electron-main/updateService.snap.ts
@@ -134,8 +134,6 @@ abstract class AbstractUpdateService2 implements IUpdateService {
export class SnapUpdateService extends AbstractUpdateService2 {
- declare readonly _serviceBrand: undefined;
-
constructor(
private snap: string,
private snapRevision: string,
@@ -175,7 +173,7 @@ export class SnapUpdateService extends AbstractUpdateService2 {
});
}
- protected doQuitAndInstall(): void {
+ protected override doQuitAndInstall(): void {
this.logService.trace('update#quitAndInstall(): running raw#quitAndInstall()');
// Allow 3 seconds for VS Code to close
diff --git a/src/vs/platform/update/electron-main/updateService.win32.ts b/src/vs/platform/update/electron-main/updateService.win32.ts
index 62d82342b36..f033ed74001 100644
--- a/src/vs/platform/update/electron-main/updateService.win32.ts
+++ b/src/vs/platform/update/electron-main/updateService.win32.ts
@@ -49,8 +49,6 @@ function getUpdateType(): UpdateType {
export class Win32UpdateService extends AbstractUpdateService {
- declare readonly _serviceBrand: undefined;
-
private availableUpdate: IAvailableUpdate | undefined;
@memoize
@@ -73,7 +71,7 @@ export class Win32UpdateService extends AbstractUpdateService {
super(lifecycleMainService, configurationService, environmentMainService, requestService, logService, productService);
}
- initialize(): void {
+ override initialize(): void {
super.initialize();
if (getUpdateType() === UpdateType.Setup) {
@@ -177,7 +175,7 @@ export class Win32UpdateService extends AbstractUpdateService {
});
}
- protected async doDownloadUpdate(state: AvailableForDownload): Promise {
+ protected async override doDownloadUpdate(state: AvailableForDownload): Promise {
if (state.update.url) {
this.nativeHostMainService.openExternal(undefined, state.update.url);
}
@@ -206,7 +204,7 @@ export class Win32UpdateService extends AbstractUpdateService {
await Promise.all(promises);
}
- protected async doApplyUpdate(): Promise {
+ protected async override doApplyUpdate(): Promise {
if (this.state.type !== StateType.Downloaded && this.state.type !== StateType.Downloading) {
return Promise.resolve(undefined);
}
@@ -242,7 +240,7 @@ export class Win32UpdateService extends AbstractUpdateService {
.then(() => this.setState(State.Ready(update)));
}
- protected doQuitAndInstall(): void {
+ protected override doQuitAndInstall(): void {
if (this.state.type !== StateType.Ready || !this.availableUpdate) {
return;
}
@@ -259,7 +257,7 @@ export class Win32UpdateService extends AbstractUpdateService {
}
}
- protected getUpdateType(): UpdateType {
+ protected override getUpdateType(): UpdateType {
return getUpdateType();
}
}
diff --git a/src/vs/platform/userDataSync/common/extensionsSync.ts b/src/vs/platform/userDataSync/common/extensionsSync.ts
index 3f96579bf34..09e1cae5fe3 100644
--- a/src/vs/platform/userDataSync/common/extensionsSync.ts
+++ b/src/vs/platform/userDataSync/common/extensionsSync.ts
@@ -95,7 +95,7 @@ export class ExtensionsSynchroniser extends AbstractSynchroniser implements IUse
/* Version 5: Introduce extension state */
protected readonly version: number = 5;
- protected isEnabled(): boolean { return super.isEnabled() && this.extensionGalleryService.isEnabled(); }
+ protected override isEnabled(): boolean { return super.isEnabled() && this.extensionGalleryService.isEnabled(); }
private readonly previewResource: URI = this.extUri.joinPath(this.syncPreviewFolder, 'extensions.json');
private readonly localResource: URI = this.previewResource.with({ scheme: USER_DATA_SYNC_SCHEME, authority: 'local' });
private readonly remoteResource: URI = this.previewResource.with({ scheme: USER_DATA_SYNC_SCHEME, authority: 'remote' });
@@ -292,7 +292,7 @@ export class ExtensionsSynchroniser extends AbstractSynchroniser implements IUse
return [{ resource: this.extUri.joinPath(uri, 'extensions.json'), comparableResource: ExtensionsSynchroniser.EXTENSIONS_DATA_URI }];
}
- async resolveContent(uri: URI): Promise {
+ async override resolveContent(uri: URI): Promise {
if (this.extUri.isEqual(uri, ExtensionsSynchroniser.EXTENSIONS_DATA_URI)) {
const installedExtensions = await this.extensionManagementService.getInstalled();
const ignoredExtensions = this.ignoredExtensionsManagementService.getIgnoredExtensions(installedExtensions);
diff --git a/src/vs/platform/userDataSync/common/globalStateSync.ts b/src/vs/platform/userDataSync/common/globalStateSync.ts
index 2e58f17ae95..8a519f72d64 100644
--- a/src/vs/platform/userDataSync/common/globalStateSync.ts
+++ b/src/vs/platform/userDataSync/common/globalStateSync.ts
@@ -81,7 +81,7 @@ export class GlobalStateSynchroniser extends AbstractSynchroniser implements IUs
@IUserDataSyncStoreService userDataSyncStoreService: IUserDataSyncStoreService,
@IUserDataSyncBackupStoreService userDataSyncBackupStoreService: IUserDataSyncBackupStoreService,
@IUserDataSyncLogService logService: IUserDataSyncLogService,
- @IEnvironmentService readonly environmentService: IEnvironmentService,
+ @IEnvironmentService environmentService: IEnvironmentService,
@IUserDataSyncResourceEnablementService userDataSyncResourceEnablementService: IUserDataSyncResourceEnablementService,
@ITelemetryService telemetryService: ITelemetryService,
@IConfigurationService configurationService: IConfigurationService,
@@ -230,7 +230,7 @@ export class GlobalStateSynchroniser extends AbstractSynchroniser implements IUs
return [{ resource: this.extUri.joinPath(uri, 'globalState.json'), comparableResource: GlobalStateSynchroniser.GLOBAL_STATE_DATA_URI }];
}
- async resolveContent(uri: URI): Promise {
+ async override resolveContent(uri: URI): Promise {
if (this.extUri.isEqual(uri, GlobalStateSynchroniser.GLOBAL_STATE_DATA_URI)) {
const localGlobalState = await this.getLocalGlobalState();
return formatAndStringify(localGlobalState);
diff --git a/src/vs/platform/userDataSync/common/keybindingsSync.ts b/src/vs/platform/userDataSync/common/keybindingsSync.ts
index 7f93636705b..5e7e71d979a 100644
--- a/src/vs/platform/userDataSync/common/keybindingsSync.ts
+++ b/src/vs/platform/userDataSync/common/keybindingsSync.ts
@@ -258,7 +258,7 @@ export class KeybindingsSynchroniser extends AbstractJsonFileSynchroniser implem
return [{ resource: this.extUri.joinPath(uri, 'keybindings.json'), comparableResource }];
}
- async resolveContent(uri: URI): Promise {
+ async override resolveContent(uri: URI): Promise {
if (this.extUri.isEqual(this.remoteResource, uri) || this.extUri.isEqual(this.localResource, uri) || this.extUri.isEqual(this.acceptedResource, uri)) {
return this.resolvePreviewContent(uri);
}
diff --git a/src/vs/platform/userDataSync/common/settingsSync.ts b/src/vs/platform/userDataSync/common/settingsSync.ts
index d4e31ee05b7..245d4cff8fd 100644
--- a/src/vs/platform/userDataSync/common/settingsSync.ts
+++ b/src/vs/platform/userDataSync/common/settingsSync.ts
@@ -247,7 +247,7 @@ export class SettingsSynchroniser extends AbstractJsonFileSynchroniser implement
return [{ resource: this.extUri.joinPath(uri, 'settings.json'), comparableResource }];
}
- async resolveContent(uri: URI): Promise {
+ async override resolveContent(uri: URI): Promise {
if (this.extUri.isEqual(this.remoteResource, uri) || this.extUri.isEqual(this.localResource, uri) || this.extUri.isEqual(this.acceptedResource, uri)) {
return this.resolvePreviewContent(uri);
}
@@ -271,7 +271,7 @@ export class SettingsSynchroniser extends AbstractJsonFileSynchroniser implement
return null;
}
- protected async resolvePreviewContent(resource: URI): Promise {
+ protected async override resolvePreviewContent(resource: URI): Promise {
let content = await super.resolvePreviewContent(resource);
if (content) {
const formatUtils = await this.getFormattingOptions();
diff --git a/src/vs/platform/userDataSync/common/snippetsSync.ts b/src/vs/platform/userDataSync/common/snippetsSync.ts
index 3068afb6c6b..0f4c7841bbe 100644
--- a/src/vs/platform/userDataSync/common/snippetsSync.ts
+++ b/src/vs/platform/userDataSync/common/snippetsSync.ts
@@ -355,7 +355,7 @@ export class SnippetsSynchroniser extends AbstractSynchroniser implements IUserD
return [];
}
- async resolveContent(uri: URI): Promise {
+ async override resolveContent(uri: URI): Promise {
if (this.extUri.isEqualOrParent(uri, this.syncPreviewFolder.with({ scheme: USER_DATA_SYNC_SCHEME, authority: 'remote' }))
|| this.extUri.isEqualOrParent(uri, this.syncPreviewFolder.with({ scheme: USER_DATA_SYNC_SCHEME, authority: 'local' }))
|| this.extUri.isEqualOrParent(uri, this.syncPreviewFolder.with({ scheme: USER_DATA_SYNC_SCHEME, authority: 'accepted' }))) {
diff --git a/src/vs/platform/userDataSync/common/userDataSyncService.ts b/src/vs/platform/userDataSync/common/userDataSyncService.ts
index b059b81ee40..c8f729409d4 100644
--- a/src/vs/platform/userDataSync/common/userDataSyncService.ts
+++ b/src/vs/platform/userDataSync/common/userDataSyncService.ts
@@ -727,7 +727,7 @@ class ManualSyncTask extends Disposable implements IManualSyncTask {
this.synchronizingResources = [];
}
- dispose(): void {
+ override dispose(): void {
this.reset();
this.isDisposed = true;
}
diff --git a/src/vs/platform/userDataSync/common/userDataSyncServiceIpc.ts b/src/vs/platform/userDataSync/common/userDataSyncServiceIpc.ts
index ee55c0be9aa..92304d39bf2 100644
--- a/src/vs/platform/userDataSync/common/userDataSyncServiceIpc.ts
+++ b/src/vs/platform/userDataSync/common/userDataSyncServiceIpc.ts
@@ -356,7 +356,7 @@ class ManualSyncTaskChannelClient extends Disposable implements IManualSyncTask
private _disposed = false;
isDiposed() { return this._disposed; }
- dispose(): void {
+ override dispose(): void {
this._disposed = true;
this.channel.call('dispose');
}
diff --git a/src/vs/platform/userDataSync/test/common/synchronizer.test.ts b/src/vs/platform/userDataSync/test/common/synchronizer.test.ts
index 06d57e78906..4a942e65db8 100644
--- a/src/vs/platform/userDataSync/test/common/synchronizer.test.ts
+++ b/src/vs/platform/userDataSync/test/common/synchronizer.test.ts
@@ -28,20 +28,20 @@ class TestSynchroniser extends AbstractSynchroniser {
onDoSyncCall: Emitter = this._register(new Emitter());
failWhenGettingLatestRemoteUserData: boolean = false;
- readonly resource: SyncResource = SyncResource.Settings;
+ override readonly resource: SyncResource = SyncResource.Settings;
protected readonly version: number = 1;
private cancelled: boolean = false;
readonly localResource = joinPath(this.environmentService.userRoamingDataHome, 'testResource.json');
- protected getLatestRemoteUserData(manifest: IUserDataManifest | null, lastSyncUserData: IRemoteUserData | null): Promise {
+ protected override getLatestRemoteUserData(manifest: IUserDataManifest | null, lastSyncUserData: IRemoteUserData | null): Promise {
if (this.failWhenGettingLatestRemoteUserData) {
throw new Error();
}
return super.getLatestRemoteUserData(manifest, lastSyncUserData);
}
- protected async doSync(remoteUserData: IRemoteUserData, lastSyncUserData: IRemoteUserData | null, apply: boolean): Promise {
+ protected async override doSync(remoteUserData: IRemoteUserData, lastSyncUserData: IRemoteUserData | null, apply: boolean): Promise {
this.cancelled = false;
this.onDoSyncCall.fire();
await this.syncBarrier.wait();
@@ -145,18 +145,18 @@ class TestSynchroniser extends AbstractSynchroniser {
await this.updateLastSyncUserData(remoteUserData);
}
- async stop(): Promise {
+ async override stop(): Promise {
this.cancelled = true;
this.syncBarrier.open();
super.stop();
}
- async triggerLocalChange(): Promise {
+ async override triggerLocalChange(): Promise {
super.triggerLocalChange();
}
onDidTriggerLocalChangeCall: Emitter = this._register(new Emitter