Conversation
this
|
🍏 Пройден линтинг и базовые тесты |
SemyonMakhaev
left a comment
SemyonMakhaev
left a comment
There was a problem hiding this comment.
🍅
Привет! Молодец, получилось красиво 👍
А как же вертикальный текст?
А как же картинки?
А цитаты? Для них нужно использовать семантичный тег.
Будет круто, если добавишь настройку темы и параметров текста через элементы input
| @font-face | ||
| { | ||
| font-family: 'Agincourt'; | ||
| src: url('./AgincourtLetPlain.ttf') format('truetype'); |
There was a problem hiding this comment.
Давай добавим также local, чтобы не загружать шрифт, если он есть на компьютере пользователя
| @font-face | ||
| { | ||
| font-family: 'Agincourt'; | ||
| src: url('./AgincourtLetPlain.ttf') format('truetype'); |
There was a problem hiding this comment.
Будет здорово, если добавишь ещё пару форматов шрифта для обеспечения кроссбраузерности. Например, woff и woff2
| .edition, | ||
| .slogan | ||
| { | ||
| font-family: Arial, serif; |
There was a problem hiding this comment.
Это основной шрифт на странице? Лучше его подключить к body, для других элементов он унаследуется. При этом можно записать в одно правило font
| .article-item span | ||
| { | ||
| font-size: 12px; | ||
| font-family: Arial, sans-serif; |
4 participants
this
Посмотреть решение