Skip to content

Зориков Кирилл#87

Open
agrossers wants to merge 11 commits intourfu-2018:masterfrom
agrossers:master
Open

Зориков Кирилл#87
agrossers wants to merge 11 commits intourfu-2018:masterfrom
agrossers:master

Conversation

@agrossers
Copy link

@agrossers agrossers commented Nov 1, 2018

@honest-hrundel honest-hrundel changed the title Зориков Кирилл Владимирович Зориков Кирилл Nov 1, 2018
@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

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

transition: .5s all;
}

p:hover

Choose a reason for hiding this comment

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

не используй теги, используй классы
поправь везде

transition: .5s all;
}

.goriz,

Choose a reason for hiding this comment

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

не используй сокращение и транслит
horizontal имел в виду? horizontal что?

}

.goriz,
.full

Choose a reason for hiding this comment

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

что за full непонятно, лучше назвать подробнее

display: none;
}

.vert,

Choose a reason for hiding this comment

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

то же самое

{
color: orange;
transform: scale(1.01);
transition: .5s all;

Choose a reason for hiding this comment

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

не используй all, лучше перечислить анимируемые свойства


p:hover
{
color: orange;

Choose a reason for hiding this comment

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

используй hex-цвета

display: block;
}

input:checked + label ~ div.card

Choose a reason for hiding this comment

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

кажется, уточнения div тут не нужны

<body>
<input type="checkbox" id="btnControl">
<label class="btn" for="btnControl"><span class="vert">|||</span><span class="goriz">≡</span></label>
<div class="card">

Choose a reason for hiding this comment

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

article?

</head>
<body>
<input type="checkbox" id="btnControl">
<label class="btn" for="btnControl"><span class="vert">|||</span><span class="goriz">≡</span></label>

Choose a reason for hiding this comment

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

иконка попадапет на первую строку с кошками и смещает ее
а еще коты должны быть отцентрованы

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