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

@malaev malaev commented Nov 1, 2018

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

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

@Xrias
Copy link
Copy Markdown

Xrias commented Nov 2, 2018

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

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

Copy link
Copy Markdown

@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.

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

Comment thread index.html
</header>
<article>
<div class="image_container">
<img alt="salem" src="https://cdn5.img.ria.ru/images/148839/96/1488399690.jpg">
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 для картинок

Comment thread index.html
<label for="view" class="hamburger">☰</label>
</header>
<article>
<div class="image_container">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css
@@ -0,0 +1,98 @@
header
Copy link
Copy Markdown

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