прошу оценить верстку

[dimadyak]

сам psd http://htmlforum.ru/index.php?showtopic=16763

 

что получилось http://fb7951fc.bget.ru/

 

какие ошибки недочеты  можете выделить. 

Заранее спасибо.

[WEB-best]

<div class="main">
                    <div class="nav">
                        <ul class="cf">
                            <li><a href="#">Home</a></li>
                            <li><a href="#">Services</a></li>
                            <li><a href="#">Support</a></li>
                            <li><a href="#">Careers</a></li>
                            <li><a href="#">About</a></li>
                            <li><a href="#">Contact</a></li>
                        </ul>
                    </div>

зачем класс ul?
когда к нему доступ можно получить и без этого класса
 

[Crystal-3]

Общее впечатление - неплохо, но мне не понравилось как вы вырезали картинку с небоскрёбом.

И ещё есть пару вопросов.

Зачем вы подключаете шрифт "trebuchet_ms" ? Это же стандартный шрифт.

И вот что это?

.sidebar, .main {    margin-bottom: -32000px;    padding-bottom: 32000px;}

Edited June 5, 2014 by Crystal-3

[WEB-best]

 

Общее впечатление - неплохо, но мне не понравилось как вы вырезали картинку с небоскрёбом.

И ещё есть пару вопросов.

Зачем вы подключаете шрифт "trebuchet_ms" ? Это же стандартный шрифт.

И вот что это?

.sidebar, .main {    margin-bottom: -32000px;    padding-bottom: 32000px;}

это такой прием сделать фон на всю высоту)

[Crystal-3]

 WEB-best, спасибо, буду знать.

[amelice]

<div class="main">

                    <div class="nav">

                        <ul class="cf">

                            <li><a href='#">Home</a></li>

                            <li><a href="#">Services</a></li>

                            <li><a href='#">Support</a></li>

                            <li><a href="#">Careers</a></li>

                            <li><a href='#">About</a></li>

                            <li><a href="#">Contact</a></li>

                        </ul>

                    </div>

зачем класс ul?

когда к нему доступ можно получить и без этого класса

 

Так есть класс общий для учета флоатов, он и применил сразу к списку.  Думаю, тут все верно.

 

По верстке мне очень понравилось. 

Нужно отметить:

-Стили, разметка очень аккуратная.

-Элементы на своем месте.

-Продуманы повторяющиеся блоки.

-Название классов тоже продуманы.

-Со знанием использованы свойства 

Н-р: многие пишут для центровки блока - margin: 0 auto; а у вас margin: auto; (вертикальные маргины при авто обнуляются.)

 

Чего нужно: 

Обратите внимание на поиск, нужно связать лейбл с текстовым инпутом. То есть чтоб при нажатии на лейбл - автоматом фокусировался инпут, можно сделать двумя путями, либо связать с id и for, либо обернуть инпут лейблом.

По поводу организации лого, я бы наверное выбрала тег strong вместо b.

 

Удачи вам!

Edited June 5, 2014 by amelice

[dimadyak]

 

Общее впечатление - неплохо, но мне не понравилось как вы вырезали картинку с небоскрёбом.

И ещё есть пару вопросов.

Зачем вы подключаете шрифт "trebuchet_ms" ? Это же стандартный шрифт.

И вот что это?

.sidebar, .main {    margin-bottom: -32000px;    padding-bottom: 32000px;}

а вы как бы вырезали?

[Crystal-3]

dimadyak, я тоже верстала этот макет и вырезала всю картинку целиком (небоскрёб с облаками), на случай если заказчику захочется немного опустить  блок с контентом то будут видны облака, а не пустота.

 

Хочу вставить картинку png, но не могу найти как это сделать.

[andrdov]

8 ошибок в валидности http://validator.w3.org/check?uri=http%3A%2F%2Ffb7951fc.bget.ru%2F&charset=%28detect+automatically%29&doctype=Inline&group=0