Skip to content

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

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

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

Conversation

@ilearnf
Copy link
Copy Markdown

@ilearnf ilearnf commented Oct 31, 2016

@honest-hrundel honest-hrundel changed the title Денис Фомин Фомин Денис Oct 31, 2016
@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

@onufrienko
Copy link
Copy Markdown

У картинок нет подписей

Comment thread index.css Outdated
background: #c5d010;
}

.half
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
.half
{
width: 50%;
float: left;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

В этом задании плавающие элементы не должны быть использованы

@onufrienko
Copy link
Copy Markdown

Почему бы не подключить дополнительные шрифты? :(
Так будет интереснее :)

Comment thread index.html
<div class="newspaper">
<div class="half">
<article class="three-columns">
<p class="formatted">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Если это заголовок, то почему бы его не обозначить как заголовок?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

это не заголовок.

Comment thread index.html Outdated
little more gungho

Are you a sensible, grown-up adult that calculates how long one tank of fuel can
last? Are you a practical,</p>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Тег лучше перенести на отдельную строку, а то с отступами что-то неладное

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

да.

Comment thread index.html Outdated
</p>
</article>
<article class="three-columns">
<img src="http://i.i-sgcm.com/news/article_events/2016/3295_4_l.jpg" alt="car"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

У изображений хорошо бы добавить атрибут title

@onufrienko
Copy link
Copy Markdown

Можно ещё использовать какой-нибудь сервис расстановки переносов

@onufrienko
Copy link
Copy Markdown

Давай пока исправишь то, что перечислила, а я позже еще посмотрю

@onufrienko
Copy link
Copy Markdown

🍅

@ilearnf
Copy link
Copy Markdown
Author

ilearnf commented Nov 10, 2016

что такое "подписи к картинкам"? сейчас там есть подписи, как я понял их определение.

@honest-hrundel
Copy link
Copy Markdown

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

@onufrienko
Copy link
Copy Markdown

выглядит не аккуратно и не хорошо
image

@onufrienko
Copy link
Copy Markdown

Подпись к фотке - это когда под картинкой есть какое-то описание. Посмотри про тег figure

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.

3 participants