-
Notifications
You must be signed in to change notification settings - Fork 86
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
base: master
Are you sure you want to change the base?
Лебедев Никита #31
Conversation
🍏 Пройден линтинг и базовые тесты |
There was a problem hiding this 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'; |
There was a problem hiding this comment.
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%; | ||
} |
There was a problem hiding this comment.
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/clothes.jpg" alt=""></span> | ||
</li> |
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
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; |
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Кажется, логичнее было бы использовать time
Посмотреть решение