Skip to content

Игнатенко Алексей#25

Open
Alexxxir wants to merge 5 commits intourfu-2018:masterfrom
Alexxxir:master
Open

Игнатенко Алексей#25
Alexxxir wants to merge 5 commits intourfu-2018:masterfrom
Alexxxir:master

Conversation

@Alexxxir
Copy link

@Alexxxir Alexxxir commented Oct 31, 2018

@honest-hrundel honest-hrundel changed the title test Игнатенко Алексей Oct 31, 2018
@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

Copy link

@nex2hex nex2hex left a comment

Choose a reason for hiding this comment

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

🍅
При длинных текстах верстка ломается https://yadi.sk/i/ERSRdAHKRttnFA

index.html Outdated
<input type="checkbox" name="show-list" id="show-list" class="show-list">
<label class="list-disabled" for="show-list">☰</label>
<label class="list-enabled" for="show-list">|||</label>
<div class="cat-info">
Copy link

Choose a reason for hiding this comment

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

article

index.html Outdated
<label class="list-enabled" for="show-list">|||</label>
<div class="cat-info">
<img src="images/1.jpg" alt="котик 1">
<a href="images/1.jpg">Гель</a>
Copy link

Choose a reason for hiding this comment

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

тэга с заголовком не хватает

index.html Outdated
<label class="list-disabled" for="show-list">☰</label>
<label class="list-enabled" for="show-list">|||</label>
<div class="cat-info">
<img src="images/1.jpg" alt="котик 1">
Copy link

Choose a reason for hiding this comment

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

картинку тоже лучше обернуть в ссылку

Aleksey and others added 2 commits December 19, 2018 20:35
@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

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