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

@agrossers agrossers commented Nov 1, 2018

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

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

Comment thread index.css
transition: .5s all;
}

p:hover
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
transition: .5s all;
}

.goriz,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css
}

.goriz,
.full
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css
display: none;
}

.vert,
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
{
color: orange;
transform: scale(1.01);
transition: .5s all;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css

p:hover
{
color: orange;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css
display: block;
}

input:checked + label ~ div.card
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.html
<body>
<input type="checkbox" id="btnControl">
<label class="btn" for="btnControl"><span class="vert">|||</span><span class="goriz">≡</span></label>
<div class="card">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

article?

Comment thread index.html
</head>
<body>
<input type="checkbox" id="btnControl">
<label class="btn" for="btnControl"><span class="vert">|||</span><span class="goriz">≡</span></label>
Copy link
Copy Markdown

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