Skip to content

Лебедев Никита#31

Open
qmzik wants to merge 5 commits into
urfu-2017:masterfrom
qmzik:master
Open

Лебедев Никита#31
qmzik wants to merge 5 commits into
urfu-2017:masterfrom
qmzik:master

Conversation

@qmzik
Copy link
Copy Markdown

@qmzik qmzik commented Oct 22, 2017

@honest-hrundel
Copy link
Copy Markdown

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

Copy link
Copy Markdown

@msmirnov msmirnov left a comment

Choose a reason for hiding this comment

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

🍅

Comment thread index.css

.font-family:checked ~ div
{
font-family: 'Bookman Old Style';
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
background: #dadada;
height: 100%;
width: 100%;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Почему-то на любом разрешении экрана виден горизонтальный скролл, Такого быть не должно.
image

Comment thread index.html
</li>
<li><h3>Где купить классную вещи и по приятной цене</h3><br>
<span><img src="img/clothes.jpg" alt=""></span>
</li>
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
</li>
<li>
<h3>Почему Бузова не может есть стоя</h3><br>
<span><img src="img/buzova.jpg" alt=""></span>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

alt у всех картинок должен быть указан (а желательно ещё и title).

Comment thread index.css

div
{
padding: 10px;
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 нужно задавать отступ?

Comment thread index.html
</li>
</ul>
</footer>
</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.

  • Не увидел переносы в тексте
  • Нет картинок с подписями
  • Нет вертикально написанного текста
  • При ширине < 1245px правая колонка уезжает вниз.

Comment thread index.html
покончил
жизнь самоубийством, потому что считал это предательством в отношении собственной страны.

<data value="11.10.2017">11 октября</data>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Кажется, логичнее было бы использовать time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants