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

@AnastasiaBabushkina AnastasiaBabushkina commented Nov 21, 2016

@honest-hrundel
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

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@dotokoto
Copy link
Copy Markdown

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

@dotokoto
Copy link
Copy Markdown

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

@dotokoto
Copy link
Copy Markdown

dotokoto commented Nov 21, 2016

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

@dotokoto
Copy link
Copy Markdown

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

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

Choose a reason for hiding this comment

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

<header>?

Comment thread index.html
</section>
</div>
<div class="rating">
<h2>РЕЙТИНГ</h2>
Copy link
Copy Markdown

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;

Comment thread index.html
<input type="checkbox" id="rating_input">
<div class="position second">
<p>Абиссинская</p>
<div class="count c9"></div>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css

.type_selectors
{
display: flex;
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.css
padding-top: 20px;
padding-left: 16px;
border-bottom: 1px solid #d3d3d3;

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

.count
{
display: flex;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css

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

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

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

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