Conversation
|
🍏 Пройден линтинг и базовые тесты |
VasiliiKuznecov
left a comment
VasiliiKuznecov
left a comment
There was a problem hiding this comment.
можно еще добавить куда-нибудь вертикальный текст
| display: block; | ||
| } | ||
|
|
||
| .copyright small:nth-child(2) |
There was a problem hiding this comment.
вместо такого селектора (по нему во-первых, не понять сразу к чему применяется стиль, во вторых, при добавлении новых элементов его придется менять) лучше добавь нужному элементу класс и примени стили к нему
| text-align: center; | ||
| } | ||
|
|
||
| .watermark |
There was a problem hiding this comment.
watermark это другое, тут же просто author
| .quote | ||
| { | ||
| margin: 0; | ||
| font-family: 'Pacifico', cursive; |
There was a problem hiding this comment.
попробуй подключить еще один шрифт, но не с гугл-фонтс, а самостоятельно
| { | ||
| margin: 0; | ||
| font-family: 'Pacifico', cursive; | ||
| text-indent: 10px; |
There was a problem hiding this comment.
тоже наверное лучше указывать в em
|
|
||
| .font-size[id='smaller']:checked ~ .newspaper | ||
| { | ||
| font-size: smaller; |
There was a problem hiding this comment.
как мне кажется, лучше использовать конкретные значения вместо ключевых слов
| @@ -1,9 +1,100 @@ | |||
| <!DOCTYPE html> | |||
| <html lang="ru"> | |||
There was a problem hiding this comment.
кажется, страница уже не на русском
| <input class="font-type" type="radio" id="sans-serif" name="font-type"> | ||
| <label for="sans-serif">Sans-serif</label><br> | ||
| <label for="night-mode">Night mode:</label> | ||
| <input class="read-mode" type="checkbox" id="night-mode"> |
There was a problem hiding this comment.
можно все эти настройки объединить в блок с семантичным тегом
| <header class="newspaper-header"> | ||
| <h1 class="newspaper-header-text">The New York Times</h1> | ||
| <hr> | ||
| <div class="copyright three-columns"> |
There was a problem hiding this comment.
кажется, название copyright тут не очен подходит, потому что здесь информация не только про это
| </div> | ||
| </header> | ||
| <hr> | ||
| <section class="newspaper-section three-columns"> |
There was a problem hiding this comment.
кажется, здесь article подошел бы больше, ибо это по сути статья
| src="http://www.ibiblio.org/eldritch/hmt/myfps2.gif" | ||
| alt="Henry Major Tomlinson" | ||
| width="307" height="369"> | ||
| <blockquote class="quote">Donec |
There was a problem hiding this comment.
переносы у blockquote в другом стиле, нежели чем у p
|
🍅 |
3 participants
Посмотреть решение