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

Кобяков Сергей #37

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

Кобяков Сергей #37

wants to merge 7 commits into from

Conversation

skobyakov
Copy link

@skobyakov skobyakov commented Nov 21, 2016

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

Copy link

@Victoria-Vladimirova Victoria-Vladimirova left a comment

Choose a reason for hiding this comment

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

Кажется, ты забыл все предыдущие замечания к задачам :с

</head>
<body>
<div class="wrapper">
<div class="content">
<div class="title">Бабуленькины котятки</div>

Choose a reason for hiding this comment

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

😱

Choose a reason for hiding this comment

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

header? h1?

<input id="select-british" type="checkbox" class="select-british">
<input id="select-blue" type="checkbox" class="select-blue">
<input id="select-siamese" type="checkbox" class="select-siamese">
<div class="cat-breed">

Choose a reason for hiding this comment

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

nav

<span class="select-breed"><span class="breed-name">Сиамская</span></span>
</label>
</div>
<div class="cat-gallery">

Choose a reason for hiding this comment

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

                <div class="cat-gallery">
                    <div class="line">
                        <div class="photo"></div>
                        <div class="photo"></div>
                    </div>
                    <div class="line">
                        <div class="photo"></div>
                        <div class="photo"></div>
                        <div class="photo"></div>
                    </div>
                    <div class="line">
                        <div class="photo"></div>
                        <div class="photo"></div>
                    </div>
                    <div class="line">
                        <div class="photo"></div>
                    </div>
                </div>

Choose a reason for hiding this comment

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

и да main

<div class="line"><div class="photo"></div><div class="photo"></div></div>
<div class="line"><div class="photo"></div></div>
</div>
<div class="cat-rating-box">

Choose a reason for hiding this comment

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

aside

</div>
</div>
</div>
<div class="footer">

Choose a reason for hiding this comment

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

footer

flex-direction: column;
}

.breed-rating

Choose a reason for hiding this comment

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

у этого класса точно такие же свойства, как у .cat-rating-info


label
{
font-family: Tahoma, sans-serif;

Choose a reason for hiding this comment

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

вместо того, чтобы везде писать это семейство, просто задавай его у body, например

.rating
{
display: flex;
flex-direction: row;

Choose a reason for hiding this comment

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

по умолчанию

background: linear-gradient(to right, #fff601, #fff);
border-left: 1px solid #b2ac01;
border-right: 1px solid #cacccb;
width: 131px;

Choose a reason for hiding this comment

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

то есть рейтинг 8 и рейтинг 6 будут одинаковой длины?

width: 200px;
height: 70px;
margin-top: 195px;
background-color: rgba(0,0,0,.37);

Choose a reason for hiding this comment

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

пробелы

@Victoria-Vladimirova
Copy link

Левая колонка с меню всегда подстраивается под высоту контейнера с фотками

2016-11-25 0 01 57

  1. котики не входят 😿
    2016-11-25 0 03 01

  2. Когда ты отключаешь категорию котиков, то у неё не должно быть ховера
    2016-11-25 0 02 15

  3. Почему категории расположены в ряд? Как на счет того, чтобы котики из одной категории находились в разных рядах? Например 1й котик и 5й

  4. выравнивание левого блока не соответствует тому, что на макете

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@skobyakov
Copy link
Author

🍏

@Victoria-Vladimirova
Copy link

🚀

@mokhov
Copy link
Contributor

mokhov commented Dec 5, 2016

2016-12-05 14-24-25

Copy link
Contributor

@mokhov mokhov left a comment

Choose a reason for hiding this comment

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

🍅

</nav>
<main>
<div class="line">
<div class="image"><img src="images/1.jpg" alt="Cat Photo"></div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Как-то описания без фантазии :(


.line:nth-child(4n-3) .image:nth-child(1):hover::after
{
content: 'Гоша, 4 года';
Copy link
Contributor

Choose a reason for hiding this comment

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

Ну нет, так никто никогда не делает – контент должен быть в HTML, а не в CSS

cursor: pointer;
}

.select-abyssinian:checked ~ main .line:nth-child(1) .image:nth-child(1) img,
Copy link
Contributor

Choose a reason for hiding this comment

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

Всё это можно было сделать гораздо проще, если бы у каждой картинки был бы нужный класс, а сейчас я могу добавить/удалить новую картинку и мне придется переделывать CSS чуть более чем полностью

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.

4 participants