Skip to content

Шумейко Евгений#106

Open
EvgeniiShumeiko wants to merge 4 commits intourfu-2018:masterfrom
EvgeniiShumeiko:master
Open

Шумейко Евгений#106
EvgeniiShumeiko wants to merge 4 commits intourfu-2018:masterfrom
EvgeniiShumeiko:master

Conversation

@EvgeniiShumeiko
Copy link
Copy Markdown

@EvgeniiShumeiko EvgeniiShumeiko commented Oct 23, 2018

@honest-hrundel honest-hrundel changed the title Евгений Шумейко Шумейко Евгений 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

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

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
border-bottom: 1px solid #000;
}

div.title
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
{
width: 130px;
height: 90px;
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.css

.big-col
{
width: calc((100% / 3) * 2);
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-свойств обращай внимание на поддержку браузерами. На правах рекламы могу порекомендовать ресурс https://caniuse.com

Comment thread index.css
word-break: normal;
text-align: justify;
font-family: Arial, sans-serif;
font-size: 14px;
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: Arial, sans-serif;
font-style: italic;
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
</div>
<div class="main-title">
<img src="https://upload.wikimedia.org/wikipedia/commons/5/58/NewYorkTimes.svg"
title="The New York Times" alt="The New York Times">
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
VOL. CLXVIII . . . No. 58,124
</span>
<span class="company">
© 2018 The New York Times Company
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;

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