Skip to content

Commit

Permalink
fix: use private variables for selection info (#5289)
Browse files Browse the repository at this point in the history
  • Loading branch information
dlockhart authored Jan 7, 2025
1 parent 65b5e4a commit 0988f6b
Showing 1 changed file with 10 additions and 6 deletions.
16 changes: 10 additions & 6 deletions components/selection/selection-mixin.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,21 +16,21 @@ export class SelectionInfo {
if (!allEnabledSelected) allEnabledSelected = false;
if (!keys) keys = [];
if (!state) state = SelectionInfo.states.none;
this._allEnabledSelected = allEnabledSelected;
this._keys = keys;
this._state = state;
this.#allEnabledSelected = allEnabledSelected;
this.#keys = keys;
this.#state = state;
}

get allEnabledSelected() {
return this._allEnabledSelected;
return this.#allEnabledSelected;
}

get keys() {
return this._keys;
return this.#keys;
}

get state() {
return this._state;
return this.#state;
}

static get states() {
Expand All @@ -42,6 +42,10 @@ export class SelectionInfo {
};
}

#allEnabledSelected;
#keys;
#state;

}

export const SelectionMixin = superclass => class extends RtlMixin(CollectionMixin(superclass)) {
Expand Down

0 comments on commit 0988f6b

Please sign in to comment.