Jump to content

Оцените верстку пожалуйста. Укажите на ошибки.


13bis
 Share

Recommended Posts

Здравствуйте!

Выдалось свободное время и решил переверстать свой основной шаблон сайта. Буду очень признателен, если укажите на ошибки и способы их устранения.

Планирую переделать сайт кардинально, решил начать со структуры основного шаблона. Сверстал страницу, косяков пока не наблюдаю, код валидный, поддержка всех браузеров кроме IE6, но так как я не эксперт в CSS прошу взглянуть на код страницы, а мало-ли...

Собственно сама страница: http://splotnikov.ru/test/

Заранее, всем откликнувшимся, высказываю свою благодарность!

Link to comment
Share on other sites

лого лучше сделать ссылкой на главную

Вы имеете ввиду про букву P в круге?

Если да, то тогда объект кликабельный и по умолчанию ведёт на главную страницу.

Edited by 13bis
Link to comment
Share on other sites

Он о "Portfolio".

Вообще-то это как бы заголовок страницы)) Логотип расположил справа ("разрыв стандартных шаблонов").

P.S.Название страницы с очепяткой..

Поправил эту мелочь. Спасибо!

Link to comment
Share on other sites

  1. Имена классам лучше давать такие, которые несут смысловую нагрузку и на одном языке. zagolovok и right-bar-portfolio вместе не смотрятся, а классы l и r мало что о себе говорят. Конечно для себя можно сделать как угодно, но когда захочется что-то поправить в шаблоне через год или два, придется вспоминать и ковыряться.
  2. Пустой <div class="hr"></div> ради границы - не есть хорошо. Стоило бы использовать для этого border.
  3. Сложная система у правого logo с z-index. Раз уж не захотели просто перекрашивать цвет шрифта при наведении курсора, как внизу, то можно было бы вместо img использовать фон. Тогда бы обошлись бы a { background: (рисунок с белой буквой) } и a:hover { background: (рисунок с зеленой буквой) }
  4. Не понял, зачем нужен <div class="clearfix">. Видимо он остался со времен табличной верстки. Лучше такие подчищать, так как в будущем они могут преподнести массу "приятных" моментов, когда верстка куда-нибудь уползет, и сложно будет обнаружить причину.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy