Skip to content

Conversation

@predatorfromekb
Copy link

@predatorfromekb predatorfromekb commented Nov 7, 2016

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

1 similar comment
@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

1 similar comment
@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

index.html Outdated
<head>
<meta charset="utf-8">
<title>Магазин Котиков</title>
<link rel="stylesheet" href="5.css">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Плохое название файла

index.html Outdated
<body>
<nav>
<figure>
<p class="img"><img src="1.png" alt=""></p>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

зачем оборачивать картинку в параграф?

Картинке надо задать атрибуты 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>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

14000 чего?
Нужны какие-то единицы

index.css Outdated
}
}

@media screen and (min-width: 501px)
Copy link

@Victoria-Vladimirova Victoria-Vladimirova Nov 10, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@media пока под запретом

index.html Outdated
<link rel="stylesheet" href="5.css">
</head>
<body>
<nav>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

обычно этот тег содержит навигацию по сайту

index.css Outdated
figure
{
margin: 1px;
float: left;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вообще эту задачу можно решить без использования float

Конкретно это свойство никогда не будет применено. Применяется свойство, описанное внутри @media screen and (min-width: 501px)

index.css Outdated
text-align: center;
border: 1px solid black;
}
nav

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

что ты пытаешься с ним сделать?

index.css Outdated
p.img
{
height: 50%;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

лишняя пустая строка

index.css Outdated
margin: 0;
padding: 0;
}
figure

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не очень хорошо писать такие селекторы. Давай использовать классы?

index.css Outdated
figure
{
width: 100%;
float: bottom;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

такого значения у float просто нет

@Victoria-Vladimirova
Copy link

Victoria-Vladimirova commented Nov 10, 2016

Описание одного котика может переехать в блок другого котика при ресайзе
2016-11-10 16 30 31

@Victoria-Vladimirova
Copy link

❗️ Решение не соответствует макетам в задании: рейтинг выполнен в виде одинокой звезды, поведение блоков на среднем и большом экране одинаковое, имя котика выполнено не в виде ссылки, имена не обрезаются, нет переключателя, нет эффектов при наведении

@Victoria-Vladimirova
Copy link

Victoria-Vladimirova commented Nov 10, 2016

❗️ Решение по ссылке не работает, т.к в html подключается не тот css-файл

@Victoria-Vladimirova
Copy link

🍅

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

1 similar comment
@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants