search

inconsumable1812 / Toxin

MetaLamp, task 2. Сайт Toxin
0 stars 0 forks source link

Scss #7

Closed ArsenGadjiusmanov closed 2 years ago

ArsenGadjiusmanov commented 2 years ago

Глобальные стили не допустимы. Допустимы только для тегов body, html и для переопределения box-sizing для всех тегов https://github.com/inconsumable1812/Toxin/blob/50e2fd3b0901cd8a823d2c95d4d8162ac62a290f/src/components/input/input.scss#L79-L81

ArsenGadjiusmanov commented 2 years ago

По стандартам использование миксов запрещено https://github.com/inconsumable1812/Toxin/blob/50e2fd3b0901cd8a823d2c95d4d8162ac62a290f/src/components/info/info.pug#L5

Нельзя также переопределять стили для другого БЭМ блока https://github.com/inconsumable1812/Toxin/blob/50e2fd3b0901cd8a823d2c95d4d8162ac62a290f/src/components/info/info.scss#L23-L25

ArsenGadjiusmanov commented 2 years ago

Тут и в других местах. Классы с префиксом js мы использует только для поиска dom elements из javascript, стилизовать ими ничего не надо image

ArsenGadjiusmanov commented 2 years ago

Именование должно быть более осмысленным (секция 1, 2, 3, 4 мало о чем говорит)

image

ArsenGadjiusmanov commented 2 years ago

Блок может влиять только на свои элементы и не может воздействовать на элементы другого блока и тем более на стили другого блока) image Тут надо сделать тоже что и с lib-ами вынести код

ArsenGadjiusmanov commented 2 years ago

Для шрифтов лучше использовать переменные image а еще лучше если ты сделаешь миксин для шрифтов и будешь юзать его

ArsenGadjiusmanov commented 2 years ago

Такие значения тоже лучше вынести в переменные image image

ArsenGadjiusmanov commented 2 years ago

Единицы измерения можно опустить для нулевых значений тут лучше указать т.к. поведение на разных браузерах может отличаться image

ArsenGadjiusmanov commented 2 years ago

image Нельзя использовать селекторы по тегам

ArsenGadjiusmanov commented 2 years ago

image Перенеси код модификатора во внутрь, чтобы был только 1 селектор на файл

ArsenGadjiusmanov commented 2 years ago

image Закомментированный код лучше всегда удалять

ArsenGadjiusmanov commented 2 years ago

image Глобальные стили не допустимы. Допустимы только для тегов body, html и для переопределения box-sizing для всех тегов

ArsenGadjiusmanov commented 2 years ago

image Именование переменных должно быть семантическим (смысловым). Обычно в дизайн системе есть главный цвет (main-color), дополнительный цвет, основной цвет шрифта, цвет заголовка и т д. Что если главный цвет сменится с color-purple на color-pink, возникнет путаница.