Укажите на недостатки

[brezze]

Посмотрите пожалуйста сайт. Буду рад любой критике. Хотелось бы понимать, в правильном ли направлении верстаю, и какие ошибки допущены в вёрстке. 

 

Brezze.github.io

[Svetlana_P]

Посмотрела бегло, заметила мелочи.

1. Если смотреть с телефона, то справа появляется белая полоска. Т.е. там сайт идет не на всю ширину.
2. Под кнопкой "Заказать оценку" декоративная волнистая полоска. На телефоне под ней появляется еще одна — тонкая и сплошная.
3. У оранжевых блоков-стрелках раздела "Как мы работаем" на телефоне появляются вертикальные полоски.
4. В блоке НЕЗАВИСИМАЯ ОЦЕНКА ИМУЩЕСТВА есть фоновая картинка с весами, от нее идут в стороны косые линии. На телефоне, при некоторых размерах, линии сильно расплываются.
5. Ссылки в блоке под "НАШИ УСЛУГИ ПО СОПРОВОЖДЕНИЮ СДЕЛОК M&A" скачут влево при первом наведении (смотрю в хроме).
6. В блоке "КАК МЫ РАБОТАЕМ" шрифт немного скачет — буква "п" странная.
7. Тексты хорошо бы оттипографить и исправить опечатки.
8. Местами не совсем понятны названия классов. Например, кнопка "Отправить" носит класс .btn и обернута в .button-wrapper. А кнопка "Заказать сопровождение" тоже с .btn, но обернута в .btn-wrap. Стили оберток зависят от родительских контейнеров, это видно в CSS. Но как-то это всё мало вяжется с БЭМ, по которому, как я вижу, организовывали большую часть стилей.

[brezze]

3 часа назад, Svetlana_P сказал:

Посмотрела бегло, заметила мелочи.

1. Если смотреть с телефона, то справа появляется белая полоска. Т.е. там сайт идет не на всю ширину.
2. Под кнопкой "Заказать оценку" декоративная волнистая полоска. На телефоне под ней появляется еще одна — тонкая и сплошная.
3. У оранжевых блоков-стрелках раздела "Как мы работаем" на телефоне появляются вертикальные полоски.
4. В блоке НЕЗАВИСИМАЯ ОЦЕНКА ИМУЩЕСТВА есть фоновая картинка с весами, от нее идут в стороны косые линии. На телефоне, при некоторых размерах, линии сильно расплываются.
5. Ссылки в блоке под "НАШИ УСЛУГИ ПО СОПРОВОЖДЕНИЮ СДЕЛОК M&A" скачут влево при первом наведении (смотрю в хроме).
6. В блоке "КАК МЫ РАБОТАЕМ" шрифт немного скачет — буква "п" странная.
7. Тексты хорошо бы оттипографить и исправить опечатки.
8. Местами не совсем понятны названия классов. Например, кнопка "Отправить" носит класс .btn и обернута в .button-wrapper. А кнопка "Заказать сопровождение" тоже с .btn, но обернута в .btn-wrap. Стили оберток зависят от родительских контейнеров, это видно в CSS. Но как-то это всё мало вяжется с БЭМ, по которому, как я вижу, организовывали большую часть стилей.

Спасибо, что уделили время. Сейчас тестирую на телефоне, действительно есть пустое пространство справа. Есть идеи почему ?)

Оранжевые блоки-стрелки нужно наверно через svg делать, потому-что эти пляски на css ни к чему хорошему не привели. 

Анимацию блоков наверно тоже лучше на телефоне отключить, как-то не очень смотриться.

[Svetlana_P]

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

Спасибо, что уделили время. Сейчас тестирую на телефоне, действительно есть пустое пространство справа. Есть идеи почему ?)

Скорее всего, какой-то блок вылезает по ширине из общего контейнера. Надо смотреть, какой именно)

А так да, в целом — на телефоне надо больше тестировать и править мелочи.