diff --git a/src/vs/editor/common/modes.ts b/src/vs/editor/common/modes.ts index 0583a5569cc75738ce2917a89bd90ae1b5231620..5e3fd67cbb8fc87a27486e443e2d6400db4ea8c1 100644 --- a/src/vs/editor/common/modes.ts +++ b/src/vs/editor/common/modes.ts @@ -339,8 +339,6 @@ export interface ISuggestSupport { triggerCharacters: string[]; - shouldAutotriggerSuggest: boolean; - filter?: IFilter; provideCompletionItems(model:editorCommon.IReadOnlyModel, position:Position, token:CancellationToken): ISuggestResult[] | Thenable; diff --git a/src/vs/editor/common/modes/supports/suggestSupport.ts b/src/vs/editor/common/modes/supports/suggestSupport.ts index 9bf09c0a2cd1a38603f2f046a9e422f9b1932176..500e654c5bb9751d5522b03c6b4bbcb8963b42a2 100644 --- a/src/vs/editor/common/modes/supports/suggestSupport.ts +++ b/src/vs/editor/common/modes/supports/suggestSupport.ts @@ -35,10 +35,6 @@ export class TextualSuggestSupport implements ISuggestSupport { return []; } - public get shouldAutotriggerSuggest(): boolean { - return true; - } - public get filter(): IFilter { return matchesStrictPrefix; } diff --git a/src/vs/editor/contrib/suggest/browser/suggest.ts b/src/vs/editor/contrib/suggest/browser/suggest.ts index caea6a07f9f226613c4b074a9c5a6951de2fbf56..70b98b6a70198f714ec50ef988a89b537397d38e 100644 --- a/src/vs/editor/contrib/suggest/browser/suggest.ts +++ b/src/vs/editor/contrib/suggest/browser/suggest.ts @@ -129,10 +129,6 @@ export class SuggestController implements IEditorContribution { } private triggerCharacterHandler(character: string, groups: ISuggestSupport[][]): void { - groups = groups.map(supports => { - return supports.filter(support => support.shouldAutotriggerSuggest); - }); - if (groups.length > 0) { this.triggerSuggest(character, groups).done(null, onUnexpectedError); } diff --git a/src/vs/editor/contrib/suggest/browser/suggestModel.ts b/src/vs/editor/contrib/suggest/browser/suggestModel.ts index 73a8221b2ef348566078d33a890f19b80848bcca..fccd01c8a006e76cfda94efcb4a6401f5148be69 100644 --- a/src/vs/editor/contrib/suggest/browser/suggestModel.ts +++ b/src/vs/editor/contrib/suggest/browser/suggestModel.ts @@ -82,7 +82,7 @@ class Context { } const supports = SuggestRegistry.all(model); - this.isAutoTriggerEnabled = supports.some(s => s.shouldAutotriggerSuggest); + this.isAutoTriggerEnabled = (supports.length > 0); } public shouldAutoTrigger(): boolean { diff --git a/src/vs/languages/css/common/css.ts b/src/vs/languages/css/common/css.ts index ee6d165c4451e2e797fb133e433140daf86b02c3..a52b08584e9b66c45604d56ad48c1c04cd085a93 100644 --- a/src/vs/languages/css/common/css.ts +++ b/src/vs/languages/css/common/css.ts @@ -359,7 +359,6 @@ export class CSSMode extends AbstractMode { modes.SuggestRegistry.register(this.getId(), { triggerCharacters: [' ', ':'], - shouldAutotriggerSuggest: true, provideCompletionItems: (model, position, token): Thenable => { return wireCancellationToken(token, this._provideCompletionItems(model.uri, position)); } diff --git a/src/vs/languages/handlebars/common/handlebars.ts b/src/vs/languages/handlebars/common/handlebars.ts index 289fee3277c38db0cb5b47a0cc2985fe583c2f8b..300ce3544ea562ebd98b355f243ce576b842633c 100644 --- a/src/vs/languages/handlebars/common/handlebars.ts +++ b/src/vs/languages/handlebars/common/handlebars.ts @@ -131,7 +131,6 @@ export class HandlebarsMode extends htmlMode.HTMLMode { modes.SuggestRegistry.register(this.getId(), { triggerCharacters: ['.', ':', '<', '"', '=', '/'], - shouldAutotriggerSuggest: true, provideCompletionItems: (model, position, token): Thenable => { return wireCancellationToken(token, this._provideCompletionItems(model.uri, position)); } diff --git a/src/vs/languages/html/common/html.ts b/src/vs/languages/html/common/html.ts index 1e954d6ff90557970aa1a6b64ad5c92f6328d70c..29da5657d31d970e836e6e74cfd00703ce2b1989 100644 --- a/src/vs/languages/html/common/html.ts +++ b/src/vs/languages/html/common/html.ts @@ -339,7 +339,6 @@ export class HTMLMode extends AbstractMode impl modes.SuggestRegistry.register(this.getId(), { triggerCharacters: ['.', ':', '<', '"', '=', '/'], - shouldAutotriggerSuggest: true, provideCompletionItems: (model, position, token): Thenable => { return wireCancellationToken(token, this._provideCompletionItems(model.uri, position)); } diff --git a/src/vs/languages/json/common/json.ts b/src/vs/languages/json/common/json.ts index d7c37175ea98e4009169dd7fb26367e769b3c07a..e250f0ca23d577121fd27d0331f8dde433d9e140 100644 --- a/src/vs/languages/json/common/json.ts +++ b/src/vs/languages/json/common/json.ts @@ -94,7 +94,6 @@ export class JSONMode extends AbstractMode { modes.SuggestRegistry.register(this.getId(), { triggerCharacters: [], - shouldAutotriggerSuggest: true, provideCompletionItems: (model, position, token): Thenable => { return wireCancellationToken(token, this._provideCompletionItems(model.uri, position)); } diff --git a/src/vs/languages/less/common/less.ts b/src/vs/languages/less/common/less.ts index 7efbfa8d827c0c2678f6585bf46c66b6d937d4fc..ac077ada0a5d54d28d7e9bdcdbdfe62f5b10382c 100644 --- a/src/vs/languages/less/common/less.ts +++ b/src/vs/languages/less/common/less.ts @@ -233,7 +233,6 @@ export class LESSMode extends AbstractMode { modes.SuggestRegistry.register(this.getId(), { triggerCharacters: [], - shouldAutotriggerSuggest: true, provideCompletionItems: (model, position, token): Thenable => { return wireCancellationToken(token, this._provideCompletionItems(model.uri, position)); } diff --git a/src/vs/languages/markdown/common/markdown.ts b/src/vs/languages/markdown/common/markdown.ts index 079c9b697bcd0f27516269c57049acb00a973804..00642e2894d036a1cb5e79f54aa8b719296898b7 100644 --- a/src/vs/languages/markdown/common/markdown.ts +++ b/src/vs/languages/markdown/common/markdown.ts @@ -239,7 +239,6 @@ export class MarkdownMode extends AbstractMode implements Modes.IEmitOutputSuppo Modes.SuggestRegistry.register(this.getId(), { triggerCharacters: [], - shouldAutotriggerSuggest: false, provideCompletionItems: (model, position, token) => { return wireCancellationToken(token, editorWorkerService.textualSuggest(model.uri, position)); } diff --git a/src/vs/languages/razor/common/razor.ts b/src/vs/languages/razor/common/razor.ts index c84e55c073c696f18d4b02c8a10e3ffae6888a0c..75869ba42ddf231832e40aa630a944fa25b83d87 100644 --- a/src/vs/languages/razor/common/razor.ts +++ b/src/vs/languages/razor/common/razor.ts @@ -80,7 +80,6 @@ export class RAZORMode extends htmlMode.HTMLMode { modes.SuggestRegistry.register(this.getId(), { triggerCharacters: ['.', ':', '<', '"', '=', '/'], - shouldAutotriggerSuggest: true, provideCompletionItems: (model, position, token): Thenable => { return wireCancellationToken(token, this._provideCompletionItems(model.uri, position)); } diff --git a/src/vs/languages/sass/common/sass.ts b/src/vs/languages/sass/common/sass.ts index 3957e40c6e4dbae01c884e0a1c5a5273a1d65794..f4a22a1cc37617e2a953f7e8d476fbd4c787318f 100644 --- a/src/vs/languages/sass/common/sass.ts +++ b/src/vs/languages/sass/common/sass.ts @@ -334,7 +334,6 @@ export class SASSMode extends AbstractMode { modes.SuggestRegistry.register(this.getId(), { triggerCharacters: [], - shouldAutotriggerSuggest: true, provideCompletionItems: (model, position, token): Thenable => { return wireCancellationToken(token, this._provideCompletionItems(model.uri, position)); } diff --git a/src/vs/languages/typescript/common/languageFeatures.ts b/src/vs/languages/typescript/common/languageFeatures.ts index f77b2c5e1545efca68318697785b47b9b0eccc1b..d40f3bd80db1e72229c84cbd8f5d3f0fb548fdf2 100644 --- a/src/vs/languages/typescript/common/languageFeatures.ts +++ b/src/vs/languages/typescript/common/languageFeatures.ts @@ -172,10 +172,6 @@ class SuggestAdapter extends Adapter implements modes.ISuggestSupport { return ['.']; } - public get shouldAutotriggerSuggest(): boolean { - return true; - } - provideCompletionItems(model:editorCommon.IReadOnlyModel, position:Position, token:CancellationToken): Thenable { const wordInfo = model.getWordUntilPosition(position); const resource = model.uri; diff --git a/src/vs/workbench/api/node/extHostLanguageFeatures.ts b/src/vs/workbench/api/node/extHostLanguageFeatures.ts index e57f93eeffd077ad14542c478ffdbc357fd39e78..528e1b60c1febefc40e419547fba5240f06f3882 100644 --- a/src/vs/workbench/api/node/extHostLanguageFeatures.ts +++ b/src/vs/workbench/api/node/extHostLanguageFeatures.ts @@ -976,7 +976,6 @@ export class MainThreadLanguageFeatures { $registerSuggestSupport(handle: number, selector: vscode.DocumentSelector, triggerCharacters: string[]): TPromise { this._registrations[handle] = modes.SuggestRegistry.register(selector, { triggerCharacters: triggerCharacters, - shouldAutotriggerSuggest: true, provideCompletionItems: (model:IReadOnlyModel, position:EditorPosition, token:CancellationToken): Thenable => { return wireCancellationToken(token, this._proxy.$provideCompletionItems(handle, model.uri, position)); },