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

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

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

Conversation

galkadaw
Copy link

@galkadaw galkadaw commented Nov 26, 2016

@honest-hrundel
Copy link

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

@vetoshko
Copy link

vetoshko commented Dec 2, 2016

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

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

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


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

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

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