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

[WebNewbe]

Доброго времени суток. Прошу оценить верстку.
ВАЖНО: если у вас не отображаются изображения в шапке страницы - отключите адблок
Почему-то он блочит изображения именно в шапке, пока решаю эту проблему
https://alex2033.github.io/

[WebNewbe]

Так, проблема с адблоком решена.

Оказывается он блокирует все блоки, в названии которых есть слово "banner". 

[npofopr]

Я бы на карте скролл убрал, но это вроде только на гугловских можно безболезненно сделать.

[mrnobody]

map.behaviors.disable(['scrollZoom']);

 

[WebNewbe]

1) Скроллинг карты был убран

2) Анимацию решил оставить только в шапке, ибо в том же сафари она глючит

3) Google PageSpeed: 77/mobile, 77/desktop - это достаточная скорость загрузки страницы?
4) Как считаете, стоит ли подобную верстку себе в портфолио добавлять? Достаточно ли она корректно реализована и достаточно ли данный макет сложен для портфолио?

[Zhukov_Anton]

Доброго дня. Заметил пару косяков при верстке:

1. Разрыв между накладкой над фоном города и верхним блоком белого цвета

2. Этот же блок, только его низ. Под этой накладкой наезжает уже следующий блок, что тоже не очень хорошо.

3. Карта слишком маленького размера по высоте, не сочетается. Так же под картой черная линия непонятная. Либо уехало у Вас что то, либо так задумано было, но это больше на поехвашее похоже

4. Меню в мобильной версии не работает

Скрины прилагаю http://yapx.ru/u/8k2B

[WebNewbe]

В 17.02.2018 в 17:45, Zhukov_Anton сказал:

Доброго дня. Заметил пару косяков при верстке:

1. Разрыв между накладкой над фоном города и верхним блоком белого цвета

2. Этот же блок, только его низ. Под этой накладкой наезжает уже следующий блок, что тоже не очень хорошо.

3. Карта слишком маленького размера по высоте, не сочетается. Так же под картой черная линия непонятная. Либо уехало у Вас что то, либо так задумано было, но это больше на поехвашее похоже

4. Меню в мобильной версии не работает

Скрины прилагаю http://yapx.ru/u/8k2B

Добрый вечер. Теперь адрес сайта -https://alex2033.github.io/Brandi/index.html Благодарю за отзыв, теперь по порядку:
1. Убрано
2. Убрано
3. Это скорее уже к дизайну. Старался делать его максимально похожим, увеличил высоту карты на 100px, а линия снизу задумана в макете
4. Она работает, попробуйте зайти с телефона. Я так понимаю у вас в браузере при разных разрешениях экрана не работает, потому что в js я прописал, что меню начинает свою работу при ширине менее 768px
if ($(window).width() < 768) {
}

Edited February 22, 2018 by WebNewbe

[andrey7287]

Норм,  можешь искать работу.

1 час назад, WebNewbe сказал:

if ($(window).width() < 768) {
}

Лень писать почему , просто не делай так никогда ...  

https://developer.mozilla.org/ru/docs/Web/API/Window/matchMedia

[npofopr]

А было бы не плохо, если напишете.

[WebNewbe]

1 час назад, andrey7287 сказал:

Норм,  можешь искать работу.

Лень писать почему , просто не делай так никогда ...  

https://developer.mozilla.org/ru/docs/Web/API/Window/matchMedia

Я согласен, мне такой метод тоже не нравится, потому что при том же каком-нибудь айпаде при горизонтальном и вертикальном отображении сайт будет себя вести не так как нужно. Просто пока что других методов не знаю. Спасибо за ссылку, обязательно ознакомлюсь

[andrey7287]

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

http://css-live.ru/articles/polnyj-kontrol-nad-kontrolnymi-tochkami-v-css-i-v-js.html

А всего лишь навсего надо было использовать matchMedia,который специально для этого существует.

PS Это уже потом он в статью добавил в качестве решения matchMedia, когда прочитал комментарии ))) А я ему ещё тут на форуме писал про это )))

Edited February 23, 2018 by andrey7287

[FiretrUCK]

Может лучше привязать background к body и .portfolio:before тоже к body ,чтобы избавиться от белой полосы

[WebNewbe]

25 минут назад, FiretrUCK сказал:

Может лучше привязать background к body и .portfolio:before тоже к body ,чтобы избавиться от белой полосы

не-не) Это за 5 минут сделанная страница, чтобы сюда размещать работы, позже уже сайт-портфолио полноценный буду делать. Вот сама работа, про которую тут идет речь https://alex2033.github.io/Brandi/index.html