Skip to content

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

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

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

Conversation

@georgy-n

@georgy-n georgy-n commented Nov 1, 2018

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