Алекпер А.
2 повідомлення
#11 років тому
Оцените вёрстку и укажите на ошибки в коде. Хоть она и проста по виду, но хочется знать где имеются ошибки.
Вот .
Спасибо! 
Дмитрий И.
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 протоколу рекомендуется писать не так:
...
А так:
//fonts.googleapis.com/css?family=PT+Sans:400,700,400italic

Это рекомендации Google.
При том, что сам гугл выдает верхнюю ссылку на google fonts 
Владимир Р.
3315 повідомлень
#11 років тому
Цитата (parkgraphic):
Одиночные теги нужно закрывать
В  HTML5 использование закрывающего слеша в одиночных тегах необязательно.
Антон Б.
1132 повідомлення
#11 років тому
vovka-morkovka, это лишь допущение, а хороший тон и подсветку синтаксиса никто не отменял.
Если на то пошло, то и браузеры могут некоторые ошибки исправлять.
А еще я бы посоветовал сделать в <nav> паддинги не у <li>, а у <a>, дабы юзеру меньше приходилось целиться в пункты меню.
И обернуть <header>, <footer> и <content>  в один блок с padding по бокам, дабы на планшетах были отступы от рамок дисплея.
Владимир Р.
3315 повідомлень
#11 років тому
Цитата (parkgraphic):
Если на то пошло, то и браузеры могут некоторые ошибки исправлять.
Ну это вы передергиваете))
Но за хороший тон и подсветку согласен.
Артём Романченко
438 повідомлень
#11 років тому
Вместо Lorem ipsum dolor sit введите текста побольше и блок победит, а хотелось бы чтоб перешло на другую строку 
тоже самое и остально 
Олег Г.
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 фанатов.