Skip to content

Бабушкина Анастасия#22

Open
AnastasiaBabushkina wants to merge 8 commits intourfu-2016:masterfrom
AnastasiaBabushkina:master
Open

Бабушкина Анастасия#22
AnastasiaBabushkina wants to merge 8 commits intourfu-2016:masterfrom
AnastasiaBabushkina:master

Conversation

@AnastasiaBabushkina
Copy link

@AnastasiaBabushkina AnastasiaBabushkina commented Nov 21, 2016

@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

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

@dotokoto
Copy link

В firefox котейки разъехались:
- mozilla firefox 2016-11-21 21 35 22

@dotokoto
Copy link

Меню едет, если уменьшить ширину экрана
- google chrome 2016-11-21 21 39 07

@dotokoto
Copy link

dotokoto commented Nov 21, 2016

Не такая длина левой колонки, должна быть вровень с последней фоткой
Если котиков мало, то появляется много пустого места между котиками
- google chrome 2016-11-21 21 44 13
- google chrome 2016-11-21 21 42 29

@dotokoto
Copy link

Футер нечестно прибит. У меня появляется скролл, если удалить котиков, а футер только если проскроллить вниз. Надо без скролла при маленьком контенте

<link rel="stylesheet" href="index.css">
</head>
<body>
<h1>Бабуленькины котятки</h1>

Choose a reason for hiding this comment

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

<header>?

</section>
</div>
<div class="rating">
<h2>РЕЙТИНГ</h2>

Choose a reason for hiding this comment

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

Можно не писать капсом в html, а в css задать text-transform: uppercase;

<input type="checkbox" id="rating_input">
<div class="position second">
<p>Абиссинская</p>
<div class="count c9"></div>

Choose a reason for hiding this comment

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

Плохо, что появляются пустые элементы в html. Можно попробовать сделать то же самое с помощью :before


.type_selectors
{
display: flex;

Choose a reason for hiding this comment

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

В этом меню можно и не использовать флекс. Если его убрать, ничего не поменяется

padding-top: 20px;
padding-left: 16px;
border-bottom: 1px solid #d3d3d3;

Choose a reason for hiding this comment

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

лишняя пустая строка


.count
{
display: flex;

Choose a reason for hiding this comment

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

Тут тоже flex не нужен


label[id='rating_label']
{
display: flex;

Choose a reason for hiding this comment

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

И здесь не нужен flex. Не нужно усложнять css, расположить текст по центру и без флексов легко )

@dotokoto
Copy link

🍅 Есть версточные баги, и не везде совпадает с макетом. Жду правок

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments