Skip to content

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

Open
predatorfromekb wants to merge 21 commits intourfu-2016:masterfrom
predatorfromekb:master
Open

Ермаков Степан#36
predatorfromekb wants to merge 21 commits intourfu-2016:masterfrom
predatorfromekb:master

Conversation

@predatorfromekb
Copy link
Copy Markdown

@predatorfromekb predatorfromekb commented Nov 7, 2016

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

1 similar comment
@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

1 similar comment
@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

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

Choose a reason for hiding this comment

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

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

Comment thread index.html Outdated
<body>
<nav>
<figure>
<p class="img"><img src="1.png" alt=""></p>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Картинке надо задать атрибуты title и alt. Значение alt отвечает на вопрос "Что изображено на картинке?"

Comment thread 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>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css Outdated
}
}

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

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

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

Choose a reason for hiding this comment

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

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

Comment thread index.css Outdated
figure
{
margin: 1px;
float: left;
Copy link
Copy Markdown

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)

Comment thread index.css Outdated
text-align: center;
border: 1px solid black;
}
nav
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css Outdated
margin: 0;
padding: 0;
}
figure
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css Outdated
figure
{
width: 100%;
float: bottom;
Copy link
Copy Markdown

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
Copy Markdown

Victoria-Vladimirova commented Nov 10, 2016

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

@Victoria-Vladimirova
Copy link
Copy Markdown

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

@Victoria-Vladimirova
Copy link
Copy Markdown

Victoria-Vladimirova commented Nov 10, 2016

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

@Victoria-Vladimirova
Copy link
Copy Markdown

🍅

@honest-hrundel
Copy link
Copy Markdown

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

1 similar comment
@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

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