[scss] IntelliSense Suggestions After Semicolon #250

src/vs/languages/css/common/services/intelliSense.ts

@@ -105,7 +105,10 @@ export class CSSIntellisense {
 	}
 
 	public getCompletionsForDeclarationProperty(result:Modes.ISuggestion[]):Modes.ISuggestion[] {
+		return this.getPropertyProposals(result);
+	}
 
+	private getPropertyProposals(result:Modes.ISuggestion[]):Modes.ISuggestion[] {
 		var properties = languageFacts.getProperties();
 
 		for (var key in properties) {
@@ -341,7 +344,7 @@ export class CSSIntellisense {
 		if (ruleSet && ruleSet.isNested()) {
 			var selector = ruleSet.getSelectors().findFirstChildBeforeOffset(this.offset);
 			if (selector && ruleSet.getSelectors().getChildren().indexOf(selector) === 0) {
-				this.getCompletionsForDeclarationProperty(result);
+				this.getPropertyProposals(result);
 			}
 		}
 		return result;

src/vs/languages/sass/common/services/intelliSense.ts

@@ -148,9 +148,9 @@ export class SASSIntellisense extends cssIntellisense.CSSIntellisense {
 		return super.getColorProposals(entry, result);
 	}
 
-	public getCompletionsForDeclarations(declarations: nodes.Declarations, result: Modes.ISuggestion[]): Modes.ISuggestion[]{
+	public getCompletionsForDeclarationProperty(result: Modes.ISuggestion[]): Modes.ISuggestion[]{
 		this.getCompletionsForSelector(null, result);
-		return super.getCompletionsForDeclarations(declarations, result);
+		return super.getCompletionsForDeclarationProperty(result);
 	}
 
 }

src/vs/languages/sass/test/common/sass-worker.test.ts

@@ -112,6 +112,11 @@ suite('SASS - Worker', () => {
 			testSuggestionsFor('.foo { .', '{ .').then((completion) => {
 				assert.equal(completion.currentWord, '');
 				assertSuggestion(completion, '.foo');
+			}),
+			// issue #250 
+			testSuggestionsFor('.foo { display: block;', 'block;').then((completion) => {
+				assert.equal(completion.currentWord, '');
+				assert.equal(0, completion.suggestions.length);
 			})
 		]).done(() => testDone(), (errors:any[]) => {
 			testDone(errors.reduce((e1, e2) => e1 || e2));