Skip to content

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

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

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

Conversation

@ilearnf

@ilearnf ilearnf commented Nov 21, 2016

Copy link
Copy Markdown

@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

ilearnf commented Nov 22, 2016

Copy link
Copy Markdown
Author

Поддержка 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

onufrienko commented Nov 24, 2016

Copy link
Copy Markdown

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

@onufrienko

Copy link
Copy Markdown

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

@onufrienko

Copy link
Copy Markdown

🍅

@honest-hrundel

Copy link
Copy Markdown

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

@ilearnf

ilearnf commented Dec 10, 2016

Copy link
Copy Markdown
Author

меню пород исправил, градиенты сделал, ширину родительского элемента у картинки добавил, а у его родительского элемента ширина задана настройками 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