Как оцените верстку?

Question

[Ксюша]

Друзья, оцените пожалуйста мою верстку.

Макет:


Ссылка на codepen (посмотрите пожалуйста также в мобильной версии 575px):
https://codepen.io/vatrushka/pen/GwMKLM?editors=1100

Я совсем новичок, посмотрите пожалуйста, всё ли делаю верно, может есть что улучшить, или что-то делаю совсем не так, может у кого-то есть полезные ссылки на материалы по вёрстке (на русском). Правильно ли задала затемнение фону, нормально ли то, что дала высоту banner (без этого нормально не могла наложить затемнение, поэтому добавила фиксированную высоту), правильно ли дала ширину текстовому контенту (заголовку и тексту в баннере, чтобы было как в макете), и т.д.

Буду благодарна за любую помощь!)

Comments

[sim3x]

0. Изучите английский
1. Заведите гитхаб и освойте гитхаб пейджес
2. Освойте сборку гульпом/вебпаком
3. Используйте imgur

[xmoonlight]

Возможно, что вопрос - это нарушение правил п. 5.11. Опросы и соцопросы.

[Максим Тимофеев]

Если Вы юзаете sass, то почему бы не избежать лишней вложенности

<div class="container">
      <div class="header__content">

В данном месте и еще ниже по коду такие места есть, где можно просто наследовать стили в css для блока от блока container или же просто иметь
<div class="container header__content">

Answer 1

[Максим]

575 ? скролл ужас какой ...
если забыть про адаптивность то нормально

Comments

[Ксюша]

Прошу прощения, исправила. Делала работу на VS Code и не всё перенесла в Codepen.

[Максим]

Ксюша, а это должго быть на таком расстоянии ?

Answer 2

[Froggyweb]

паддинги у каждого элемента banner__content 'это жесть
нижний отступ у текстовых блоков лучше задавать в em, т.к. он зависит от размера шрифта.
вертикальные отступы я бы сделал через margin
если это попытка БЭМ тоже есть вопросы например к блоку list в меню