Оцените верcтку
2 сообщения
26 сообщений
#11 лет назад
При подключении файлов по HTTP протоколу рекомендуется писать не так:ссылка
А так:
ссылка
Это рекомендации Google.
Классы и идентификаторы не совсем понятны при мимоходном просмотре: "header_w", "up_div", "f_up" и т.п. Рекомендуется давать более говорящие названия.
В одном месте Вы пишете двойные кавычки, в другом - одинарные. Лучше придерживаться одного стиля. Лучше использовать двойные.
Ну про CSS я ничего не говорю, не смотрел nofntkmyj. Я к тому, что бывает кучу лишнего CSS кода пишут, не подозревая, что стиль можно было задать одной строкой. Но повторюсь - я не утверждаю, что у Вас в CSS может быть что-то не лучшим способом реализовано
.Ну, а валидатор выдал, что ошибок грубых нет. Хотя, наверное Вы и без меня валидатором пользовались
. Успехов.
2 сообщения
#11 лет назад
Lyodik, спасибо, что ответили 
Про имена классов согласен, поправлю! Кавычки и шрифты тоже.
Валидатором проверял.
3315 сообщений
#11 лет назад
alikitto, Советую использовать box-sizing: border-box;
Логотип всегда должен быть ссылкой на стартовую(шапка и футер).
У вас несколько основных блоков сайта width: 1000px; margin: 0 auto. Целесообразнее использовать один класс с заданными правилами и просто цеплять его к нужным блокам.
Блок .work у вас по идее должен выступать контейнером для элементов .workcont, а на деле он содержит только заголовок, что странно.
Почитайте про семантические блоки HTML5 и научитесь их использовать в верстке.
Не нашел на странице H1-H2, но есть H3 и H4.
А что, если клиент захочет заменить/добавить логотипы? Верстка - это не тупое позиционирование блоков из дизайна, а реализация представления данных из БД.
Давайте осмысленные имена для классов, отражающие суть контента.
1132 сообщения
#11 лет назад
Одиночные теги нужно закрыватьТабуляцию в разметке делайте
Пути пишите относительные
Спрайты используйте по-назначению, а не для вывода информации
Остальное вроде выше написано.
Цитата (Lyodik):
При подключении файлов по HTTP протоколу рекомендуется писать не так:При том, что сам гугл выдает верхнюю ссылку на google fonts
ссылка...
А так:
//fonts.googleapis.com/css?family=PT+Sans:400,700,400italic
Это рекомендации Google.

3315 сообщений
#11 лет назад
Цитата (parkgraphic):Одиночные теги нужно закрыватьВ HTML5 использование закрывающего слеша в одиночных тегах необязательно.
1132 сообщения
#11 лет назад
vovka-morkovka, это лишь допущение, а хороший тон и подсветку синтаксиса никто не отменял.Если на то пошло, то и браузеры могут некоторые ошибки исправлять.
А еще я бы посоветовал сделать в <nav> паддинги не у <li>, а у <a>, дабы юзеру меньше приходилось целиться в пункты меню.
И обернуть <header>, <footer> и <content> в один блок с padding по бокам, дабы на планшетах были отступы от рамок дисплея.
3315 сообщений
#11 лет назад
Цитата (parkgraphic):Если на то пошло, то и браузеры могут некоторые ошибки исправлять.Ну это вы передергиваете))
Но за хороший тон и подсветку согласен.
438 сообщений
2 сообщения
#11 лет назад
Указывайте относительные пути.background: url("../images/sprite.png");или
background: url("/images/sprite.png");Еще не понятно почему у вас форматирование разное.
Цитата (Lyodik):
При подключении файлов по HTTP протоколу рекомендуется писать не так:
ссылка...
А так:
//fonts.googleapis.com/css?family=PT+Sans:400,700,400italic
Это только для того, что бы в случае если сайт грузится по https то и ресурсы по нему грузились, и не было руганины о "потенциально опысных ресурсах".
Цитата (parkgraphic):
vovka-morkovka, это лишь допущение, а хороший тон и подсветку синтаксиса никто не отменял.
Хороший тон - готовить семантичную и валидную верстку, а не поддерживать призоти XHTML фанатов.




