Skip to content

Comments

Назарова Галина#40

Open
galkadaw wants to merge 2 commits intourfu-2016:masterfrom
galkadaw:master
Open

Назарова Галина#40
galkadaw wants to merge 2 commits intourfu-2016:masterfrom
galkadaw:master

Conversation

@galkadaw
Copy link

@galkadaw galkadaw commented Nov 26, 2016

@honest-hrundel
Copy link

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

@vetoshko
Copy link

vetoshko commented Dec 2, 2016

Фильтр по породе работает только если нажать на текст, нужно поправить

image
Тут блоки с котиками уехали

image
Тут видны границы картинки. Нужно что-то с этим сделать

index.css Outdated

.sort-rates
{
direction: rtl;
Copy link

Choose a reason for hiding this comment

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

А если появится еще текст?
Для него отдельно направление будешь задавать?

index.css Outdated
margin-top: 15px;
}

.sort-rates > .sort-rating
Copy link

Choose a reason for hiding this comment

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

Названия не описывают сути. Выглядят одинаково

@honest-hrundel
Copy link

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

@vetoshko
Copy link

vetoshko commented Dec 3, 2016

🚀

@honest-hrundel honest-hrundel assigned maxvipon and unassigned vetoshko Dec 3, 2016
</ul>
<div class="photos flex">
<div class="photo abissinskaya">
<img src="img/Абиссинская1.jpg" alt="Абиссинская кот Гоша" width="250" height="286">
Copy link

Choose a reason for hiding this comment

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

На будущее: чтобы не заливать в репозиторий тонну картинок, используй просто ссылки на них

14 2016-12-05 11-19-23

@maxvipon
Copy link

maxvipon commented Dec 5, 2016

При меньшем количестве фотографий макет ломается:

image

image

image

@maxvipon
Copy link

maxvipon commented Dec 5, 2016

Из задания:

Подвал должен быть прибит к нижней границе окна браузера, если котики влезают в один экран.

image

Не прибит

<div class="description">Клеопатра, 3 месяца</div>
<div class="close-photo"></div>
</div>
<div class="photo britanskaya">
Copy link

Choose a reason for hiding this comment

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

тут прекрасно подойдет <figure>

<div class="photo abissinskaya">
<img src="img/Абиссинская1.jpg" alt="Абиссинская кот Гоша" width="250" height="286">
<div class="description">Гоша, 2 года</div>
<div class="close-photo"></div>
Copy link

Choose a reason for hiding this comment

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

Зачем этот элемент?

<div class="sort-rates">
<input type="checkbox" id="sort-rating-flex" class="sort-rating-checkbox">
<label for="sort-rating-flex" class="sort-rating-checkbox">по убыванию</label>
<div class="rates">
Copy link

Choose a reason for hiding this comment

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

Зачем эта обертка?

</main>
<footer>
<span>©ООО Бабуленькины котятки</span>
<a href="#">Наш адрес: ул.Котовая, 5</a>
Copy link

Choose a reason for hiding this comment

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

Зачем эта ссылка?

<div class="rates">
<div class="rate-item nine">
<div>Абиссинская</div>
<div class="rate"></div>
Copy link

Choose a reason for hiding this comment

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

Лишний элемент. Как сделать по-другому?

<label for="sort-rating-flex" class="sort-rating-checkbox">по убыванию</label>
<div class="rates">
<div class="rate-item nine">
<div>Абиссинская</div>
Copy link

Choose a reason for hiding this comment

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

Нужен ли этот <div>?

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