Vertical scroll при помощи Javascript

[ish]

И он с таким же успехом станет задыхаться от наговнокоденого яваскрипта

[atd]

от вот такой штуки:

nextval = parseInt($('#'+scroll_text_id).css('top')) — scrollerstep + 'px';
step_l = step_l — scrollerstep;
$('#'+scroll_text_id).css('top', nextval);

на крупной странице и с несколькими рекламными блоками задыхаться они будут не меньше.

Говнокод он и в африке говнокод, хоть флэш, хоть js

[le0pard]

>>после пятой итерации блок начал «плыть»
Что это значит? Куда «плыть»?

[atlete]

если прокручиваемому тексту поставить рамку, то видно что граница прокручиваемого текста не совпадает с границей блока. Получается идет сдвиг по оси Y, если не ошибаюсь.

[yroman]

А чего у вас все кишки наружу торчат? Оформили бы как jquery plugin тогда уж, раз jquery используете.

[Stark]

А что если у меня на странице уже есть ф-ция set_Interval? Или scrollText?
Читайте комментарий выше хабраюзера yroman — такие вещи, нужно оформлять как плагин, если юзаете jQuery.

[piumosso]

Почему бы вам не инкапсулировать всё это в jQuery плагин?
К тому же, наверняка, такие имеются)

[atlete]

к сожалению такого плагина не нашел. Возможно плохо искал.
Найду время — сделаю плагин.

[Anonym]

Это ппц.

[koteus]

Возможно автор тонкий тролль. Все же выскажу свое мнение:

Несмотря на var-ы, эта куча переменных в начале создается в глобальной области видимости. Таким образом все переменные и функции являются глобальными. Допустим автор написал другой подобный скрипт и там тоже есть переменная step. Вот тут то и собака зарыта.

Можно использовать структуру (Object Literal):

var myScroller = {
options: {
scrollerstep: 1,
...
lefttime: 0
},
runScroll: function() {
...
},
scroll_run: function() {
...
}
};
// Вызов
$(function(){
...
myScroller .scroll_run();
});

Или использовать Self Invoking Anonymouse Function:

var myScroller = (function(){
this.options = {
scrollerstep: 1,
...
lefttime: 0
};
this.runScroll = function() {
...
};
this.scroll_run = function() {
...
};
return this;
})();

// Вызов
$(function(){
...
myScroller .scroll_run();
});