Skip to content

Потапов Лев#92

Open
malaev wants to merge 3 commits intourfu-2018:masterfrom
malaev:master
Open

Потапов Лев#92
malaev wants to merge 3 commits intourfu-2018:masterfrom
malaev:master

Conversation

@malaev
Copy link

@malaev malaev commented Nov 1, 2018

@honest-hrundel honest-hrundel changed the title закончил Потапов Лев Nov 1, 2018
@honest-hrundel
Copy link

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

@Xrias
Copy link

Xrias commented Nov 2, 2018

"Также от вас требуется:

Обрезать длинные имена котиков (=^ ◡ ^=)
Реализовать переключатель вывода котиков: плитками или списком
Реализовать эффекты при наведении на имя котика, фото, категорию, и плитку в целом
Фоточки котиков в плитках должны быть одного размера, чтобы блоки выглядели аккуратно <( ̄︶ ̄)>" Нет длинных имен. Нет эффектов при наведении.

Copy link

@Xrias Xrias left a comment

Choose a reason for hiding this comment

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

За Салема жирный лойс и за бутер тоже.

</header>
<article>
<div class="image_container">
<img alt="salem" src="https://cdn5.img.ria.ru/images/148839/96/1488399690.jpg">
Copy link

Choose a reason for hiding this comment

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

title для картинок

<label for="view" class="hamburger">☰</label>
</header>
<article>
<div class="image_container">
Copy link

Choose a reason for hiding this comment

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

А почему этот див это див, а не section? Давай поменяем.

@@ -0,0 +1,98 @@
header
Copy link

Choose a reason for hiding this comment

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

Всем тегам давай дадим классы, тегов в css остаться не должно.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants