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

sel: perf optimizations 

Signed-off-by: Varun Patil <radialapps@gmail.com>
monorepo
Varun Patil 2023-10-31 19:10:28 -07:00
parent 5a4fc07eca
commit c5c72dabd8
1 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/components/SelectionManager.vue
View File

@ -184,6 +184,7 @@ export default defineComponent({
prevOver: null as IPhoto | null, prevOver: null as IPhoto | null,
touchScrollInterval: 0, touchScrollInterval: 0,
touchScrollDelta: 0, touchScrollDelta: 0,
touchMoveSelFrame: 0,
}), }),
mounted() { mounted() {
@ -455,6 +456,7 @@ export default defineComponent({
// Prevent scrolling // Prevent scrolling
event.preventDefault(); event.preventDefault();
event.stopPropagation();
// This should never happen // This should never happen
if (!touch) return; if (!touch) return;
@ -491,12 +493,20 @@ export default defineComponent({
}; };
this.touchScrollInterval = window.requestAnimationFrame(fun); this.touchScrollInterval = window.requestAnimationFrame(fun);
} }
} else { } else if (this.touchScrollInterval) {
window.cancelAnimationFrame(this.touchScrollInterval); window.cancelAnimationFrame(this.touchScrollInterval);
this.touchScrollInterval = 0; this.touchScrollInterval = 0;
} }
// Trigger multiselection update in next frame
// If there's no anchor then this is pointless cause it gets cancelled
// If already scrolling up/down then the effect will be called anyway
if (this.touchAnchor && !this.touchScrollInterval && !this.touchMoveSelFrame) {
this.touchMoveSelFrame = window.requestAnimationFrame(() => {
this.touchMoveSelFrame = 0;
this.touchMoveSelect(touch, rowIdx); this.touchMoveSelect(touch, rowIdx);
});
}
}, },
/** Multi-select triggered by touchmove */ /** Multi-select triggered by touchmove */