RPJosh
/
memories
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

timeline: use map for preloads 

Signed-off-by: Varun Patil <radialapps@gmail.com>
pull/900/head
Varun Patil 2023-10-28 01:38:22 -07:00
parent a151d9cbd4
commit 10ebef22d3
1 changed files with 4 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/components/Timeline.vue
View File

@ -744,12 +744,7 @@ export default defineComponent({
// Store the preloads in a separate map.
// This is required since otherwise the inner detail objects
// do not become reactive (which happens only after assignment).
const preloads: {
[dayId: number]: {
day: IDay;
detail: IPhoto[];
};
} = {};
const preloads = new Map<number, IPhoto[]>();
let prevDay: IDay | null = null;
for (const day of data) {
@ -763,10 +758,7 @@ export default defineComponent({
// Store the preloads
if (day.detail) {
preloads[day.dayid] = {
day: day,
detail: day.detail,
};
preloads.set(day.dayid, day.detail);
delete day.detail;
}
@ -832,8 +824,8 @@ export default defineComponent({
// Iterate the preload map
// Now the inner detail objects are reactive
for (const dayId in preloads) {
this.processDay(Number(dayId), preloads[dayId].detail);
for (const dayId of preloads.keys()) {
this.processDay(dayId, preloads.get(dayId)!);
}
// Notify parent components about stats