Прошу критики по верстке

[Temik2704]

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

Скажите пожалуйста насколько по вашему мнению я справился с версткой учебного макета.

Прошу указать на ошибки.

 

Стоит отметить,что ширина центральной колонки по макету была 1170 пикс,я верстал под 1024,поэтому немного переделывал на ходу.

Проблемы которые я осознаю,но не знаю как решить  :

1.Один блок div находится в теге h2,но он там нужен,либо переделывать все h2 на блоки,что увеличит кол-во кода.

2.Если уменьшить окно браузера до размеров <1024пикс то появляется полоса прокрутки (это норма) , но почему-то обрезается почти весь сайт справа(кроме некоторых блоков).Это как я понял из-за overflow hidden,но даже если их все поубирать то все равно будет обрезание.

 

Макет:http://radikal.cc/lfp/s017.radikal.ru/i426/1401/e8/2f06f73fb93c.jpg/htm

PSD:http://vk.com/doc209718014_181719773?hash=e918a4fff4938e409d&dl=2de16a8e2ec1bbc196

 

Результат:http://www.anhiora.esy.es/

[advokatua]

За сколько времени весь макет сверстал?

Банальных визуальных косяков не нашел.Хотел было сказать, что это лучшая верстка, которую за последнее время выкладывали на оценку, но затем сжал окно и тут )) Странно, что хватило опыта набросать весь макет, а на такую мелочь не хватило

 

По коду - в хтмл перебор с айдишками.

В css:

* {  padding: 0;  margin: 0;  box-sizing: border-box;  -moz-box-sizing: border-box;}

 Также слайдеры неплохо бы допилить.

 

ЗЫ Макет ничего так, тоже сделаю на днях   

Edited January 27, 2014 by advokatua

[Temik2704]

Пилил очень неспешно,дня 3-4 по 2-3 часа.

Вот хотелось бы услышать вкратце как исправить этот косяк с обрезанием (пытался задавать главным блокам(которые id) ширину 100% и как-то играться с overflow, но ничего не вышло,буду рад услышать советы по поводу этого косяка,в разделе ошибок на форуме мне так и не подсказали)

 

 

 

По коду - в хтмл перебор с айдишками.

В css:

* {
padding: 0;
margin: 0;
box-sizing: border-box;
-moz-box-sizing: border-box;
}

 

И скажите пожалуйста почему так нельзя делать?Вроде box-sizing поддерживается большинством версий браузеров.

Edited January 27, 2014 by Temik2704

[advokatua]

Вроде box-sizing поддерживается большинством версий браузеров.

Дело не в боксайзинге, а в универсальном селекторе. Хотя, относительно применения его к свойству box-sizing: border-box; надо подумать и посмотреть, если ли какая-то побочка, а то сегодня и сам думал - зачем к каждому блоку пилить эту боксовую модель, когда можно повесить ее глобально, что в разы проще и уменьшает лишнюю нагрузку на мозг. )

 

Поспешил похвалить. Макет действительно не самый нубский, причем судя по дизайну он предполагает адаптацию к более мелким разрешениям, а  значит верстать нужно куда внимательнее. В твоем случае конкретный косяк нашел на меню. Когда заказчику потребуется добавить в верхнее меню 2 доп. пункта, то посмотри, что произойдет. Сам сейчас его верстаю, ломаю голову. )

 

Косяк с обрезанием в данном случае вообще не столь важный выходит.

Edited January 30, 2014 by advokatua

[klierik]

советую почитать про ошибки по ссылке в подписи

[Ge1i0N]

<a href="#top"><img src="images/anchor-top.png" alt="anchor-top"></a>

Мне кажется что вставку картинки можно было бы заменить backgroundом в css.

 

Плюс картинку в верхнем слайдере неплохо бы разместить по центру, так как уменьшении масштаба она сползает к левому краю браузера.

Edited January 30, 2014 by Ge1i0N

[advokatua]

Плюс картинку в верхнем слайдере неплохо бы разместить по центру

Так она ведь репитится на всю ширину.