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

[codeliner]

На выходных сверстал, если можно так выразиться, еще одну страницу.

Макет в png  http://codeliner.ucoz.com/fadelicious/home.png

Макет в psd  https://yadi.sk/i/Td7IVc0pqKKNc

Сверстанная веб-страница  http://codeliner.ucoz.com/fadelicious/index.html

Прошу объективной критики и дельных советов на будущее 

Edited March 21, 2016 by codeliner

[codeliner]

Хм, более 30 просмотров и ни одного ответа. В других темах, я смотрю, коээфициент - от 10 до 20 просмотров на ответ. Интересно, это связано с простотой страницы, или еще с чем-нибудь 

[Spotlight]

1. Логотип должен быть ссылкой и в теге img, он не должен добавляться через css.
2. Блокам offer я бы дал обертку div.wrap-offer и ему задал margin-top: 50px; а не каждому блоку давать отступ сверху и так как там вы используете float: left; то нужно убрать обтекание у блока.
3. Много ненужных свойств для блока global-container http://pixs.ru/showimage/2016032222_3973083_21223897.png
4. Ненужно давать чуть ли ни каждому блоку border: 0px solid black;
5. Изображения в слайдере должны добавляться через тег img.
6. Также не понятно почему вы дали копирайту left: -50%; margin-left: 170px; если он без этих свойств выглядит так как по макету.
7. Шрифт в меню, кнопках View details и в тексте приветствия не такой как в макете.
8. Зачем давать блоку second-footer width: 200%;  left: -50%; ?
9. Зачем в head тег <style> ?

Edited March 22, 2016 by Spotlight

[alejandro13]

1. Я бы в блоках offer кнопку "view details" делал обычной ссылкой, а не тегом button - зачем он там?

2. Зачем вешать столько свойств на абзац? http://prntscr.com/ainp4t 

И что вообще дает width и height:auto?

[Int]

  On 3/22/2016 at 8:10 PM, Spotlight said:

8. Зачем давать блоку second-footer width: 200%;  left: -50%; ?

Expand  

Видимо, это чтобы не делать transform: translateX(-50%)

 

У меню Contact справа полоска осталась.

[codeliner]

  On 3/22/2016 at 8:10 PM, Spotlight said:

1. Логотип должен быть ссылкой и в теге img, он не должен добавляться через css.

Expand  

Полностью согласен

  On 3/22/2016 at 8:10 PM, Spotlight said:

2. Блокам offer я бы дал обертку div.wrap-offer и ему задал margin-top: 50px; а не каждому блоку давать отступ сверху и так как там вы используете float: left; то нужно убрать обтекание у блока.

Expand  

Тоже об этом подумал, когда уже сделал страницу. У меня из-за того, что они идут без обертки произошло схлопывание, из-за чего были проблемы с размещением футера. Поэтому у global-containerа идет overflow:auto.

  On 3/22/2016 at 8:10 PM, Spotlight said:

3. Много ненужных свойств для блока global-container http://pixs.ru/showimage/2016032222_3973083_21223897.png

Expand  

Да, но height:auto нужно, т.к. без него футер правильно не позиционируется. Overflow:auto тоже нужно, о чем я писал выше.

 

  On 3/22/2016 at 8:10 PM, Spotlight said:

4. Ненужно давать чуть ли ни каждому блоку border: 0px solid black;

Expand  

Просто изначально я даю каждому блоку border:1px solid black, чтобы лучше видеть его очертания и, таким образом, нагляднее видеть результат кода. А потом я это свойство не удаляю, а лишь обнуляю, чтобы в случае необходимости можно было бы быстрее включить границу

 

  On 3/22/2016 at 8:10 PM, Spotlight said:

5. Изображения в слайдере должны добавляться через тег img.

Expand  

Почему обязательно так? Или вы имеете в виду, что эти изображения должны быть ссылками?

 

  On 3/22/2016 at 8:10 PM, Spotlight said:

6. Также не понятно почему вы дали копирайту left: -50%; margin-left: 170px; если он без этих свойств выглядит так как по макету.

Expand  

Там было разное отображение в различных браузерах, поэтому пришлось ставить такие заплатки. Поэтому, кстати, подключены еще отдельные стили для IE, для Firefox и для Safari. По другому как-то не получалось.

  On 3/22/2016 at 8:10 PM, Spotlight said:

7. Шрифт в меню, кнопках View details и в тексте приветствия не такой как в макете.

Expand  

Это да, шрифты я подключать еще пока не научился.

 

  On 3/22/2016 at 8:10 PM, Spotlight said:

8. Зачем давать блоку second-footer width: 200%;  left: -50%; ?

Expand  

Чтобы скрыть внутреннюю тень слева и справа (т.к. она внутренняя, то способ с псевдоэлементом не прокатывает). По этой же причине его высота на 20 пикселей больше, чем видимая, и он смещен на 20 пикселей вверх (margin-top:-20px).

 

  On 3/22/2016 at 8:10 PM, Spotlight said:

9. Зачем в head тег <style>

Expand  

Видимо забыл удалить (подключал внутренние стили, когда не срабатывали подключенные - там проблемы со специфичностью были).

 

  On 3/22/2016 at 8:36 PM, alejandro13 said:

Я бы в блоках offer кнопку "view details" делал обычной ссылкой, а не тегом button - зачем он там?

Expand  

Это да, согласен.

  On 3/22/2016 at 8:36 PM, alejandro13 said:

2. Зачем вешать столько свойств на абзац? http://prntscr.com/ainp4t 

И что вообще дает width и height:auto?

Expand  

Ну там процентов 80 свойств нужные. А width, и height auto - назначаю просто, чтобы проговаривать про себя, что у блока вот эти свойства, и к чему это приводит и.т.д. - в учебных целях, одним словом.

 

  On 3/24/2016 at 11:52 AM, Int said:

Видимо, это чтобы не делать transform: translateX(-50%)

Expand  

Похоже, можно было и так сделать, но ширину все равно больше 100% надо назначать - и так и так.

 

  On 3/24/2016 at 11:52 AM, Int said:

У меню Contact справа полоска осталась.

Expand  

Забыл указать li:last-child a {background:none;}

Спасибо всем за ответы!

Edited March 26, 2016 by codeliner

[jkikksa]

Решил заказчик добавить еще один пункт меню... и дописать немного текста в колонки...