Почему не работает код при прокрутке страницы?

Question

[sunnyrio]

Почему не работает код при прокрутке страницы?
Если делаю на щелчек кнопки то все работает.

window.addEventListener("scroll", common());

function common() 
{
var task_win = document.getElementById("el").getBoundingClientRect(); 
var task_win_height = document.getElementById("el").offsetHeight;
var browser_height = document.body.clientHeight;
var save_scroll = 0;
var scroll_top = $(window).scrollTop();
var task_win_top = 0;


if (task_win.top < 1 && task_win.bottom > browser_height)
{
	if (scroll_top > save_scroll) { task_win_top = task_win_top + scroll_top - save_scroll }
	if (scroll_top < save_scroll) { task_win_top = task_win_top + save_scroll - scroll_top }

	
$('#el').css({"top": task_win_top});
}


};

Answer 1

[abberati]

потому что он срабатывает в момент добавления обработчика.
вот так правильно
window.addEventListener("scroll", common);

"почему?" спросите вы. а я вам отвечу

Comments

[sunnyrio]

ясно а как сохранить в переменную save_scroll текущую позицию скролла чтобы потом ее использовать для сравнения?

[teotlu]

sunnyrio: нужно вынести эту переменную из функции-обработчика скролла наружу, a изменять её внутри обработчика в конце. Дабы она не была глобальной, можно всё обернуть в самовызывающуюся анонимную функцию (function() { ... })();

Типа такого: https://jsfiddle.net/mtdaw7s2/
Правда, всё равно ваш if никогда не выполняется.

[sunnyrio]

teotlu: почему не выполниться?

[teotlu]

sunnyrio: ну, если вы посмотрите в консоль (я ваши вывел параметры туда), то увидите, что оба указанных условия не выполняются никогда. Наверное, условия заданы неправильно, знак где-то не тот, или ещё что-то в этом роде. Там почти всё время отрицательные значения в task_win.top и task_win.bottom.