Restrict html/css to only handling file and untitled files. Part of microsoft#48777

octref · octref · commit 0dfeec2eefa7 · 2018-04-26T10:58:35.000-07:00
diff --git a/extensions/css-language-features/client/src/cssMain.ts b/extensions/css-language-features/client/src/cssMain.ts
@@ -27,7 +27,14 @@ export function activate(context: ExtensionContext) {
 		debug: { module: serverModule, transport: TransportKind.ipc, options: debugOptions }
 	};
 
-	let documentSelector = ['css', 'scss', 'less'];
+	let documentSelector = [
+		{ language: 'css', scheme: 'file' },
+		{ language: 'css', scheme: 'untitled' },
+		{ language: 'scss', scheme: 'file' },
+		{ language: 'scss', scheme: 'untitled' },
+		{ language: 'less', scheme: 'file' },
+		{ language: 'less', scheme: 'untitled' },
+	];
 
 	// Options to control the language client
 	let clientOptions: LanguageClientOptions = {
diff --git a/extensions/html-language-features/client/src/htmlMain.ts b/extensions/html-language-features/client/src/htmlMain.ts
@@ -47,7 +47,14 @@ export function activate(context: ExtensionContext) {
 		debug: { module: serverModule, transport: TransportKind.ipc, options: debugOptions }
 	};
 
-	let documentSelector = ['html', 'handlebars', 'razor'];
+	let documentSelector = [
+		{ language: 'html', scheme: 'file' },
+		{ language: 'html', scheme: 'untitled' },
+		{ language: 'handlebars', scheme: 'file' },
+		{ language: 'handlebars', scheme: 'untitled' },
+		{ language: 'razor', scheme: 'file' },
+		{ language: 'razor', scheme: 'untitled' }
+	];
 	let embeddedLanguages = { css: true, javascript: true };
 
 	// Options to control the language client
diff --git a/extensions/html-language-features/server/src/htmlServerMain.ts b/extensions/html-language-features/server/src/htmlServerMain.ts
@@ -171,7 +171,12 @@ connection.onDidChangeConfiguration((change) => {
 		let enableFormatter = globalSettings && globalSettings.html && globalSettings.html.format && globalSettings.html.format.enable;
 		if (enableFormatter) {
 			if (!formatterRegistration) {
-				let documentSelector: DocumentSelector = [{ language: 'html' }, { language: 'handlebars' }]; // don't register razor, the formatter does more harm than good
+				let documentSelector: DocumentSelector = [
+					{ language: 'html', scheme: 'file' },
+					{ language: 'html', scheme: 'untitled' },
+					{ language: 'handlebars', scheme: 'file' },
+					{ language: 'handlebars', scheme: 'untitled' }
+				]; // don't register razor, the formatter does more harm than good
 				formatterRegistration = connection.client.register(DocumentRangeFormattingRequest.type, { documentSelector });
 			}
 		} else if (formatterRegistration) {
