From 67c6c94a6df62f5795ca38b115b5d3bd8e272e9f Mon Sep 17 00:00:00 2001 From: LieutenantCobretti1998 Date: Thu, 11 Jan 2024 13:29:40 +0400 Subject: [PATCH] Bug with dots and double-clicking arrows. --- 13-Advanced-DOM-Bankist/final/script.js | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/13-Advanced-DOM-Bankist/final/script.js b/13-Advanced-DOM-Bankist/final/script.js index a0015c55ee..60837c51ab 100644 --- a/13-Advanced-DOM-Bankist/final/script.js +++ b/13-Advanced-DOM-Bankist/final/script.js @@ -280,11 +280,13 @@ const slider = function () { e.key === 'ArrowRight' && nextSlide(); }); - dotContainer.addEventListener('click', function (e) { - if (e.target.classList.contains('dots__dot')) { - const { slide } = e.target.dataset; + // * Here is my advice to change the curslide update because it fails to update the curslide which creates a bug of double clicking + dotContainer.addEventListener("click", function (e) { + if (e.target.classList.contains("dots__dot")) { + const {slide} = e.target.dataset; + curSlide = Number(slide); // Update curSlide here goToSlide(slide); - activateDot(slide); + activateDot(curSlide); } }); };