kiwka / icons-task

Page with 9 simple icons on it

Hi there 👋

icons-task's Issues

Ошибки и замечания

1. Форматирования html. Если мы в один блок вкладываем 2 и более блоков, каждый новый блок лучше записывать каскадом с новой строки.
   <li class="shape icons__item icon_home">
<div class="shape gradient">
<div class="shape symbol_home"&gt;&lt;/div>
</div>
</li>
2. <div class="shape gradient"> - на мой взгляд избыточный блок. Конечно верстальщик должен заранее думать о гибкости кода и легкости весенний изменений, но я не могу придумать почему бы не перенести класс .gradieny и .shape к icons__item, а <div class="shape gradient"> удалить.
3. Дробить стили для блока по разным классам нужно только если мы знаем, что будут варианты без этих параметров. Например, будут иконки квадратные или без градиента. Если такого нет, то можно смело переносить параметры из .shape и .gradient в .icons__item
4. Модификаторы с именами .icon_home для цветов иконок не очень хороши, т. к. если программист при помощи html захочет "перекрасить" все в один цвет, он всем иконкам назначит .icon_home. Получится что-то странное. Лучше писать имена, которые соответствуют тому, что делает этот класс. В нашем случае .icon_green явно лучше.