Оцените верстку

Учусь верстать, сделала верстку макета, если кому-нибудь не жалко потратить немного времени, посмотрите, пожалуйста, что получилось, выскажите замечания.

Единственное, что у меня не получилось - настроить слайдер carousel в IE7, не могу разобраться, в чем там дело, в ие8,9 и других браузерах слайдер работает.

Вложения

index.rar (1.59 Мб, 33 просмотров)

0

В меню, Главная должна менять цвет при :hover;

1

Добавлено через 1 минуту
danil-html, в меню Главная сделана как активная в настоящий момент страница, она не меняет текст при :hover.

0

Сообщение от Heidel Добавлено через 1 минуту danil-html, в меню Главная сделана как активная в настоящий момент страница, она не меняет текст при :hover.

Тогда почему она реагирует, когда наводишь не нее?

1

danil-html, а, ну да, действительно, это наверное надо убрать.

0

Посмотрел минут 10. Впринципе верстка нормальная, если все выглядит как надо и сверстано на первый взгляд более менее красиво, то никто придираться не будет. Но если нужна критика, то их есть у меня:

1. Пункт левого меню должен нажиматься по всей области, а не только по тексту.

2. Комментариев слишком много, но это субъективно. Открывающие комментарии к дивам по большей части не нужны. Закрывающие полезны. Если смотреть на код, то это так выглядит
И без комментария понятно, что тут идет шапка. ИД у дива полностью повторяет комментарий. Он только мешается.
Но вот при закрытии этого же самого дива комментарий будет полезен, потому что в закрывающем тэге никакого ИД не написано.

3. В классе .clear нужно прописать {clear:both; height:0; font-size:0;}. Просто на всякий случай. Если этого не прописывать, то они в некоторых случаях могут ненулевую высоту занимать. И еще, не уверен что все дивы с классом clear нужны на странице. Они нужны только чтобы float прервать. Возможно в верстке они просто так стоят, вобщем то не особо и мешают, но критика же

4. В href у ссылок нужно не просто пустоту оставлять, а писать
Такой вариант позволит сразу оценивать :active у ссылок и гарантирует что ссылка будет выглядеть как обычная из-за того что адрес у нее заполнен. Кроме того если такие ссылки проще переделывать при натягивании этого макета на CMS, так как сразу видно какие ссылкии еще не затронули.


Думаю хватит пока. Писал дольше, чем это искал)
Если поискать, то еще наверное можно что-то заметить.

1

Alorian, большое спасибо, что посмотрели

подскажите, а как сделать так, чтобы пункт левого меню нажимался по всей области, а не только по тексту ?
width же работает только для блочных элементов. получается, надо задать для ссылки display: block и width: 100%
или как-то еще?

Сообщение от Alorian В href у ссылок нужно не просто пустоту оставлять, а писать Код HTML 1 <a href="javascript:void(0)">Каталог декоров</a>

а это точно? я ни в чьей верстке такого не видела)

0

Сообщение от Heidel display: block и width: 100%

Примерно так. Только width:100% не нужно. При display:block элемент сам займет всю доступную ширину. Главное внутренних отступов от <li> не делать. Все отступы только внутри самой ссылки.

Сообщение от Heidel а это точно? я ни в чьей верстке такого не видела)

А где какой то прием нужно встретить чтобы начать его использовать?)
И сколько раз его нужно встретить?

http://www.artlebedev.ru/tools... tml/links/

Для того чтобы ссылка стала «кликабельной», у нее обязательно прописывается атрибут href. Но если оставить href пустым, то MSIE интерпретирует ее как ссылку на текущую директорию. Например, если страница имеет адрес /folder/page.html, то href="" внутри такой страницы ссылается на /folder/. Остальные браузеры интерпретируют пустой href как ссылку на текущую страницу. В обоих случаях при клике на ссылку происходит перезагрузка страницы.

Если верстка лежит просто в файле, а не на сайте, то при первом просмотре инстиктивно хочется покликать по ссылкам и посмотреть как это все будет примерно работать. Если у всех ссылок стоит пустой адрес, то при каждом клике по ссылке придется возвращаться назад, так как он скорее всего будет вести в пустоту.

При натягивании на CMS потом гораздо легче найти в файле строку "javascript:void(0)", чем просто две кавычки. Ссылка с "javascript:void(0)" практически наверняка еще не обработана, а пустой адрес легко пропустить. На работающем сайте практически ото всех ссылок с "javascript:void(0)" конечно нужно будет избавиться.

В целом это только рекомендация, как практически и все в верстке. Я уж точно не заставляю вставлять в адресе "javascript:void(0)", но сам так делаю.

1

Alorian, ясно, большое спасибо за объяснения

0