Skip to content

Commit f2c7205

Browse files
authored
Revert "fix: workaround for es2015 inheritance not always working (#13834)" (#13868)
This reverts commit a22a9fa. This change caused a new issue where Closure Compiler was not able to do dead-code elimination due to the function call with unknown side effects.
1 parent 7cfa0c5 commit f2c7205

File tree

15 files changed

+23
-51
lines changed

15 files changed

+23
-51
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
### Bug Fixes
66

7+
* **build:** material not working with ES2015 ([#13709](https://github.com/angular/material2/issues/13709)) ([e9103a6](https://github.com/angular/material2/commit/e9103a6)), closes [#12760](https://github.com/angular/material2/issues/12760) [#13695](https://github.com/angular/material2/issues/13695)
78
* **button-toggle:** webkit tap highlight conflicting with ripples ([#13721](https://github.com/angular/material2/issues/13721)) ([abd0278](https://github.com/angular/material2/commit/abd0278))
89
* **cdk-platform:** pass `{}` to `@NgModule` since passing nothing breaks ([#13792](https://github.com/angular/material2/issues/13792)) ([5abb644](https://github.com/angular/material2/commit/5abb644))
910
* **checkbox:** disabled property not being coerced ([#13755](https://github.com/angular/material2/issues/13755)) ([cee8c65](https://github.com/angular/material2/commit/cee8c65)), closes [#13739](https://github.com/angular/material2/issues/13739)

src/lib/core/constructor-metadata-inherit.ts

Lines changed: 0 additions & 21 deletions
This file was deleted.

src/lib/core/public-api.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,4 +20,3 @@ export * from './selection/index';
2020

2121
// TODO: don't have this
2222
export * from './testing/month-constants';
23-
export * from './constructor-metadata-inherit';

src/lib/input/autosize.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88

99
import {CdkTextareaAutosize} from '@angular/cdk/text-field';
1010
import {Directive, Input} from '@angular/core';
11-
import {_inheritCtorParametersMetadata} from '@angular/material/core';
1211

1312
/**
1413
* Directive to automatically resize a textarea to fit its content.
@@ -46,4 +45,5 @@ export class MatTextareaAutosize extends CdkTextareaAutosize {
4645
}
4746

4847
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
49-
_inheritCtorParametersMetadata(MatTextareaAutosize, CdkTextareaAutosize);
48+
(MatTextareaAutosize as any)['ctorParameters'] = () =>
49+
(CdkTextareaAutosize as any)['ctorParameters'];

src/lib/stepper/step-label.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,11 @@
88

99
import {Directive} from '@angular/core';
1010
import {CdkStepLabel} from '@angular/cdk/stepper';
11-
import {_inheritCtorParametersMetadata} from '@angular/material/core';
1211

1312
@Directive({
1413
selector: '[matStepLabel]',
1514
})
1615
export class MatStepLabel extends CdkStepLabel {}
1716

1817
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
19-
_inheritCtorParametersMetadata(MatStepLabel, CdkStepLabel);
18+
(MatStepLabel as any)['ctorParameters'] = () => (CdkStepLabel as any)['ctorParameters'];

src/lib/stepper/stepper-button.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
import {Directive} from '@angular/core';
1010
import {CdkStepper, CdkStepperNext, CdkStepperPrevious} from '@angular/cdk/stepper';
1111
import {MatStepper} from './stepper';
12-
import {_inheritCtorParametersMetadata} from '@angular/material/core';
1312

1413
/** Button that moves to the next step in a stepper workflow. */
1514
@Directive({
@@ -36,5 +35,6 @@ export class MatStepperNext extends CdkStepperNext {}
3635
export class MatStepperPrevious extends CdkStepperPrevious {}
3736

3837
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
39-
_inheritCtorParametersMetadata(MatStepperNext, CdkStepperNext);
40-
_inheritCtorParametersMetadata(MatStepperPrevious, CdkStepperPrevious);
38+
(MatStepperNext as any)['ctorParameters'] = () => (CdkStepperNext as any)['ctorParameters'];
39+
(MatStepperPrevious as any)['ctorParameters'] = () =>
40+
(CdkStepperPrevious as any)['ctorParameters'];

src/lib/stepper/stepper.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ import {
3838
} from '@angular/core';
3939
import {FormControl, FormGroupDirective, NgForm} from '@angular/forms';
4040
import {DOCUMENT} from '@angular/common';
41-
import {_inheritCtorParametersMetadata, ErrorStateMatcher} from '@angular/material/core';
41+
import {ErrorStateMatcher} from '@angular/material/core';
4242
import {Subject} from 'rxjs';
4343
import {takeUntil, distinctUntilChanged} from 'rxjs/operators';
4444

@@ -124,7 +124,7 @@ export class MatStepper extends CdkStepper implements AfterContentInit {
124124
}
125125

126126
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
127-
_inheritCtorParametersMetadata(MatStepper, CdkStepper);
127+
(MatStepper as any)['ctorParameters'] = () => (CdkStepper as any)['ctorParameters'];
128128

129129
@Component({
130130
moduleId: module.id,

src/lib/table/cell.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ import {
1414
CdkHeaderCell,
1515
CdkHeaderCellDef,
1616
} from '@angular/cdk/table';
17-
import {_inheritCtorParametersMetadata} from '@angular/material/core';
1817

1918
/**
2019
* Cell definition for the mat-table.
@@ -47,9 +46,10 @@ export class MatHeaderCellDef extends CdkHeaderCellDef {}
4746
export class MatFooterCellDef extends CdkFooterCellDef {}
4847

4948
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
50-
_inheritCtorParametersMetadata(MatCellDef, CdkCellDef);
51-
_inheritCtorParametersMetadata(MatHeaderCellDef, CdkHeaderCellDef);
52-
_inheritCtorParametersMetadata(MatFooterCellDef, CdkFooterCellDef);
49+
(MatCellDef as any)['ctorParameters'] = () => (CdkCellDef as any)['ctorParameters'];
50+
(MatHeaderCellDef as any)['ctorParameters'] = () => (CdkHeaderCellDef as any)['ctorParameters'];
51+
(MatFooterCellDef as any)['ctorParameters'] = () => (MatFooterCellDef as any)['ctorParameters'];
52+
5353
/**
5454
* Column definition for the mat-table.
5555
* Defines a set of cells available for a table column.

src/lib/table/row.ts

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ import {
1919
CdkRow,
2020
CdkRowDef,
2121
} from '@angular/cdk/table';
22-
import {_inheritCtorParametersMetadata} from '@angular/material/core';
2322

2423
/**
2524
* Header row definition for the mat-table.
@@ -56,9 +55,9 @@ export class MatFooterRowDef extends CdkFooterRowDef {}
5655
export class MatRowDef<T> extends CdkRowDef<T> {}
5756

5857
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
59-
_inheritCtorParametersMetadata(MatHeaderRowDef, CdkHeaderRowDef);
60-
_inheritCtorParametersMetadata(MatFooterRowDef, CdkFooterRowDef);
61-
_inheritCtorParametersMetadata(MatRowDef, CdkRowDef);
58+
(MatHeaderRowDef as any)['ctorParameters'] = () => (CdkHeaderRowDef as any)['ctorParameters'];
59+
(MatFooterRowDef as any)['ctorParameters'] = () => (CdkFooterRowDef as any)['ctorParameters'];
60+
(MatRowDef as any)['ctorParameters'] = () => (CdkRowDef as any)['ctorParameters'];
6261

6362
/** Footer template container that contains the cell outlet. Adds the right class and role. */
6463
@Component({

src/lib/table/table.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88

99
import {CDK_TABLE_TEMPLATE, CdkTable} from '@angular/cdk/table';
1010
import {ChangeDetectionStrategy, Component, ViewEncapsulation} from '@angular/core';
11-
import {_inheritCtorParametersMetadata} from '@angular/material/core';
1211

1312
/**
1413
* Wrapper for the CdkTable with Material design styles.
@@ -31,4 +30,4 @@ export class MatTable<T> extends CdkTable<T> {
3130
}
3231

3332
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
34-
_inheritCtorParametersMetadata(MatTable, CdkTable);
33+
(MatTable as any)['ctorParameters'] = () => (CdkTable as any)['ctorParameters'];

src/lib/tabs/tab-label.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88

99
import {Directive} from '@angular/core';
1010
import {CdkPortal} from '@angular/cdk/portal';
11-
import {_inheritCtorParametersMetadata} from '@angular/material/core';
1211

1312
/** Used to flag tab labels for use with the portal directive */
1413
@Directive({
@@ -17,4 +16,4 @@ import {_inheritCtorParametersMetadata} from '@angular/material/core';
1716
export class MatTabLabel extends CdkPortal {}
1817

1918
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
20-
_inheritCtorParametersMetadata(MatTabLabel, CdkPortal);
19+
(MatTabLabel as any)['ctorParameters'] = () => (CdkPortal as any)['ctorParameters'];

src/lib/tree/node.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ import {
1919
QueryList,
2020
} from '@angular/core';
2121
import {
22-
_inheritCtorParametersMetadata,
2322
CanDisable, CanDisableCtor,
2423
HasTabIndex,
2524
HasTabIndexCtor,
@@ -78,7 +77,7 @@ export class MatTreeNodeDef<T> extends CdkTreeNodeDef<T> {
7877
}
7978

8079
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
81-
_inheritCtorParametersMetadata(MatTreeNodeDef, CdkTreeNodeDef);
80+
(MatTreeNodeDef as any)['ctorParameters'] = () => (CdkTreeNodeDef as any)['ctorParameters'];
8281

8382
/**
8483
* Wrapper for the CdkTree nested node with Material design styles.

src/lib/tree/padding.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
*/
88
import {CdkTreeNodePadding} from '@angular/cdk/tree';
99
import {Directive, Input} from '@angular/core';
10-
import {_inheritCtorParametersMetadata} from '@angular/material/core';
1110

1211
/**
1312
* Wrapper for the CdkTree padding with Material design styles.
@@ -26,4 +25,5 @@ export class MatTreeNodePadding<T> extends CdkTreeNodePadding<T> {
2625
}
2726

2827
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
29-
_inheritCtorParametersMetadata(MatTreeNodePadding, CdkTreeNodePadding);
28+
(MatTreeNodePadding as any)['ctorParameters'] = () =>
29+
(CdkTreeNodePadding as any)['ctorParameters'];

src/lib/tree/toggle.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88

99
import {CdkTreeNodeToggle} from '@angular/cdk/tree';
1010
import {Directive, Input} from '@angular/core';
11-
import {_inheritCtorParametersMetadata} from '@angular/material/core';
1211

1312
/**
1413
* Wrapper for the CdkTree's toggle with Material design styles.
@@ -25,4 +24,4 @@ export class MatTreeNodeToggle<T> extends CdkTreeNodeToggle<T> {
2524
}
2625

2726
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
28-
_inheritCtorParametersMetadata(MatTreeNodeToggle, CdkTreeNodeToggle);
27+
(MatTreeNodeToggle as any)['ctorParameters'] = () => (CdkTreeNodeToggle as any)['ctorParameters'];

src/lib/tree/tree.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
import {CdkTree} from '@angular/cdk/tree';
1010
import {ChangeDetectionStrategy, Component, ViewChild, ViewEncapsulation} from '@angular/core';
1111
import {MatTreeNodeOutlet} from './outlet';
12-
import {_inheritCtorParametersMetadata} from '@angular/material/core';
1312

1413
/**
1514
* Wrapper for the CdkTable with Material design styles.
@@ -34,4 +33,4 @@ export class MatTree<T> extends CdkTree<T> {
3433
}
3534

3635
// TODO(devversion): workaround for https://github.com/angular/material2/issues/12760
37-
_inheritCtorParametersMetadata(MatTree, CdkTree);
36+
(MatTree as any)['ctorParameters'] = () => (CdkTree as any)['ctorParameters'];

0 commit comments

Comments
 (0)