Форма логина
Посмотреть и потрогать - в комментариях.
Использован фрагмент картины Васи Ложкина "Родина слышит".
Если интересна реализация, она тут: http://www.panthers.ru/tamer/login/
Писал на скорую руку, поэтому может быть с ошибками и не для смартфонных браузеров.
показать ответы
Я не люблю обсуждать чужую работу, но сомневаюсь, что профессионал бы стал писать вот это
eval('eye' + person + 'Left').style.left = xLeft + 'px';
и городить эти переменные. Легче было создать объект persons, с полями eyeFirstLeft, eyeFirstRight, eyeSecondLeft, eyeSecondRight, и тупо обращаться к ним
var personName = 'eye' + person + 'Left';
if(personName in persons) {
var person = persons[personName];
person.style.left = xLeft + 'px';
}
Можно не делать проверку, и сразу писать короче (если нужно решение в лоб, с самым очевидным способом).
persons['eye' + person + 'Left'].style.left = xLeft + 'px';
А так же, вместо чем для обоих глаз высчитывать координаты, нужно было использовать общие координаты для них, а для второго глаза скорректировать отступ через margin.
Это было намного проще и быстрее. А использовать eval для получения объекта, ну, лучше бы тогда использовал огород из условных конструкций.
Это реально не годится для продакшина, если он новичок и использовал это чисто для демонстрации, то ок. Но оправдывать, что такое мог написать профессионал, потому, что лень - неверно. Поэтому я вынужден согласиться с мнением UndesirableAlien, какое бы оно резкое не было. Сейчас на рынке мы имеем огромное количество кадров, которые не желают учиться, т.к. им платят и за такие решения. Этот код именно демонстрирует полное отсутствие желания учиться, поэтому автор либо просто любитель (или верстальщик), либо программист, который в данный момент не пригоден для профессии.
раскрыть ветку (1)
Абсолютно согласен, вдобавок еще куча "магических чисел". Которые хорошо бы вынести в отдельный модуль с константами, ну или хотя бы в начало модуля.
Куча нарушений DRY в коде, можно сделать гораздо короче. Скрипт не минифицирован, нифига не кроссбраузерный.
Разметка вообще через валидатор не проходит. Для поля 'email' указан тип 'text', не будет встроенной валидации. Свойства в стилях можно сократить в shorthand.
показать ответы
раскрыть ветку (1)
Насчет использования картинки - к Васе Ложкину, сам у него нигде не отыскал "можно/нельзя" :)
Остальное - где угодно и как угодно, но опять же код в скрипте лучше привести к более белому и пушистому виду.
показать ответы
раскрыть ветку (1)
Относительно самой картинки лучше у kudelin-alex (at) mail.ru спросить.
Остальное можно, но лучше переписать на 'высокоэльфийский' язык, в соответствии с современными тенденциями :)