Skip to content

Commit c523bfc

Browse files
Fixed issue where requestStateUpdate was not getting called on Edge (#482)
Co-authored-by: Shane Weaver <[email protected]>
1 parent 48c6f4c commit c523bfc

File tree

1 file changed

+9
-18
lines changed

1 file changed

+9
-18
lines changed

src/components/mgt-agenda/mgt-agenda.ts

Lines changed: 9 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ export class MgtAgenda extends MgtTemplatedComponent {
6868
}
6969

7070
this._date = value;
71-
this.requestStateUpdate(true);
71+
this.reloadState();
7272
}
7373

7474
/**
@@ -88,7 +88,7 @@ export class MgtAgenda extends MgtTemplatedComponent {
8888
}
8989

9090
this._groupId = value;
91-
this.requestStateUpdate(true);
91+
this.reloadState();
9292
}
9393

9494
/**
@@ -108,7 +108,7 @@ export class MgtAgenda extends MgtTemplatedComponent {
108108
}
109109

110110
this._days = value;
111-
this.requestStateUpdate(true);
111+
this.reloadState();
112112
}
113113

114114
/**
@@ -128,7 +128,7 @@ export class MgtAgenda extends MgtTemplatedComponent {
128128
}
129129

130130
this._eventQuery = value;
131-
this.requestStateUpdate(true);
131+
this.reloadState();
132132
}
133133

134134
/**
@@ -236,20 +236,6 @@ export class MgtAgenda extends MgtTemplatedComponent {
236236
`;
237237
}
238238

239-
/**
240-
* Request to reload the state.
241-
* Use reload instead of load to ensure loading events are fired.
242-
*
243-
* @protected
244-
* @memberof MgtBaseComponent
245-
*/
246-
protected async requestStateUpdate(force?: boolean) {
247-
if (force) {
248-
this.events = null;
249-
}
250-
super.requestStateUpdate(force);
251-
}
252-
253239
/**
254240
* Render the loading state
255241
*
@@ -539,6 +525,11 @@ export class MgtAgenda extends MgtTemplatedComponent {
539525
}
540526
}
541527

528+
private async reloadState() {
529+
this.events = null;
530+
this.requestStateUpdate(true);
531+
}
532+
542533
private onResize() {
543534
this._isNarrow = this.offsetWidth < 600;
544535
}

0 commit comments

Comments
 (0)