Skip to content

Commit 76a09d9

Browse files
authored
fix(cdk/accordion): enforce stricter types for inputs (#20287)
BREAKING CHANGE: The `disabled` and `expanded` properties in the `AccordionItem` are now strict to `boolean`.
1 parent 5519131 commit 76a09d9

File tree

5 files changed

+11
-15
lines changed

5 files changed

+11
-15
lines changed

src/cdk/accordion/accordion-item.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,8 @@ export class CdkAccordionItem implements OnDestroy {
6060

6161
/** Whether the AccordionItem is expanded. */
6262
@Input()
63-
get expanded(): any { return this._expanded; }
64-
set expanded(expanded: any) {
63+
get expanded(): boolean { return this._expanded; }
64+
set expanded(expanded: boolean) {
6565
expanded = coerceBooleanProperty(expanded);
6666

6767
// Only emit events and update the internal value if the value changes.
@@ -90,9 +90,9 @@ export class CdkAccordionItem implements OnDestroy {
9090

9191
/** Whether the AccordionItem is disabled. */
9292
@Input()
93-
get disabled() { return this._disabled; }
94-
set disabled(disabled: any) { this._disabled = coerceBooleanProperty(disabled); }
95-
private _disabled: boolean = false;
93+
get disabled(): boolean { return this._disabled; }
94+
set disabled(disabled: boolean) { this._disabled = coerceBooleanProperty(disabled); }
95+
private _disabled = false;
9696

9797
/** Unregister function for _expansionDispatcher. */
9898
private _removeUniqueSelectionListener: () => void = () => {};

src/material/expansion/expansion-panel-header.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ export class MatExpansionPanelHeader implements AfterViewInit, OnDestroy, Focusa
114114
* Whether the associated panel is disabled. Implemented as a part of `FocusableOption`.
115115
* @docs-private
116116
*/
117-
get disabled() {
117+
get disabled(): boolean {
118118
return this.panel.disabled;
119119
}
120120

src/material/expansion/expansion-panel.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -240,8 +240,6 @@ export class MatExpansionPanel extends CdkAccordionItem implements AfterContentI
240240
}
241241

242242
static ngAcceptInputType_hideToggle: BooleanInput;
243-
static ngAcceptInputType_expanded: BooleanInput;
244-
static ngAcceptInputType_disabled: BooleanInput;
245243
}
246244

247245
/**

tools/public_api_guard/cdk/accordion.d.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,10 @@ export declare class CdkAccordionItem implements OnDestroy {
1818
accordion: CdkAccordion;
1919
closed: EventEmitter<void>;
2020
destroyed: EventEmitter<void>;
21-
get disabled(): any;
22-
set disabled(disabled: any);
23-
get expanded(): any;
24-
set expanded(expanded: any);
21+
get disabled(): boolean;
22+
set disabled(disabled: boolean);
23+
get expanded(): boolean;
24+
set expanded(expanded: boolean);
2525
expandedChange: EventEmitter<boolean>;
2626
readonly id: string;
2727
opened: EventEmitter<void>;

tools/public_api_guard/material/expansion.d.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,8 +67,6 @@ export declare class MatExpansionPanel extends CdkAccordionItem implements After
6767
ngOnDestroy(): void;
6868
open(): void;
6969
toggle(): void;
70-
static ngAcceptInputType_disabled: BooleanInput;
71-
static ngAcceptInputType_expanded: BooleanInput;
7270
static ngAcceptInputType_hideToggle: BooleanInput;
7371
static ɵcmp: i0.ɵɵComponentDefWithMeta<MatExpansionPanel, "mat-expansion-panel", ["matExpansionPanel"], { "disabled": "disabled"; "expanded": "expanded"; "hideToggle": "hideToggle"; "togglePosition": "togglePosition"; }, { "opened": "opened"; "closed": "closed"; "expandedChange": "expandedChange"; "afterExpand": "afterExpand"; "afterCollapse": "afterCollapse"; }, ["_lazyContent"], ["mat-expansion-panel-header", "*", "mat-action-row"]>;
7472
static ɵfac: i0.ɵɵFactoryDef<MatExpansionPanel, [{ optional: true; skipSelf: true; }, null, null, null, null, { optional: true; }, { optional: true; }]>;
@@ -100,7 +98,7 @@ export declare class MatExpansionPanelDescription {
10098
export declare class MatExpansionPanelHeader implements AfterViewInit, OnDestroy, FocusableOption {
10199
_animationMode?: string | undefined;
102100
collapsedHeight: string;
103-
get disabled(): any;
101+
get disabled(): boolean;
104102
expandedHeight: string;
105103
panel: MatExpansionPanel;
106104
constructor(panel: MatExpansionPanel, _element: ElementRef, _focusMonitor: FocusMonitor, _changeDetectorRef: ChangeDetectorRef, defaultOptions?: MatExpansionPanelDefaultOptions, _animationMode?: string | undefined);

0 commit comments

Comments
 (0)