Skip to content

Потапов Лев#101

Open
malaev wants to merge 5 commits intourfu-2018:masterfrom
malaev:master
Open

Потапов Лев#101
malaev wants to merge 5 commits intourfu-2018:masterfrom
malaev:master

Conversation

@malaev
Copy link
Copy Markdown

@malaev malaev commented Oct 23, 2018

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

🍅 Не пройден линтинг или базовые тесты

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

Comment thread index.css

h4,
h5,

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.css
{
font-size: 40px;
text-align: center;
font-family: 'Old English Text MT';
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Надо установить фолбек-шрифт на случай, если такого шрифта не будет или он не загрузится.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

А еще надо добавить local(...) для каждого шрифта.

Comment thread index.css
left: 70px;
}

section
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Давай не будем использовать теги в css, а повесим на них классы.

Comment thread index.css
.mainNewImage
{
display: inline-block;
filter: grayscale(100%);
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
<hr class="textHr">
</section>
<section class="mainNewImage">
<img alt="mainImage"
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