Ермаков Степан#36
Ермаков Степан#36predatorfromekb wants to merge 21 commits intourfu-2016:masterfrom predatorfromekb:master
Conversation
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
1 similar comment
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
1 similar comment
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройден линтинг и базовые тесты |
index.html
Outdated
| <head> | ||
| <meta charset="utf-8"> | ||
| <title>Магазин Котиков</title> | ||
| <link rel="stylesheet" href="5.css"> |
index.html
Outdated
| <body> | ||
| <nav> | ||
| <figure> | ||
| <p class="img"><img src="1.png" alt=""></p> |
There was a problem hiding this comment.
зачем оборачивать картинку в параграф?
Картинке надо задать атрибуты title и alt. Значение alt отвечает на вопрос "Что изображено на картинке?"
index.html
Outdated
| <div class="name">Рыжик</div> | ||
| <div class="category">Беспородный</div> | ||
| <div class="rating"><img src="star.png" alt=""></div> | ||
| <div class="price">14000</div> |
There was a problem hiding this comment.
14000 чего?
Нужны какие-то единицы
index.css
Outdated
| } | ||
| } | ||
|
|
||
| @media screen and (min-width: 501px) |
There was a problem hiding this comment.
@media пока под запретом
index.html
Outdated
| <link rel="stylesheet" href="5.css"> | ||
| </head> | ||
| <body> | ||
| <nav> |
There was a problem hiding this comment.
обычно этот тег содержит навигацию по сайту
index.css
Outdated
| figure | ||
| { | ||
| margin: 1px; | ||
| float: left; |
There was a problem hiding this comment.
Вообще эту задачу можно решить без использования float
Конкретно это свойство никогда не будет применено. Применяется свойство, описанное внутри @media screen and (min-width: 501px)
index.css
Outdated
| text-align: center; | ||
| border: 1px solid black; | ||
| } | ||
| nav |
There was a problem hiding this comment.
что ты пытаешься с ним сделать?
index.css
Outdated
| p.img | ||
| { | ||
| height: 50%; | ||
|
|
index.css
Outdated
| margin: 0; | ||
| padding: 0; | ||
| } | ||
| figure |
There was a problem hiding this comment.
Не очень хорошо писать такие селекторы. Давай использовать классы?
index.css
Outdated
| figure | ||
| { | ||
| width: 100%; | ||
| float: bottom; |
There was a problem hiding this comment.
такого значения у float просто нет
|
Описание одного котика может переехать в блок другого котика при ресайзе |
|
❗️ Решение не соответствует макетам в задании: рейтинг выполнен в виде одинокой звезды, поведение блоков на среднем и большом экране одинаковое, имя котика выполнено не в виде ссылки, имена не обрезаются, нет переключателя, нет эффектов при наведении |
|
❗️ Решение по ссылке не работает, т.к в html подключается не тот css-файл |
|
🍅 |
|
🍅 Не пройден линтинг или базовые тесты |
1 similar comment
|
🍅 Не пройден линтинг или базовые тесты |
|
🍅 Не пройден линтинг или базовые тесты |
|
🍏 Пройден линтинг и базовые тесты |
3 participants
Посмотреть решение