Skip to content

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

Open
skobyakov wants to merge 7 commits intourfu-2016:masterfrom
skobyakov:master
Open

Кобяков Сергей#37
skobyakov wants to merge 7 commits intourfu-2016:masterfrom
skobyakov:master

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.

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

index.html Outdated
<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?

index.html Outdated
<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

index.html Outdated
<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

index.html Outdated
<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

index.html Outdated
</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

index.css Outdated

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 будут одинаковой длины?

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

Comments