JS почему не работает анимация при прокрутке?

Question

[Cheizer]

Друзья, был человеческий код, а после компрессии перенес на другой сайт и он не работает, если в несжатом я еще более менее понимаю, то тут не могу разобраться что не так :(
Был код, делал он следующее, у рисунков которые попадали в область прокрутки браузера, и менял класс fadeInSoon заменялся классом FadeIn. По умолчанию рисунки не видны, и при прокрутке просто плавно появлялись, но там уже css я понимаю все. Но теперь JS этоn не работает :( что не так?

function s() {
    var e = function () {
        u = !0
    };
    document.addEventListener("scroll", e, !1), document.addEventListener("touchmove", e, !1), setInterval(function () {
        u && !p ? (y = $(document).scrollTop(), d(y), u = !1) : p && $(".fadeInSoon").css("opacity", 1)
    }, 250)
}

$(".fadeInSoon").each(function () {
        e >= $(this).offset().top - $(window).height() && $(this).removeClass("fadeInSoon").addClass("fadeIn")
})

Comments

[Никита Ермиличев]

в консоле что?

[Cheizer]

Никита Ермиличев, порядок, ошибок нет

[Николай Чуприк]

Вы сначала сами бы попробовали бы разобраться в этой порнографии. Рекомендую вам ответить сначала на вопросы:

1. Откуда взялась и чему равна переменная p?
2. Какое поведение вы ожидаете от функции e()?
3. Какова функция d()?

... и многие другие.

[Cheizer]

Никита Ермиличев, fadeInSoon не подменяется fadeIn, почему то не работает момент "прокрутки"

[Cheizer]

Николай Чуприк, да вот в том то и дело, после компрессии кода так стало :)) а исходного уже нет.

[Никита Ермиличев]

Роман Кузнецов, можно ссылочку на ресурс? или на ручку пример выложите

[Николай Чуприк]

Роман Кузнецов, тут чинить нечего, надо заново написать, и всё.

[Cheizer]

Николай Чуприк, да, тоже к этому склоняюсь

Answer 1

[Николай Чуприк]

$( document ).scroll( function() {
    var y = $( this ).scrollTop();
    $( 'fadeInSoon' ).each( function()  {
        if ( $( this ).offset().top < y )
            $( this ).removeClass( 'fadeInSoon' ).addClass( 'FadeIn' );
    } )
} )

Не проверял.

Comments

[Cheizer]

Спасибо, сдобавил класс точку $( '.fadeInSoon' ).each( function() { и заработало :)

[Николай Чуприк]

Ставьте брейкпоинт, трассируйте в отладчике.

[Cheizer]

Николай Чуприк, я даже не понимаю что значат эти слова :))

[Cheizer]

А как бы заставить пораньше срабатывать этот scrollTop?

[Николай Чуприк]

Роман Кузнецов, что значит "пораньше"? Можно в if заменить y на y + 100, например. Тогда присвоение класса произойдёт когда картинка на 100 пикселей ниже экрана.

[Cheizer]

А как сделать что бы как только картинка появится в окне сразу происходила подмена класса? Сейчас получается только когда она посередине экрана, тогда и появляется.