Skip to content

Commit 3d8252c

Browse files
Kingwlweswigham
authored andcommitted
avoid space before dot question (#33840)
1 parent 01b3d41 commit 3d8252c

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

src/services/formatting/rules.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,8 @@ namespace ts.formatting {
5959
// in other cases there should be no space between '?' and next token
6060
rule("NoSpaceAfterQuestionMark", SyntaxKind.QuestionToken, anyToken, [isNonJsxSameLineTokenContext], RuleAction.DeleteSpace),
6161

62-
rule("NoSpaceBeforeDot", anyToken, SyntaxKind.DotToken, [isNonJsxSameLineTokenContext], RuleAction.DeleteSpace),
63-
rule("NoSpaceAfterDot", SyntaxKind.DotToken, anyToken, [isNonJsxSameLineTokenContext], RuleAction.DeleteSpace),
62+
rule("NoSpaceBeforeDot", anyToken, [SyntaxKind.DotToken, SyntaxKind.QuestionDotToken], [isNonJsxSameLineTokenContext], RuleAction.DeleteSpace),
63+
rule("NoSpaceAfterDot", [SyntaxKind.DotToken, SyntaxKind.QuestionDotToken], anyToken, [isNonJsxSameLineTokenContext], RuleAction.DeleteSpace),
6464

6565
rule("NoSpaceBetweenImportParenInImportType", SyntaxKind.ImportKeyword, SyntaxKind.OpenParenToken, [isNonJsxSameLineTokenContext, isImportTypeContext], RuleAction.DeleteSpace),
6666

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
/// <reference path='fourslash.ts' />
2+
3+
/////*1*/a ?. b ?. c . d;
4+
/////*2*/o . m() ?. length;
5+
6+
format.document();
7+
goTo.marker("1");
8+
verify.currentLineContentIs("a?.b?.c.d;");
9+
goTo.marker("2");
10+
verify.currentLineContentIs("o.m()?.length;");

0 commit comments

Comments
 (0)