Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

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

wants to merge 8 commits into from

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

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

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