макету это уже мы сами). На первый взгляд количество кода выглядит избыточным.
Пример: пункты горизонтального меню и бокового меню визуально и технически одинаковы. Те же эффекты наведения, параметры текста.
При этом верстальщик создаёт стиль отдельно и для того и для другого.
строка 2671 - для бокового меню
.main_menu .menu .item a{
font-weight: bold;
font-size: 13px;
line-height: 26px;
letter-spacing: 0.08em;
color: #262626;
font-family: var(--font_family2);
transition: .2s linear;
text-decoration: none;
}
.main_menu .menu .item a:hover{
color: #dfa94a;
}
.main_menu .menu .item.active a{
color: #7A7A7A;
}
Строки 1100 -1114 - для пунктов горизонтального меню каталога
header .menu .item a{
font-weight: bold;
font-size: 13px;
line-height: 26px;
text-align: center;
letter-spacing: 0.08em;
color: #262626;
text-decoration: none;
transition: .2s linear;
font-family: var(--font_family2);
}
header .menu .item a:hover{
color: #DFA94A;
}
Может я что-то недопонимаю, но мне видится что дублирование их - излишним. Я вижу что у них разные параметры text-align, но наличие всего лишь одного отличающегося параметра не являетс достаточной причиной создавать новый класс? Или я неправ?
сразу видно, что методологии нет, стили написаны очень хаотично, поддерживать такое будет тяжело семантически где-то нормально теги поставлены, где-то дивы налеплены, при чем в некоторых местах их сильно больше, чем нужно
По методологии имеете ввиду разбивку таблицы на функциональные группы элементов, как оно реализуется в шаблонах: Wrap, Buttons and Links и т.д ? Да действительно верстальщик бьет таблицу на разделы (Header, Typography и тд) - кажется это не очень правильно даже для неспециалиста)
под методологией лично я имею в виду бэм, но есть и другие варианты, здесь же все стили в кучу скинуты
можно сократить количество кода в куске css на 6 строк :)
нету основной семантической разметки. Все в кучу - footer одинокий внизу, остальное запихано в main. В некоторых секциях нет заголовков.
в мозилле полоски бургера разной высоты.
Обсуждают сегодня