Make single heritage interface print more like the class superClass (#18215)
This commit is contained in:
1 parent
beecab8e2e
commit
5bc3cd61c8
12 files changed
+713
-176
No files matched your search
@@ -1,4 +1,4 @@
|
||||
#### Fix the inconsistent printing between class and interface (#18094, #18091 by @fisker)
|
||||
#### Fix the inconsistent printing between class and interface (#18094, #18091, #18215 by @fisker)
|
||||
|
||||
##### The extra indentation for type parameters in class has been removed
|
||||
|
||||
@@ -72,6 +72,67 @@ export class AreaConfig<ES extends ExprRef | SignalRef>
|
||||
implements MarkConfig<ES>, PointOverlayMixins<ES>, LineOverlayMixins<ES> {}
|
||||
```
|
||||
|
||||
##### Align single heritage print with super class
|
||||
|
||||
<!-- prettier-ignore -->
|
||||
```tsx
|
||||
// Input
|
||||
class ExtendsLongOneWithGenerics
|
||||
extends
|
||||
Bar<
|
||||
SomeLongTypeSomeLongTypeSomeLongTypeSomeLongType,
|
||||
ToBreakLineToBreakLineToBreakLine,
|
||||
> {}
|
||||
class ExtendsLongOneWithGenerics
|
||||
implements
|
||||
Bar<
|
||||
SomeLongTypeSomeLongTypeSomeLongTypeSomeLongType,
|
||||
ToBreakLineToBreakLineToBreakLine,
|
||||
> {}
|
||||
interface ExtendsLongOneWithGenerics
|
||||
extends
|
||||
Bar<
|
||||
SomeLongTypeSomeLongTypeSomeLongTypeSomeLongType,
|
||||
ToBreakLineToBreakLineToBreakLine,
|
||||
> {}
|
||||
|
||||
// Prettier stable
|
||||
class ExtendsLongOneWithGenerics extends Bar<
|
||||
SomeLongTypeSomeLongTypeSomeLongTypeSomeLongType,
|
||||
ToBreakLineToBreakLineToBreakLine
|
||||
> {}
|
||||
class ExtendsLongOneWithGenerics
|
||||
implements
|
||||
Bar<
|
||||
SomeLongTypeSomeLongTypeSomeLongTypeSomeLongType,
|
||||
ToBreakLineToBreakLineToBreakLine
|
||||
> {}
|
||||
interface ExtendsLongOneWithGenerics
|
||||
extends Bar<
|
||||
SomeLongTypeSomeLongTypeSomeLongTypeSomeLongType,
|
||||
ToBreakLineToBreakLineToBreakLine
|
||||
> {}
|
||||
|
||||
// Prettier main
|
||||
interface Interface extends GenericEnvironment<
|
||||
SomeType,
|
||||
AnotherType,
|
||||
YetAnotherType
|
||||
> {}
|
||||
class ExtendsLongOneWithGenerics extends Bar<
|
||||
SomeLongTypeSomeLongTypeSomeLongTypeSomeLongType,
|
||||
ToBreakLineToBreakLineToBreakLine
|
||||
> {}
|
||||
class ExtendsLongOneWithGenerics implements Bar<
|
||||
SomeLongTypeSomeLongTypeSomeLongTypeSomeLongType,
|
||||
ToBreakLineToBreakLineToBreakLine
|
||||
> {}
|
||||
interface ExtendsLongOneWithGenerics extends Bar<
|
||||
SomeLongTypeSomeLongTypeSomeLongTypeSomeLongType,
|
||||
ToBreakLineToBreakLineToBreakLine
|
||||
> {}
|
||||
```
|
||||
|
||||
##### Inconsistent opening brace print logic of class and interface body
|
||||
|
||||
In Prettier v2.3, to improve visual separation between class head and body, [we start to print `{` of class body on a new line when the class has multiple heritages](https://prettier.io/blog/2021/05/09/2.3.0.html#improve-visual-separation-between-header-and-body-in-classes-with-multiline-headers-10085-by-sosukesuzuki).
|
||||
|
||||
Reference in new issue
Block a user