Прокомментируйте верстку

[fattest]

Оцените и прокомментируйте небольшую работу.

Просьба в пустую не ругать, а ткнуть носом в недочеты.

http://jsfiddle.net/cenSY/5/

[wwt]

1. Текст Login Page Template я бы сделал более читаемым шрифтом.

2. не стоит вот так:

*{
margin:0;
padding:0;
}

сбрасывать отступы и поля. Не все элементы их имеют по умолчанию, а такая запись будет перебирать все.

3. Не надо для полей логина и пароля ставить по умолчанию значение, для ввода своего логина и пароля придется стереть сначала написанное, лучше используйте плейсхолдеры.

4. Не очень понравилось абсолютное позиционирование иконок над полями.Вы уверены что во всех браузерах и при разных разрешениях они они останутся там где должны?Я бы наверное обернул бы прозрачное поле в обертку и стилизовал уже её, положив иконку на фон.

5. Ну и не стоит использовать id для стилизации для этого есть классы. А id только если это нужно еще для чего-нибудь.

[fattest]

1. Текст Login Page Template я бы сделал более читаемым шрифтом.

2. не стоит вот так:

*{
margin:0;
padding:0;
}

сбрасывать отступы и поля. Не все элементы их имеют по умолчанию, а такая запись будет перебирать все.

3. Не надо для полей логина и пароля ставить по умолчанию значение, для ввода своего логина и пароля придется стереть сначала написанное, лучше используйте плейсхолдеры.

4. Не очень понравилось абсолютное позиционирование иконок над полями.Вы уверены что во всех браузерах и при разных разрешениях они они останутся там где должны?Я бы наверное обернул бы прозрачное поле в обертку и стилизовал уже её, положив иконку на фон.

5. Ну и не стоит использовать id для стилизации для этого есть классы. А id только если это нужно еще для чего-нибудь.

1. Не спорю, читабельность плохая. Это проба использования нестандартных шрифтов.

2. Так как я начинающий и по советам специалистов читал чужие шаблоны, мне показалось что данная инструкция является чуть ли не стандартом. Практически на каждом втором сайте встречается такая запись. Как так? Спасибо за совет по данному пункту, только сейчас понял, что это дополнительная и не "маленькая" нагрузка на браузер.

3. Понял, тормознул немного.

4. Так как блок не резиновый, иконки не убегают и отображаются корректно во всех браузерах (проверенно). Тоже думал, о таком решении, но решил, что из-за дополнительных элементов и эффектов будет доп. нагрузка.

5. Мое мнение. Использование классов нужно в том месте где элементы должны иметь одинаковые свойства, а если элемент присутствует на странице в одном экземпляре, то id. (Если не трудно, по-подробней объясните если я не прав).

[wwt]

5. Мое мнение. Использование классов нужно в том месте где элементы должны иметь одинаковые свойства, а если элемент присутствует на странице в одном экземпляре, то id. (Если не трудно, по-подробней объясните если я не прав).

Ну для стилей и создан такой атрибут как class который может содержать сколько угодно классов перечисленных через пробел. А id служит для идентификации элемента в скриптовых языках и он может быть только 1 на весь документ. Это конечно не мешает использовать в CSS и его, в CSS существует большое количество селекторов. Можно и по другим атрибутам идентифицировать элементы, но зачем если в этом нет необходимости и можно использовать классы?

Есть такая вещь как приоритет селекторов про них тоже можете почитать. У id и class разные приоритеты.

ЗЫ Вообще в этом нет ничего плохого, вы можете использовать любые селекторы.