Евдокимов Виктор #45
Евдокимов Виктор #45
Conversation
|
🍏 Пройден линтинг и базовые тесты |
|
There was a problem hiding this comment.
Посмотри повнимательнее на именование классов по БЭМу. Получилось не везде. Можешь почитать вот эту документацию: https://ru.bem.info/methodology/key-concepts/
Если что-то непонятно, то спрашивай
| <div class="header__setting">Настройка</div> | ||
| <div class="header__login"> | ||
| <p class="header__login_title">login</p> | ||
| <p class="header__login_mail"> |
There was a problem hiding this comment.
_ в БЭМ обычно используется для указания модификатора. А у тебя просто элементы. Лучше переименовать в header__login-mail, header__login-title
И везде в остальных классах посмотри на это тоже
| </div> | ||
| </div> | ||
| </article> | ||
| <article class="advertising-1"> |
There was a problem hiding this comment.
А вот тут как раз стоит использовать модификаторы. У тебя один по сути блок - реклама. Но он может выглядеть по-разному, модификаторы как раз про это.
То есть сделай вместо advertising-1 -> advertising advertising_size_big, например. И у второй рекламы соответственно так же, но с другим модификатором
| </section> | ||
| <section class="search-layout"> | ||
| <article class="tabs"> | ||
| <a href="#" class="tabs_tab main-link">Видео</a> |
There was a problem hiding this comment.
Если это элемент, то надо использовать двойное нижнее подчеркивание
| <span>Фиксики: большой секрет</span> | ||
| <a href="#" class="link">мультфильм</a> | ||
| </div> | ||
| <div class="column-widgits__widget__content"> |
Посмотреть решение