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

[oleg126]

Спасибо всем за предыдущие отзывы, прошу оценить верстку заново, может что еще не так?

http://testsiteoleg126.tk/CreativeDesignServices/

[ryter]

padding отрицательный не может быть 

ссылки на соц сети думаю лучьше разделить классами и не обязательно вставлять тег  i 

спрайт лучьше клеить в *.png иконки на прозрачном фоне

hr в вашем случае думаю лишний можно просто border-top использовать 

в списке вместо img лучьше использовать :before так будет намного удобнее

OUR PORTFOLIO 

тут думаю всеже лучьше использовать img а не background-position

в спрайт картинки портфолио не к чему клеить

[cyklop77]

в блоке our portfolio используй float или display: inline-block. сейчас последний блок съехал вниз

[oleg126]

Странно но на компе все норм, а в инете съехал(

[amelice]

<meta charset="utf-8"> перед title
Ширина блока такая же как у контейнера. То есть можно не указывать ширину для header, #content, portfolio, footer так как указана для .wrapper
Можно было обойтись без абсолюта для навигации.
Так же чам отличаются блоки - c-block, c-block2?
Черный бордер который сделан отдельным блоком .c-line-block  это ж обычный бордер.
Блок .content-previev-img и ниже следующие h1 и h2.. почему они раздельные?)
 

[oleg126]

перед title

 

Исправил!

 

Ширина блока такая же как у контейнера. То есть можно не указывать ширину для header, #content, portfolio, footer так как указана для .wrapper

 

 

Исправил!

 

Можно было обойтись без абсолюта для навигации.

 

Как?

 

@amelice,  

Так же чам отличаются блоки - c-block, c-block2?

 

 

Там разные стили для текстов и картинки!

 

Черный бордер который сделан отдельным блоком .c-line-block  это ж обычный бордер.

 

 

Я знаю, просто я делая бордер для h3 не знал, как его отодвинуть от самого текста в h3, и решил сделать как знаю! Если поможете буду рад!

 

Блок .content-previev-img и ниже следующие h1 и h2.. почему они раздельные?)

 

 

Ну потому что там же разные стили текста, поэтому один в h1,а другой в h2!

 

OUR PORTFOLIO  тут думаю всеже лучьше использовать img а не background-position

 

 

Говорили, что лучше много CSS кода, чем HTML. Единственное что, background-position лучше запихнуть в main.css.

 

спрайт лучьше клеить в *.png иконки на прозрачном фоне

 

 

Так а у самих картинок уже есть фон, как я его уберу?

 

Исправленный сайт - http://testsiteoleg126.tk/CreativeDesignServices/

[amelice]

Я знаю, просто я делая бордер для h3 не знал, как его отодвинуть от самого текста в h3, и решил сделать как знаю! Если поможете буду рад!

http://cssdeck.com/labs/pukhgqc8 

[ryter]

 

перед title

 

Исправил!

 

 

 

Ширина блока такая же как у контейнера. То есть можно не указывать ширину для header, #content, portfolio, footer так как указана для .wrapper

 

 

Исправил!

 

 

 

Можно было обойтись без абсолюта для навигации.

 

Как?

 

@amelice,  

 

Так же чам отличаются блоки - c-block, c-block2?

 

 

Там разные стили для текстов и картинки!

 

 

 

Черный бордер который сделан отдельным блоком .c-line-block  это ж обычный бордер.

 

 

Я знаю, просто я делая бордер для h3 не знал, как его отодвинуть от самого текста в h3, и решил сделать как знаю! Если поможете буду рад!

 

 

 

Блок .content-previev-img и ниже следующие h1 и h2.. почему они раздельные?)

 

 

Ну потому что там же разные стили текста, поэтому один в h1,а другой в h2!

 

 

 

OUR PORTFOLIO  тут думаю всеже лучьше использовать img а не background-position

 

 

Говорили, что лучше много CSS кода, чем HTML. Единственное что, background-position лучше запихнуть в main.css.

 

 

 

спрайт лучьше клеить в *.png иконки на прозрачном фоне

 

 

Так а у самих картинок уже есть фон, как я его уберу?

 

Исправленный сайт - http://testsiteoleg126.tk/CreativeDesignServices/

 

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

с css не нужно перебарщивать контент должен быть контентом в частности картинки в контенте не путать с иконками 

[oleg126]

Спасибо, исправил, теперь есть еще что то?

 

http://testsiteoleg126.tk/CreativeDesignServices/

 

 

перед title

 

Исправил!

 

 

 

Ширина блока такая же как у контейнера. То есть можно не указывать ширину для header, #content, portfolio, footer так как указана для .wrapper

 

 

Исправил!

 

 

 

Можно было обойтись без абсолюта для навигации.

 

Как?

 

@amelice,  

 

Так же чам отличаются блоки - c-block, c-block2?

 

 

Там разные стили для текстов и картинки!

 

 

 

Черный бордер который сделан отдельным блоком .c-line-block  это ж обычный бордер.

 

 

Я знаю, просто я делая бордер для h3 не знал, как его отодвинуть от самого текста в h3, и решил сделать как знаю! Если поможете буду рад!

 

 

 

Блок .content-previev-img и ниже следующие h1 и h2.. почему они раздельные?)

 

 

Ну потому что там же разные стили текста, поэтому один в h1,а другой в h2!

 

 

 

OUR PORTFOLIO  тут думаю всеже лучьше использовать img а не background-position

 

 

Говорили, что лучше много CSS кода, чем HTML. Единственное что, background-position лучше запихнуть в main.css.

 

 

 

спрайт лучьше клеить в *.png иконки на прозрачном фоне

 

 

Так а у самих картинок уже есть фон, как я его уберу?

 

Исправленный сайт - http://testsiteoleg126.tk/CreativeDesignServices/

 

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

с css не нужно перебарщивать контент должен быть контентом в частности картинки в контенте не путать с иконками 

 

 

Все, мой косяк был, исправил на png, ща все норм?

Кстати, сейчас заметил, что в Опере и IE в our portfolio последнее изображение чуть чуть вниз съехало, как исправить, в Мозилле все норм

[webmasterFast]

Хотел спросить, верстка это у вас хобби ?

А то я подумал вы бесплатно все верстаите.

 

По верстке норма.

Картинка последняя ушла в портфолио, поставьте 0px у тебя 33px стояло:

 

#portfolio nav a {

1. margin-right: 0px;
2. margin-bottom: 20px;

}

Edited June 11, 2014 by webmasterFast

[Ironimus]

Еще футер не полностью прижат к низу.

[oleg126]

Хотел спросить, верстка это у вас хобби ?

 

Конечно хобби, но хочу, что бы стало профессией!)

 

Картинка последняя ушла в портфолио, поставьте 0px у тебя 33px стояло:   #portfolio nav a { margin-right: 0px; margin-bottom: 20px; }

 

 

 

Спасибо исправил!

 

Еще футер не полностью прижат к низу.

 

 

Вроде визуально прижат, или что исправить надо?

[Ironimus]

Что-то не очень он так прижат. 

UPD. Попробовал приблизить — сразу прижался. Потом что я не делал футер был прижат.
 

Edited June 11, 2014 by Ironimus