Прошу высказать замечания как сделать лучше
В двух словах. Написал панель навигации. И так как я совсем недавно начал изучать js. Прошу высказать более опытных разработчиков как писать лучше.
За мысли и пожелания всех благодарю заранее :) Сам пример. http://jsfiddle.net/vlasenkofedor/hyQnb/ |
Poznakomlus,
хорошо бы если указал 5 страниц то 5 страниц всегда и было. |
рони,
не совсем понял, поясни. Сейчас так inrow: 5, //количество в ряду items: 5, // всего позиций current: 2, //текущая что имел ввиду? |
Poznakomlus,
нажми на >> сколько стало? 2 |
Цитата:
|
Poznakomlus,
при нажатии на последнюю должно быть 8,9,10,11,12 а не 11,12 -- чтоб нескакал размер и под курсором невозникал провал |
Спасибо, дописал и обновил( В карму не могу +.). Ждем замечаний и предложений дальше.
|
Poznakomlus,
:write: вариант ...
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";
|
рони,
Принято и обновлено. Еще раз Спасибо за оптимизацию! Код обновил http://jsfiddle.net/vlasenkofedor/hyQnb/ |
Poznakomlus,
пора шаг вводить для > и < от 1 до inrow ))), по умолчанию для таких кнопок обычно 1 |
Пример поправил и обновил. Добавил немного комментариев.
Жду ваших комментариев по улучшению и оптимизации :) http://jsfiddle.net/vlasenkofedor/hyQnb/ |
| Часовой пояс GMT +3, время: 00:32. |