Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

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

wants to merge 5 commits into from

Conversation

qmzik
Copy link

@qmzik qmzik commented Oct 22, 2017

@honest-hrundel
Copy link

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

Copy link

@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.

🍅


.font-family:checked ~ div
{
font-family: 'Bookman Old Style';

Choose a reason for hiding this comment

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

А что если у пользователя нет этого шрифта?

background: #dadada;
height: 100%;
width: 100%;
}

Choose a reason for hiding this comment

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

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

</li>
<li><h3>Где купить классную вещи и по приятной цене</h3><br>
<span><img src="img/clothes.jpg" alt=""></span>
</li>

Choose a reason for hiding this comment

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

Картинки в правой колонке имеют бОльший оригинальный размер, чем показывается пользователю. Это не хорошо, потому что приходится грузить лишние данные. Картинки должны быть ужаты.

</li>
<li>
<h3>Почему Бузова не может есть стоя</h3><br>
<span><img src="img/buzova.jpg" alt=""></span>

Choose a reason for hiding this comment

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

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


div
{
padding: 10px;

Choose a reason for hiding this comment

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

Это очень странный селектор. Почему у всех div нужно задавать отступ?

</li>
</ul>
</footer>
</div>

Choose a reason for hiding this comment

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

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

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

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

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