Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ермаков Степан #36

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Ермаков Степан #36

wants to merge 21 commits into from

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

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

@@ -2,7 +2,131 @@
<html lang="ru">
<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.

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

</head>
<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 отвечает на вопрос "Что изображено на картинке?"

<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 чего?
Нужны какие-то единицы

}
}

@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 пока под запретом

</head>
<body>
<nav>

Choose a reason for hiding this comment

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

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

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)

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.

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

p.img
{
height: 50%;

Choose a reason for hiding this comment

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

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

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.

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

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