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

@ilearnf ilearnf commented Nov 21, 2016

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

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

1 similar comment
@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@onufrienko
Copy link
Copy Markdown

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

@ilearnf
Copy link
Copy Markdown
Author

ilearnf commented Nov 22, 2016

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

@onufrienko
Copy link
Copy Markdown

А flex?

Comment thread index.html
<p>maincoone</p>
</div>
</article>
<article class="card" title="bengal">
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

.breed-category:hover
{
background: #ccf;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Comment thread index.css

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

Choose a reason for hiding this comment

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

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

@onufrienko
Copy link
Copy Markdown

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

@onufrienko
Copy link
Copy Markdown

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

@onufrienko
Copy link
Copy Markdown

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

@onufrienko
Copy link
Copy Markdown

onufrienko commented Nov 24, 2016

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

@onufrienko
Copy link
Copy Markdown

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

@onufrienko
Copy link
Copy Markdown

🍅

@honest-hrundel
Copy link
Copy Markdown

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

@ilearnf
Copy link
Copy Markdown
Author

ilearnf commented Dec 10, 2016

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@onufrienko
Copy link
Copy Markdown

🚀

@forshtreter
Copy link
Copy Markdown

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