Skip to content

Патанин Максим#84

Open
NorDDay wants to merge 1 commit intourfu-2018:masterfrom
NorDDay:master
Open

Патанин Максим#84
NorDDay wants to merge 1 commit intourfu-2018:masterfrom
NorDDay:master

Conversation

@NorDDay
Copy link
Copy Markdown

@NorDDay NorDDay commented Oct 23, 2018

@honest-hrundel
Copy link
Copy Markdown

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

Copy link
Copy Markdown

@SemyonMakhaev SemyonMakhaev left a comment

Choose a reason for hiding this comment

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

🍅

Привет! Получилось красиво 👍

Давай используем для цитат семантичные теги.
Будет здорово, если реализуешь настройку темы и параметров текста через элементы input

Comment thread index.css
@@ -0,0 +1,150 @@
@import url('https://fonts.googleapis.com/css?family=Diplomata');
@import url('https://fonts.googleapis.com/css?family=Ubuntu');
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Давай подключим хотя бы один шрифт через font-face, как вам показывали на лекции. При этом обрати внимание на инструкцию local. Желательно добавить несколько форматов для обеспечения кроссбраузерности, например, woff и woff2

Comment thread index.css
p:first-letter
{
font-size: 15px;
font-family: 'Diplomata', cursive;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно в одно правило - font

Comment thread index.css
{
font-family: 'Diplomata', cursive;
font-size: 50px;
text-align: center;
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
up one of the more obscure Latin words, consectetur, from a Lorem Ipsum passage, and going
through the cites of the word in classical literature, discovered the undoubtable source.
</p>
<img src="img/cat.jpg" alt="cat">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Для изображений обязательно всегда нужно добавлять атрибут title. Это необходимо для обеспечения доступности

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