Fix duplicate display events

This commit is contained in:
MDMCK10 2023-05-12 20:38:13 +02:00
parent af48aa7e19
commit a694fb38bc

View File

@ -63,6 +63,18 @@ const forceVotePanel = document.getElementById("forceVotePanel");
// needed to scroll to bottom // needed to scroll to bottom
const chatListDiv = document.querySelector(".chat-table"); const chatListDiv = document.querySelector(".chat-table");
let events = new Map();
function addListener(element, event, id, callback) {
events.set(id, callback);
element.addEventListener(event, callback);
}
function removeListener(element, event, id) {
element.removeEventListener(event, events.get(id));
events.delete(id);
}
class CollabVMClient { class CollabVMClient {
eventemitter = createNanoEvents(); eventemitter = createNanoEvents();
socket; socket;
@ -740,16 +752,14 @@ async function openVM(url, node) {
await vm.connectToVM(node); await vm.connectToVM(node);
vmlist.style.display = "none"; vmlist.style.display = "none";
vmview.style.display = "block"; vmview.style.display = "block";
display.addEventListener('mousemove', (e) => vm.mouseevent(e, undefined), {capture: true}) addListener(display, 'mousemove', 'displayMove', (e) => vm.mouseevent(e, undefined));
display.addEventListener('mousedown', (e) => vm.mouseevent(e, true), {capture: true}); addListener(display, 'mousedown', 'displayDown', (e) => vm.mouseevent(e, true));
display.addEventListener('mouseup', (e) => vm.mouseevent(e, false), {capture: true}); addListener(display, 'mouseup', 'displayUp', (e) => vm.mouseevent(e, false));
display.addEventListener('wheel', (e) => {vm.mousewheelhandler(e);e.preventDefault();return false;}, {capture: true}); addListener(display, 'wheel', 'displayWheel', (e) => {vm.mousewheelhandler(e);e.preventDefault();return false;}); // BUG: mousewheelhandler seems to be broken!
display.addEventListener('contextmenu', (e) => e.preventDefault()); addListener(display, 'contextmenu', 'displayContextMenu', (e) => e.preventDefault());
display.addEventListener('click', () => { addListener(display, 'click', 'displayClick', () => { if (turn === -1) vm.turn(); });
if (turn === -1) vm.turn(); addListener(display, 'keydown', 'displayKeyDown', (e) => vm.keyevent(e, true));
}, {capture: true}); addListener(display, 'keyup', 'displayKeyUp', (e) => vm.keyevent(e, false));
display.addEventListener('keydown', (e) => vm.keyevent(e, true), {capture: true});
display.addEventListener('keyup', (e) => vm.keyevent(e, false), {capture: true});
} }
function screenshotVM() { function screenshotVM() {
return new Promise((res, rej) => { return new Promise((res, rej) => {
@ -778,6 +788,14 @@ function cleanup() {
usernameSpan.classList = "input-group-text bg-dark text-light"; usernameSpan.classList = "input-group-text bg-dark text-light";
display.height = 0; display.height = 0;
display.width = 0; display.width = 0;
removeListener(display, 'mousemove', 'displayMove');
removeListener(display, 'mousedown', 'displayDown');
removeListener(display, 'mouseup', 'displayUp');
removeListener(display, 'wheel', 'displayWheel');
removeListener(display, 'contextmenu', 'displayContextMenu');
removeListener(display, 'click', 'displayClick');
removeListener(display, 'keydown', 'displayKeyDown');
removeListener(display, 'keyup', 'displayKeyUp');
} }
buttons.home.addEventListener('click', async () => returnToVMList()); buttons.home.addEventListener('click', async () => returnToVMList());
buttons.screenshot.addEventListener('click', async () => { buttons.screenshot.addEventListener('click', async () => {