From f3fdd9d9d03ba9918cf429eb0c6d7ab8e35cfe2b Mon Sep 17 00:00:00 2001 From: hbhftgf <59216141+hbhftgf@users.noreply.github.com> Date: Sun, 7 Jan 2024 10:35:39 +0800 Subject: [PATCH 1/5] Update script.js replace while-loop with .contains() --- .../script.js | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js b/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js index 6a3202467e..a3790bd61e 100755 --- a/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js +++ b/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js @@ -8,7 +8,7 @@ table.onmouseover = function(event) { if (currentElem) return; let target = event.target.closest('td'); - + // we moved not into a - ignore if (!target) return; @@ -29,15 +29,9 @@ table.onmouseout = function(event) { if (!currentElem) return; // we're leaving the element – where to? Maybe to a descendant? - let relatedTarget = event.relatedTarget; - - while (relatedTarget) { - // go up the parent chain and check – if we're still inside currentElem - // then that's an internal transition – ignore it - if (relatedTarget == currentElem) return; - - relatedTarget = relatedTarget.parentNode; - } + // if we're still inside currentElem + // then that's an internal transition – ignore it + if (currentElem.contains(event.relatedTarget)) return; // we left the . really. onLeave(currentElem); From 9dfb34a1e9b25e990c1a9ee0d21624d623733724 Mon Sep 17 00:00:00 2001 From: hbhftgf <59216141+hbhftgf@users.noreply.github.com> Date: Sun, 7 Jan 2024 10:51:07 +0800 Subject: [PATCH 2/5] Update script.js From 0dd19221a6c6b02632f0e897461ad917f1dbfeaf Mon Sep 17 00:00:00 2001 From: hbhftgf <59216141+hbhftgf@users.noreply.github.com> Date: Sun, 7 Jan 2024 10:51:44 +0800 Subject: [PATCH 3/5] Update script.js --- .../mouseenter-mouseleave-delegation-2.view/script.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js b/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js index a3790bd61e..6abab0d96f 100755 --- a/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js +++ b/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js @@ -8,7 +8,7 @@ table.onmouseover = function(event) { if (currentElem) return; let target = event.target.closest('td'); - + // we moved not into a - ignore if (!target) return; From c65ccf9716f815889daf535159989c51a9287265 Mon Sep 17 00:00:00 2001 From: hbhftgf <59216141+hbhftgf@users.noreply.github.com> Date: Sun, 7 Jan 2024 10:52:23 +0800 Subject: [PATCH 4/5] Update script.js --- .../mouseenter-mouseleave-delegation-2.view/script.js | 1 - 1 file changed, 1 deletion(-) diff --git a/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js b/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js index 6abab0d96f..2864293213 100755 --- a/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js +++ b/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js @@ -28,7 +28,6 @@ table.onmouseout = function(event) { // e.g. from to another if (!currentElem) return; - // we're leaving the element – where to? Maybe to a descendant? // if we're still inside currentElem // then that's an internal transition – ignore it if (currentElem.contains(event.relatedTarget)) return; From b5313c603a93ff9b496f956c95b5663ecd18390f Mon Sep 17 00:00:00 2001 From: hbhftgf <59216141+hbhftgf@users.noreply.github.com> Date: Sun, 7 Jan 2024 10:53:08 +0800 Subject: [PATCH 5/5] Update script.js --- .../mouseenter-mouseleave-delegation-2.view/script.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js b/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js index 2864293213..aa75ac0261 100755 --- a/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js +++ b/2-ui/3-event-details/3-mousemove-mouseover-mouseout-mouseenter-mouseleave/mouseenter-mouseleave-delegation-2.view/script.js @@ -28,8 +28,7 @@ table.onmouseout = function(event) { // e.g. from to another if (!currentElem) return; - // if we're still inside currentElem - // then that's an internal transition – ignore it + // if we're still inside currentElem, then that's an internal transition – ignore it if (currentElem.contains(event.relatedTarget)) return; // we left the . really.