search

Nurvive / MetaLamp-2-step

1 stars 0 forks source link

Миксы #2

Closed OlgaYubil closed 2 years ago

OlgaYubil commented 2 years ago

По нашим стандартам не разрешается использование миксов https://github.com/fullstack-development/front-end-best-practices/blob/master/CSS/README.md#4.5

Снимок экрана 2022-01-26 в 17 26 48 Снимок экрана 2022-01-26 в 17 27 03
OlgaYubil commented 2 years ago

Миксин называется subscription-text-field, значит и класс должен называться так же

Снимок экрана 2022-01-26 в 17 25 51
OlgaYubil commented 2 years ago

Здесь тоже примеси. Кроме этого, в одном файле два миксина, так быть не должно. И вообще в папке header не должно находиться посторонних файлов.

Снимок экрана 2022-01-27 в 19 18 08
OlgaYubil commented 2 years ago

Диаграмму имеет смысл выделить в отдельный компонент

Снимок экрана 2022-01-27 в 19 12 08
OlgaYubil commented 2 years ago

Не все миксы убрал

Снимок экрана 2022-01-31 в 10 38 45
OlgaYubil commented 2 years ago

Какой-то посторонний jsClass

Снимок экрана 2022-01-31 в 10 37 53
OlgaYubil commented 2 years ago

Не все миксы убрал

Снимок экрана 2022-02-02 в 10 13 36 Снимок экрана 2022-02-02 в 10 18 14
OlgaYubil commented 2 years ago

js- это только префикс к существующему классу

Снимок экрана 2022-02-03 в 11 12 00
OlgaYubil commented 2 years ago
Снимок экрана 2022-02-07 в 13 31 04