Skip to content

Фомин Денис#28

Open
ilearnf wants to merge 20 commits intourfu-2016:masterfrom
ilearnf:master
Open

Фомин Денис#28
ilearnf wants to merge 20 commits intourfu-2016:masterfrom
ilearnf:master

Conversation

@ilearnf
Copy link

@ilearnf ilearnf commented Nov 21, 2016

@honest-hrundel honest-hrundel changed the title Денис Фомин Фомин Денис Nov 21, 2016
@honest-hrundel
Copy link

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

1 similar comment
@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@onufrienko
Copy link

Что-то явно пошло не так, и я не буду смотреть, пока не поправишь
image

@ilearnf
Copy link
Author

ilearnf commented Nov 22, 2016

Поддержка Grid включена? Какой браузер?

@onufrienko
Copy link

А flex?

<p>maincoone</p>
</div>
</article>
<article class="card" title="bengal">

Choose a reason for hiding this comment

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

Кажется, отступы пропали


.breed-category:hover
{
background: #ccf;

Choose a reason for hiding this comment

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

здесь #ccf, а ниже blue. Давай единообразно делать


.rating-bar[title='2']
{
width: 20%;

Choose a reason for hiding this comment

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

попробуй заюзать attr(), вдруг получится убрать повторение правил :)

@onufrienko
Copy link

C grid'ом я посмотрю, но flex-то будет?

@onufrienko
Copy link

Рейтинг должен быть градиентом

@onufrienko
Copy link

Включила поддержку grid, яндекс браузер
image

@onufrienko
Copy link

onufrienko commented Nov 24, 2016

Ты задаешь у картинок ширину в 100%, но у родительских элементов ширина не задана. И поэтому картинки не круто растягиваются. Надо подумать о выравнивании картинок, чтобы выглядело адекватно.

@onufrienko
Copy link

И еще я не вижу, чтобы что-то происходило при клике на меню слева.

@onufrienko
Copy link

🍅

@honest-hrundel
Copy link

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

@ilearnf
Copy link
Author

ilearnf commented Dec 10, 2016

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@honest-hrundel
Copy link

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

@onufrienko
Copy link

🚀

@forshtreter
Copy link

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