diff --git a/src/services/formatting/formattingContext.ts b/src/services/formatting/formattingContext.ts index b7573526f78..52de4105df6 100644 --- a/src/services/formatting/formattingContext.ts +++ b/src/services/formatting/formattingContext.ts @@ -29,7 +29,7 @@ module ts.formatting { private contextNodeBlockIsOnOneLine: boolean; private nextNodeBlockIsOnOneLine: boolean; - constructor(private sourceFile: SourceFile, public formattingRequestKind: FormattingRequestKind) { + constructor(public sourceFile: SourceFile, public formattingRequestKind: FormattingRequestKind) { } public updateContext(currentRange: TextRangeWithKind, currentTokenParent: Node, nextRange: TextRangeWithKind, nextTokenParent: Node, commonParent: Node) { diff --git a/src/services/formatting/rules.ts b/src/services/formatting/rules.ts index b5470579787..697463831e1 100644 --- a/src/services/formatting/rules.ts +++ b/src/services/formatting/rules.ts @@ -92,6 +92,7 @@ module ts.formatting { public NoSpaceBeforeComma: Rule; public SpaceAfterCertainKeywords: Rule; + public SpaceAfterLetConstInVariableDeclaration: Rule; public NoSpaceBeforeOpenParenInFuncCall: Rule; public SpaceAfterFunctionInFuncDecl: Rule; public NoSpaceBeforeOpenParenInFuncDecl: Rule; @@ -283,6 +284,7 @@ module ts.formatting { this.NoSpaceBeforeComma = new Rule(RuleDescriptor.create2(Shared.TokenRange.Any, SyntaxKind.CommaToken), RuleOperation.create2(new RuleOperationContext(Rules.IsSameLineTokenContext), RuleAction.Delete)); this.SpaceAfterCertainKeywords = new Rule(RuleDescriptor.create4(Shared.TokenRange.FromTokens([SyntaxKind.VarKeyword, SyntaxKind.ThrowKeyword, SyntaxKind.NewKeyword, SyntaxKind.DeleteKeyword, SyntaxKind.ReturnKeyword, SyntaxKind.TypeOfKeyword]), Shared.TokenRange.Any), RuleOperation.create2(new RuleOperationContext(Rules.IsSameLineTokenContext), RuleAction.Space)); + this.SpaceAfterLetConstInVariableDeclaration = new Rule(RuleDescriptor.create4(Shared.TokenRange.FromTokens([SyntaxKind.LetKeyword, SyntaxKind.ConstKeyword]), Shared.TokenRange.Any), RuleOperation.create2(new RuleOperationContext(Rules.IsSameLineTokenContext, Rules.IsStartOfVariableDeclarationList), RuleAction.Space)); this.NoSpaceBeforeOpenParenInFuncCall = new Rule(RuleDescriptor.create2(Shared.TokenRange.Any, SyntaxKind.OpenParenToken), RuleOperation.create2(new RuleOperationContext(Rules.IsSameLineTokenContext, Rules.IsFunctionCallOrNewContext, Rules.IsPreviousTokenNotComma), RuleAction.Delete)); this.SpaceAfterFunctionInFuncDecl = new Rule(RuleDescriptor.create3(SyntaxKind.FunctionKeyword, Shared.TokenRange.Any), RuleOperation.create2(new RuleOperationContext(Rules.IsFunctionDeclContext), RuleAction.Space)); this.NoSpaceBeforeOpenParenInFuncDecl = new Rule(RuleDescriptor.create2(Shared.TokenRange.Any, SyntaxKind.OpenParenToken), RuleOperation.create2(new RuleOperationContext(Rules.IsSameLineTokenContext, Rules.IsFunctionDeclContext), RuleAction.Delete)); @@ -356,6 +358,7 @@ module ts.formatting { this.SpaceAfterFunctionInFuncDecl, this.NewLineAfterOpenBraceInBlockContext, this.SpaceAfterGetSetInMember, this.NoSpaceBetweenReturnAndSemicolon, this.SpaceAfterCertainKeywords, + this.SpaceAfterLetConstInVariableDeclaration, this.NoSpaceBeforeOpenParenInFuncCall, this.SpaceBeforeBinaryKeywordOperator, this.SpaceAfterBinaryKeywordOperator, this.SpaceAfterVoidOperator, @@ -634,6 +637,11 @@ module ts.formatting { return context.TokensAreOnSameLine(); } + static IsStartOfVariableDeclarationList(context: FormattingContext): boolean { + return context.currentTokenParent.kind === SyntaxKind.VariableDeclarationList && + context.currentTokenParent.getStart(context.sourceFile) === context.currentTokenSpan.pos; + } + static IsNotFormatOnEnter(context: FormattingContext): boolean { return context.formattingRequestKind != FormattingRequestKind.FormatOnEnter; } diff --git a/tests/cases/fourslash/formattingInDestructuring1.ts b/tests/cases/fourslash/formattingInDestructuring1.ts new file mode 100644 index 00000000000..21865b07f72 --- /dev/null +++ b/tests/cases/fourslash/formattingInDestructuring1.ts @@ -0,0 +1,22 @@ +/// + +////interface let { } +/////*1*/var x: let []; +//// +////function foo() { +//// 'use strict' +/////*2*/ let [x] = []; +/////*3*/ const [x] = []; +/////*4*/ for (let[x] = [];x < 1;) { +//// } +////} + +format.document(); +goTo.marker("1"); +verify.currentLineContentIs("var x: let[];"); +goTo.marker("2"); +verify.currentLineContentIs(" let [x] = [];"); +goTo.marker("3"); +verify.currentLineContentIs(" const [x] = [];"); +goTo.marker("4"); +verify.currentLineContentIs(" for (let [x] = []; x < 1;) {"); \ No newline at end of file