NoRedInk/noredink-ui
1
1
Fork 0
mirror of https://github.com/NoRedInk/noredink-ui.git synced 2024-09-17 10:17:09 +03:00
Code Issues Packages Projects Releases Wiki Activity

Use once: true, simpler

Browse Source
This commit is contained in:
Juliano Solanho 2024-08-13 16:19:49 -03:00
parent 855a13bede
commit 720826fc96
1 changed files with 2 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
component-catalog/index.html
View File

@ -21,8 +21,6 @@
// start listening to events when the user starts dragging/clickingdown
app.ports.highlighterListen.subscribe(([id, scrollFriendly]) => {
const touchMoveListener = handleTouchEvents("move", id);
const mouseUpHandler = onDocumentUp(id, "mouse");
const touchEndHandler = onDocumentUp(id, "touch");
window.requestAnimationFrame(() => {
var highlighter = document.getElementById(id);
@ -38,11 +36,11 @@
// Register global listeners for when we release the mouse/touchpad outside a highlighter
highlighter.addEventListener("mousedown", (_) => {
console.log("registering mouseup handler");
document.addEventListener("mouseup", mouseUpHandler);
document.addEventListener("mouseup", onDocumentUp(id, "mouse"), {once: true});
});
highlighter.addEventListener("touchstart", (_) => {
console.log("registering touchend handler");
document.addEventListener("touchend", touchEndHandler);
document.addEventListener("touchend", onDocumentUp(id, "touch"), {once: true});
});
if (!scrollFriendly) {
// scroll-friendly mode only registers a touchmove listener after a longpress
@ -141,14 +139,6 @@
// This helps us not report touchmove when touch starts on one highlighter and moves
// into another
document.getElementById(id)?.removeEventListener("touchmove", touchMoveListener);
// Deregister onDocumentUp listeners, so we avoid sending unnecessary messages to elm
document.removeEventListener("touchend", touchEndHandler);
} else if (device === "mouse") {
if (window.debugHighlighter) {
console.log("deregistering mouse listeners");
}
// Deregister onDocumentUp listeners, so we avoid sending unnecessary messages to elm
document.removeEventListener("mouseup", mouseUpHandler);
}
};
}