Witam,

Zaznaczę, że uczę się JS tym bardziej obiektowego

przerabiam slider, całość jest zrobiona na klasie z konstruktorem,

tak pokrótce pierwsza metoda generuje slider czyli:

- pobiera element który zamienimy na slider
- tworzy kontener(wrapper) dla slajdow
- pobiera elementy slajdów (dzieci wrappera)
- dodaje klase (slider__item) każdemu dziecku

następna metoda zmienia slidy, pokrótce dodaje klase active do aktualnego slidu:

[XML] pobierz, plaintext 
changeSlide(index) {
 
        // petla po slajdach usuwająca klase active
        this.slides.forEach(slide => {
            slide.classList.remove("active");
            slide.setAttribute("hidden", true);
        });
 
        //dodanie tylko wykrytemu
        this.slides[index].classList.add("active");
        this.slides[index].setAttribute("hidden", false);
 
        //aktualny slider przestawiany na wybrany
        this.currentSlide = index;
 
    }
[XML] pobierz, plaintext 

Chciałbym dodać klase before i after tj. przed i po active, myślę zrobić to dość prosto na ifach (skupiam się tylko na before, after bedzie analogicznie)

[XML] pobierz, plaintext 
this.slides.forEach(slide => {
            slide.classList.remove("active");
            slide.setAttribute("hidden", true);
            slide.classList.remove("before");
        });
 
        //dodanie tylko wykrytemu
        this.slides[index].classList.add("active");
        this.slides[index].setAttribute("hidden", false);
 
        let last = this.slides.length - 1;
 
        // dodanie klas przed i po
        let tmp=this.slides[index];
        if (tmp == 0 ) {
            this.slides[index + last].classList.add("before");
        } else {
            this.slides[index - 1].classList.add("before");
        }
        //aktualny slider przestawiany na wybrany
        this.currentSlide = index;
}
[XML] pobierz, plaintext 

Nie wiem dlaczego if tak działa, że pobiera tylko instrukcje z else? i faktycznie dodaje before ale dla slida 0 już nie.
Oczywiście w slidzie ją jeszcze metody prev, next, pagination.

Z góry dzięki za każdą pomoc.

A sprawdzales w ogole co zawiera TMP?

tak to wygląda w konsoli (dla tmp):

https://ibb.co/pLy9hR3

po przejściu (next) wszystkich slidów

I gdzie tam widzisz tmp rowne 0? Bo ja nigdzie. Czemu wiec sie dziwisz ze
if (tmp == 0)
ci nie dziala?

nom racja, tylko trzymałem się tego że działa instrukcja dla else, hmm ale ok już rozumiem, dzięki.

A jakaś mała podpowiedź jak to ugryźć?

ok działa - zamiast tmp dałem this.currentSlide - takie banały ale przy nauce tak jest... dzięki za pomoc.

Serdecznie pozdrawiam