Skip to content

Commit e9853ae

Browse files
josephperrottmmalerba
authored andcommitted
refactor(material-experimental/mdc-checkbox): remove usage of term blacklist (#20655)
Changes _attrBlacklist symbol to be _medFoundationIgnoredAttrs, which is more descriptive of what the symbol represents. (cherry picked from commit 34485d9)
1 parent 44ef3a0 commit e9853ae

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/material-experimental/mdc-checkbox/checkbox.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ export class MatCheckbox extends _MatCheckboxMixinBase implements AfterViewInit,
217217
* MDC uses animation events to determine when to update `aria-checked` which is unreliable.
218218
* Therefore we disable it and handle it ourselves.
219219
*/
220-
private _attrBlacklist = new Set(['aria-checked']);
220+
private _mdcFoundationIgnoredAttrs = new Set(['aria-checked']);
221221

222222
/** The `MDCCheckboxAdapter` instance for this checkbox. */
223223
private _checkboxAdapter: MDCCheckboxAdapter = {
@@ -230,13 +230,13 @@ export class MatCheckbox extends _MatCheckboxMixinBase implements AfterViewInit,
230230
isIndeterminate: () => this.indeterminate,
231231
removeNativeControlAttr:
232232
(attr) => {
233-
if (!this._attrBlacklist.has(attr)) {
233+
if (!this._mdcFoundationIgnoredAttrs.has(attr)) {
234234
this._nativeCheckbox.nativeElement.removeAttribute(attr);
235235
}
236236
},
237237
setNativeControlAttr:
238238
(attr, value) => {
239-
if (!this._attrBlacklist.has(attr)) {
239+
if (!this._mdcFoundationIgnoredAttrs.has(attr)) {
240240
this._nativeCheckbox.nativeElement.setAttribute(attr, value);
241241
}
242242
},

0 commit comments

Comments
 (0)