Показать сообщение отдельно
  #6 (permalink)  
Старый 17.06.2008, 01:20
Аватар для Gvozd
Матрос
Отправить личное сообщение для Gvozd Посмотреть профиль Найти все сообщения от Gvozd
 
Регистрация: 04.04.2008
Сообщений: 6,246

типа "сам не знаю что пишу"))))
+ множественное повторение схожих функций для разных объектов, etc...
по поводу вашего скрипта:
во первых дивы butLeft и butRight вы бы хоть раскрасили. я сперва завытыкал в пустую страницу и не мог понять что от меня хотят. не шутите так в общем)
с точки зрения эстетики, формочка мне понравилась. хорошо смотрится, и не броско.
по поводу кода:
timerID = setTimeout("moveIn()", 1);
				if(start >= 0) {
					clearTimeout(timerID);
				}

к моменту перехода к внутренему блока условия, таймаут уже сработает, и не нуждается в очистке. но это мелкая придирка.
if(start <= 2 && start >= -365) {

заместо - 365 стоит задать реальный размер двигаемого блока, т.к. в разных браузерах, при разных настройках, цифра может и отличатся. но это отже не критично. в принципе больше ничего такого не вижу.
Кстати, а в каком месте тут моя помощ? не могу вспомнить.
Ответить с цитированием