Skip to content

Набойченко Георгий#95

Open
georgy-n wants to merge 4 commits intourfu-2018:masterfrom
georgy-n:master
Open

Набойченко Георгий#95
georgy-n wants to merge 4 commits intourfu-2018:masterfrom
georgy-n:master

Conversation

@georgy-n
Copy link
Copy Markdown

@georgy-n georgy-n commented 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

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

Comment thread index.css
display: none;
}

img
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

используй класс

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.html
<body>
<input id="FullInfoSwitch" class="full-info-switch" type="checkbox">
<label class="full-info-switch-pic" for="FullInfoSwitch"></label>
<article 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.

котики должны располагаться по центру

Comment thread index.html
<title>Задача «Про котиков»</title>
</head>
<body>
<input id="FullInfoSwitch" class="full-info-switch" type="checkbox">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

id тоже лучше через -

@VasiliiKuznecov
Copy link
Copy Markdown

при наведении на пункты списка карточки прыгают, хочется, чтобы такого не было

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