Add formatting rule for import type import keyword and open paren (#2…

…3872)
microsoft · May 3, 2018 · 0d7d75e · 0d7d75e
1 parent 406ca06
commit 0d7d75e
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/src/services/formatting/rules.ts b/src/services/formatting/rules.ts
@@ -59,6 +59,8 @@ namespace ts.formatting {
             rule("NoSpaceBeforeDot", anyToken, SyntaxKind.DotToken, [isNonJsxSameLineTokenContext], RuleAction.Delete),
             rule("NoSpaceAfterDot", SyntaxKind.DotToken, anyToken, [isNonJsxSameLineTokenContext], RuleAction.Delete),
 
+            rule("NoSpaceBetweenImportParenInImportType", SyntaxKind.ImportKeyword, SyntaxKind.OpenParenToken, [isNonJsxSameLineTokenContext, isImportTypeContext], RuleAction.Delete),
+
             // Special handling of unary operators.
             // Prefix operators generally shouldn't have a space between
             // them and their target unary expression.
@@ -641,6 +643,10 @@ namespace ts.formatting {
         return context.contextNode.kind === SyntaxKind.ArrowFunction;
     }
 
+    function isImportTypeContext(context: FormattingContext): boolean {
+        return context.contextNode.kind === SyntaxKind.ImportType;
+    }
+
     function isNonJsxSameLineTokenContext(context: FormattingContext): boolean {
         return context.TokensAreOnSameLine() && context.contextNode.kind !== SyntaxKind.JsxText;
     }

diff --git a/tests/cases/fourslash/importTypeFormatting.ts b/tests/cases/fourslash/importTypeFormatting.ts
@@ -0,0 +1,9 @@
+/// <reference path="fourslash.ts"/>
+
+////var y: import("./c2").mytype;
+////var z: import ("./c2").mytype;
+
+format.document();
+verify.currentFileContentIs(
+`var y: import("./c2").mytype;
+var z: import("./c2").mytype;`);