Исправление ошибок в БЭМ #2
Conversation
| @@ -1,7 +1,7 @@ | |||
| # Личный проект «Кэт энерджи» | |||
|
|
|||
| * Студент: [Анастасия Филиппович](https://up.htmlacademy.ru/adaptive-individual/2/user/605225). | |||
| * Наставник: `Неизвестно`. | |||
| * Наставник: [Виктория Никитина](https://htmlacademy.ru/profile/id1217273). | |||
| <span class="visually-hidden">Открыть меню</span> | ||
| </button> | ||
| <ul class="main-navigation__list"> | ||
| <li class="main-navigation__list list-item"> |
There was a problem hiding this comment.
бэм блок list-item выглядит лишним
list-item__link -> main-navigation__link будет достаточно
| <section class="hero"> | ||
| <h2 class="hero__title">Функциональное питание для котов</h2> | ||
| <p class="hero__advertisement">Занялся собой? Займись котом!</p> | ||
| <img class="hero__product-image" src="#" width="280" height="270" alt="Продукт Cat energy"> |
There was a problem hiding this comment.
контентные изображения - помещай в обертку(див) для стилизации пикч
| <img class="hero__product-image" src="#" width="280" height="270" alt="Продукт Cat energy"> | ||
| <a class="hero__link" href="#">Подобрать программу</a> | ||
| </section> | ||
| <div class="block"> |
There was a problem hiding this comment.
из названия класса не понятно что оно означает
There was a problem hiding this comment.
также в этой секции 2 элемента, у тебя они разделены буд-то они разные по смыслу
| <section class="advantages"> | ||
| <h2 class="advantages__title">Как это работает</h2> | ||
| <ul class="advantages__list"> | ||
| <li class="advantages__list list-item"> |
There was a problem hiding this comment.
тоже блок list-item ? точно ли будут стили одинаковые или для чего блок?
выглядит также лишним
| </li> | ||
| </ul> | ||
| <p class="result-example__cost">Затраты на питание: 15 000 руб.</p> | ||
| <img class="result-example__image" src="#" width="280" height="256" alt="Кот: до и после"> |
There was a problem hiding this comment.
тут явно не хватает разметки
нужно и блок расположить и 2х котов отобразить, и разделитель и тд
| <p class="adress-container__offer">Приглашаем к сотрудничеству дилеров!</p> | ||
| <p class="adress-container__place">ул. Большая Конюшенная, д. 19/8 Санкт-Петербург</p> | ||
| </div> | ||
| <img class="map-image" src="#" width="320" height="362" alt="Карта с обозначением расположения магазина"> |
There was a problem hiding this comment.
картинка карты сама по себе почему-то
ожидается что будет блок, в котором будет контактная информация и карта
| </div> | ||
| <img class="map-image" src="#" width="320" height="362" alt="Карта с обозначением расположения магазина"> | ||
| <footer class="footer"> | ||
| <svg class="footer__text-logo" width="128" height="24"></svg> |
No description provided.