Skip to content

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

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

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

Conversation

@galkadaw

@galkadaw galkadaw commented Nov 26, 2016

Copy link
Copy Markdown

@honest-hrundel

Copy link
Copy Markdown

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

@vetoshko

vetoshko commented Dec 2, 2016

Copy link
Copy Markdown

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

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

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

Comment thread index.css Outdated

.sort-rates
{
direction: rtl;

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 Outdated
margin-top: 15px;
}

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

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

@honest-hrundel

Copy link
Copy Markdown

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

@vetoshko

vetoshko commented Dec 3, 2016

Copy link
Copy Markdown

🚀

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

Copy link
Copy Markdown

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

maxvipon commented Dec 5, 2016

Copy link
Copy Markdown

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

image

image

image

@maxvipon

maxvipon commented Dec 5, 2016

Copy link
Copy Markdown

Из задания:

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

image

Не прибит

Comment thread index.html
<div class="description">Клеопатра, 3 месяца</div>
<div class="close-photo"></div>
</div>
<div class="photo britanskaya">

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.html
<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
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.html
<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
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.html
</main>
<footer>
<span>©ООО Бабуленькины котятки</span>
<a href="#">Наш адрес: ул.Котовая, 5</a>

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.html
<div class="rates">
<div class="rate-item nine">
<div>Абиссинская</div>
<div class="rate"></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.

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

Comment thread index.html
<label for="sort-rating-flex" class="sort-rating-checkbox">по убыванию</label>
<div class="rates">
<div class="rate-item nine">
<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.

Нужен ли этот <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