Разметка #3
Разметка #3
Conversation
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
source/index.html
Outdated
| <p class="site__logo-title-bold">CAT</p> | ||
| <p class="site__logo-title-default">ENERGY</p> |
There was a problem hiding this comment.
текст тоже изобраение, лучше объедини все в 1но целую картинку.
Это нужно с точки зрения эстетики, типо чтоб пока шрифт грузиться не портил логотип
source/index.html
Outdated
| <section class="programs"> | ||
| <div class="programs__wrapper"> | ||
| <header class="programs__header"> | ||
| <h2 class="programs__title">Функциональное питание для котов</h2> |
There was a problem hiding this comment.
можно visually-hidden убрать и это оставить h1
source/index.html
Outdated
| <header class="programs__header"> | ||
| <h2 class="programs__title">Функциональное питание для котов</h2> | ||
| <p class="programs__slogan">Занялся собой? Займись котом!</p> | ||
| <button class="programs__button">Подобрать программу</button> |
There was a problem hiding this comment.
- это ссылка на страницу подбор программы
- рекомендую добавить класс button
source/index.html
Outdated
| Борис сбросил 5 кг за 2 месяца, просто<br> | ||
| заменив свой обычный корм на Cat Energy<br> | ||
| Slim. Отличный результат без изнуряющих<br> | ||
| тренировок! При этом он не менял своих<br> | ||
| привычек и по-прежнему спит по 16 часов<br> | ||
| в день. |
There was a problem hiding this comment.
смело удаляй br)))
у нас теперь адаптив и текст будет переноситься
source/index.html
Outdated
| Конюшенная, д. 19/8<br> | ||
| Санкт-Петербург | ||
| </p> | ||
| <img src="img/location.png" width="320px" height="362px" alt="Расположение офиса CAT ENERGY"> |
There was a problem hiding this comment.
точку можно убрать, твоя задача поставить карту через ifraime
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
There was a problem hiding this comment.
https://yoksel.github.io/html-tree/ сервис по проверки БЭМ нотации
source/catalog.html
Outdated
| <div class="card__description"> | ||
| <div class="card__definition"> | ||
| <p class="card__definition-name">Масса</p> | ||
| <p class="card__definition-value">500 г</p> | ||
| </div> | ||
| <div class="card__definition"> | ||
| <p class="card__definition-name">Вкус</p> | ||
| <p class="card__definition-value">Курица</p> | ||
| </div> | ||
| <div class="card__definition"> | ||
| <p class="card__definition-name">Цена</p> | ||
| <p class="card__definition-value">700 ₽</p> | ||
| </div> | ||
| </div> |
There was a problem hiding this comment.
более семантично было бы через table
source/catalog.html
Outdated
| </head> | ||
| <body class="page__body"> | ||
| <header class="main-header"> | ||
| <a class="site__logo"> |
There was a problem hiding this comment.
блок site потерялся)
тут можно просто .logo написать
source/catalog.html
Outdated
| <nav class="main-nav"> | ||
| <button class="main-nav__toggle" type="button"><span class="visually-hidden">Открыть меню</span></button> | ||
| <div class="main-nav__wrapper"> | ||
| <ul class="main-nav__list secondary-nav__list"> |
There was a problem hiding this comment.
блок secondary-nav потерен
думаю данный элемент вообще можно будет удалить и модифиактор secondary-nav__item--active
source/catalog.html
Outdated
| </header> | ||
|
|
||
| <main class="main-container"> | ||
| <h1 class="catalog__title">Каталог продукции</h1> |
There was a problem hiding this comment.
блок catalog ниже, а элемент не может выходить из блока
source/catalog.html
Outdated
| <p class="additional-goods-card__description-name">1 литр</p> | ||
| <p class="additional-goods-card__description-price">100 ₽</p> | ||
| </div> | ||
| <a class="button card__link">Заказать</a> |
source/form.html
Outdated
| <div class="form__additional control"> | ||
| <h2 class="control__title">Дополнительно</h2> | ||
| <ul class="control__list"> | ||
| <li class="additional-info__item"> |
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
🎓 Разметка
💥 https://htmlacademy-adaptive.github.io/2553233-cat-energy-1/3/