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

[начинающий верстаг]

дайте оценку хотя бы по 5 ти бальной шкале portfoliox.ru/test

Edited December 3, 2017 by начинающий верстаг

[npofopr]

3

[начинающий верстаг]

6 минут назад, npofopr сказал:

3

почему поставили 3 ?

[andrew312]

body > div > div.header > div.header_wrap > div > div.header_right > span.tel

Зачем использовать span.tel и давать ему display:block? Не проще ли было взять div?

—

При ширине 1024 ,

@media screen and (max-width: 1080px)

style.css:838

.header_wrap {

height: 600px;

}

Ограничение высоты блока приведет к тому, что блок не растянется при добавлении текста. и текст уедет за пределы фона.

[начинающий верстаг]

11 минуту назад, andrew312 сказал:

body > div > div.header > div.header_wrap > div > div.header_right > span.tel

Зачем использовать span.tel и давать ему display:block? Не проще ли было взять div?

—

При ширине 1024 ,

@media screen and (max-width: 1080px)

style.css:838

.header_wrap {

height: 600px;

}

Ограничение высоты блока приведет к тому, что блок не растянется при добавлении текста. и текст уедет за пределы фона.

как думаешь, можно этот сайт показывать в портфолио?
 

[andrew312]

Да, можно конечно. Не всем же полезут в верстку ковыряться. Внешне главное чтобы все соответствовало макету.

[AlexZaw]

В мобильном варианте ваше гамбургер меню плохо видно на светлом фоне. И что за красная линия у логотипа?

 

[начинающий верстаг]

7 минут назад, AlexZaw сказал:

В мобильном варианте ваше гамбургер меню плохо видно на светлом фоне. И что за красная линия у логотипа?

 

это фишка такая, в psd макете тоже красная линия

[AlexZaw]

Я конечно не дизайнер, но красная линия как то выпадает из общего стиля, и, как мне кажется, если вы выложите такую работу в портфолио то многие сочтут ее за вашу ошибку, а не ошибку дизайнера или макета. Был бы это реальный проект я бы уточнил этот момент у дизайнера. На учебном макете либо убрал бы линию совсем, либо изменил бы цвет

[начинающий верстаг]

спасибо, ваш отзыв помог мне!!!

7 минут назад, AlexZaw сказал:

Я конечно не дизайнер, но красная линия как то выпадает из общего стиля, и, как мне кажется, если вы выложите такую работу в портфолио то многие сочтут ее за вашу ошибку, а не ошибку дизайнера или макета. Был бы это реальный проект я бы уточнил этот момент у дизайнера. На учебном макете либо убрал бы линию совсем, либо изменил бы цвет

спасибо