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

@skobyakov skobyakov 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

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

Copy link
Copy Markdown

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

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

Comment thread index.html Outdated
<body>
<div class="wrapper">
<div class="content">
<div class="title">Бабуленькины котятки</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.

😱

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? h1?

Comment thread 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">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nav

Comment thread index.html Outdated
<span class="select-breed"><span class="breed-name">Сиамская</span></span>
</label>
</div>
<div class="cat-gallery">
Copy link
Copy Markdown

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>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

и да main

Comment thread 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">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

aside

Comment thread index.html Outdated
</div>
</div>
</div>
<div class="footer">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

footer

Comment thread index.css
flex-direction: column;
}

.breed-rating
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css Outdated

label
{
font-family: Tahoma, sans-serif;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css
.rating
{
display: flex;
flex-direction: row;
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
background: linear-gradient(to right, #fff601, #fff);
border-left: 1px solid #b2ac01;
border-right: 1px solid #cacccb;
width: 131px;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css Outdated
width: 200px;
height: 70px;
margin-top: 195px;
background-color: rgba(0,0,0,.37);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

пробелы

@Victoria-Vladimirova
Copy link
Copy Markdown

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

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

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@skobyakov
Copy link
Copy Markdown
Author

🍏

@Victoria-Vladimirova
Copy link
Copy Markdown

🚀

@mokhov
Copy link
Copy Markdown
Contributor

mokhov commented Dec 5, 2016

2016-12-05 14-24-25

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

🍅

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

Choose a reason for hiding this comment

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

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

Comment thread index.css

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

Choose a reason for hiding this comment

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

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

Comment thread index.css
cursor: pointer;
}

.select-abyssinian:checked ~ main .line:nth-child(1) .image:nth-child(1) img,
Copy link
Copy Markdown
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