Apply suggestions from code review

Co-authored-by: NightKev <34855794+DayKev@users.noreply.github.com>
This commit is contained in:
Wlowscha 2025-02-24 22:20:04 +01:00 committed by GitHub
parent 41c75bf4a5
commit 2b0f9357a4
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 5 additions and 5 deletions

View File

@ -91,7 +91,7 @@ export class FilterBar extends Phaser.GameObjects.Container {
* @param index the index of the column to retrieve
* @returns the associated DropDownColumn if it exists, undefined otherwise
*/
getColumn(index: number) : DropDownColumn {
public getColumn(index: number): DropDownColumn {
return this.columns[index];
}
@ -194,7 +194,7 @@ export class FilterBar extends Phaser.GameObjects.Container {
return this.getFilter(col).getVals();
}
resetSelection(col: DropDownColumn): void {
public resetSelection(col: DropDownColumn): void {
this.dropDowns[col].resetToDefault();
this.updateFilterLabels();
}

View File

@ -892,7 +892,7 @@ export default class PokedexUiHandler extends MessageUiHandler {
success = true;
} else if (this.filterMode && !this.filterBar.getFilter(this.filterBarCursor).hasDefaultValues()) {
this.filterBar.resetSelection(this.filterBarCursor);
this.updateStarters;
this.updateStarters();
success = true;
} else if (this.filterTextMode && !(this.filterText.getValue(this.filterTextCursor) === this.filterText.defaultText)) {
this.filterText.resetSelection(this.filterTextCursor);

View File

@ -1088,7 +1088,7 @@ export default class StarterSelectUiHandler extends MessageUiHandler {
/**
* Set default value for the caught dropdown, which only shows caught mons
*/
resetCaughtDropdown(): void {
public resetCaughtDropdown(): void {
const caughtDropDown: DropDown = this.filterBar.getFilter(DropDownColumn.CAUGHT);
caughtDropDown.resetToDefault();
@ -1338,7 +1338,7 @@ export default class StarterSelectUiHandler extends MessageUiHandler {
} else {
this.filterBar.resetSelection(this.filterBarCursor);
}
this.updateStarters;
this.updateStarters();
success = true;
} else if (this.statsMode) {
this.toggleStatsMode(false);