Javascript-форум (https://javascript.ru/forum/)
-   Элементы интерфейса (https://javascript.ru/forum/dom-window/)
-   -   Прошу высказать замечания как сделать лучше (https://javascript.ru/forum/dom-window/48975-proshu-vyskazat-zamechaniya-kak-sdelat-luchshe.html)

Vlasenko Fedor 23.07.2014 23:14

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

рони 23.07.2014 23:27

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

Vlasenko Fedor 23.07.2014 23:37

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

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

рони 23.07.2014 23:38

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

Vlasenko Fedor 23.07.2014 23:43

Цитата:

Сообщение от рони (Сообщение 322492)
Poznakomlus,
нажми на >> сколько стало? 2

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

рони 23.07.2014 23:48

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

Vlasenko Fedor 24.07.2014 00:25

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

рони 24.07.2014 00:55

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";

Vlasenko Fedor 24.07.2014 01:16

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

рони 24.07.2014 01:24

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


Часовой пояс GMT +3, время: 02:17.