Прошу высказать замечания как сделать лучше

[Vlasenko Fedor]

В двух словах. Написал панель навигации. И так как я совсем недавно начал изучать js. Прошу высказать более опытных разработчиков как писать лучше.
За мысли и пожелания всех благодарю заранее
Сам пример. http://jsfiddle.net/vlasenkofedor/hyQnb/

[рони]

Poznakomlus,
хорошо бы если указал 5 страниц то 5 страниц всегда и было.

[Vlasenko Fedor]

рони,
не совсем понял, поясни.
Сейчас так

inrow: 5, //количество в ряду
items: 5, // всего позиций
current: 2, //текущая

что имел ввиду?

[рони]

Poznakomlus,
нажми на >> сколько стало? 2

[Vlasenko Fedor]

Poznakomlus,
нажми на >> сколько стало? 2

то есть при нажатии на >>автоматом current последняя должна быть ?

[рони]

Poznakomlus,
при нажатии на последнюю должно быть 8,9,10,11,12 а не 11,12 -- чтоб нескакал размер и под курсором невозникал провал

[Vlasenko Fedor]

Спасибо, дописал и обновил( В карму не могу +.). Ждем замечаний и предложений дальше.

[рони]

Poznakomlus,
вариант ...

var Lis = el.parentNode.children;
                    for (var i=0; i<Lis.length; i++)  {
                      Lis[i].className = Lis[i] == el ? "active": "";
                    }

вместо

var ul = el.parentNode;
                    
                    for (var childItem in ul.childNodes) {
                        var child_el = ul.childNodes[childItem];
                        if (child_el.nodeType == 1) {
                            child_el.removeAttribute("class");
                        }
                    }
                    el.className = "active";

[Vlasenko Fedor]

рони,
Принято и обновлено. Еще раз Спасибо за оптимизацию! Код обновил
http://jsfiddle.net/vlasenkofedor/hyQnb/

[рони]

Poznakomlus,
пора шаг вводить для > и < от 1 до inrow ))), по умолчанию для таких кнопок обычно 1