Антонов Алексей - #24
AlekseyAnt wants to merge 1 commit into
Conversation
|
🍏 Пройден линтинг и базовые тесты |
|
🍏 Пройден линтинг и базовые тесты |
1 similar comment
|
🍏 Пройден линтинг и базовые тесты |
| .h-view:checked ~ .main .cat | ||
| { | ||
| display: block; | ||
| width: 1000px; |
There was a problem hiding this comment.
Когда показывается списком, то если разрешение экрана у пользователя меньше 1000px, он будет расстроен
| margin: 15px 0; | ||
| } | ||
|
|
||
| .h-view:checked ~ .main .img-content |
There was a problem hiding this comment.
Давай переименуем h-view и v-view на более понятные названия
|
|
||
| .h-view:checked ~ .main .description | ||
| { | ||
| width: 400px; |
There was a problem hiding this comment.
Давай сделаем верстку поадаптивнее, например, можно посмотреть в сторону min-width или max-width
| <h1 class="logo">Мяндекс.<span>Муррркет</span></h1> | ||
| <p> | ||
| <label for="vertical"> | ||
| <img src="images/v-view.jpg" alt="Горизонтальные плитки" title="Горизонтальные плитки"> |
There was a problem hiding this comment.
Молодец, что сделал не обычным инпутом 👍
|
Привет. Мне очень понравилась твоя работа) Есть пара некритичных замечаний, но я надеюсь, что ты их поправишь 😉 Еще есть последний вопрос: будешь ли делать звездочки? |
|
🚀 |
|
🍏 Пройден линтинг и базовые тесты |
| <main class="main"> | ||
| <div class="cat male"> | ||
| <div class="img-content"> | ||
| <img src="images/01.jpg" alt="" title="Генрих 4"> |
|
|
||
| .price:after | ||
| { | ||
| content: '₽'; |
There was a problem hiding this comment.
В случае с ценой так делать, пожалуй, не стоит, потому что текст в псевдоэлементе нельзя выделить и скопировать. Но в целом пододное внимание к мелочам – это очень хорошо 👍 Так что можно не исправлять :)
| </p> | ||
| </header> | ||
| <main class="main"> | ||
| <div class="cat male"> |
There was a problem hiding this comment.
Алексей, многовато div'ов. Здесь, например, подойдёт article, для обёртки картинки figure, для имени кота – заголовок.
|
🍅 |
|
Пока помидор, но в целом очень даже неплохо, по звёздочке вобще никаких вопросов, только мелочёвку подправить. |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройден линтинг и базовые тесты |
|
🍏 Пройден линтинг и базовые тесты |
4 participants
Посмотреть решение