Skip to content

Стихин Семен#77

Open
samstikhin wants to merge 17 commits into
urfu-2017:masterfrom
samstikhin:master
Open

Стихин Семен#77
samstikhin wants to merge 17 commits into
urfu-2017:masterfrom
samstikhin:master

Conversation

@samstikhin
Copy link
Copy Markdown

@samstikhin samstikhin commented Oct 24, 2017

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@samstikhin
Copy link
Copy Markdown
Author

Знаю, что картинок нет, они пока сильно плывут при изменении окна.

@VasiliiKuznecov
Copy link
Copy Markdown

Привет! Твой ментор пока в отпуске, поэтому сейчас переназначу на другого
В следующий раз можешь попросить, чтобы тебе назначили обратно старого)

@VasiliiKuznecov
Copy link
Copy Markdown

🔔

Copy link
Copy Markdown

@trixartem trixartem left a comment

Choose a reason for hiding this comment

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

Замечания:

  • Нет цитат
  • Маловато картинок с подписями
  • Первая буква не выделена
  • посмотри задание внимательно
    🍅

Comment thread index.css Outdated
@font-face
{
font-family: machina;
src: url('fonts/machina.ttf');
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 Outdated

header
{
font: 70pt machina;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

pt? Почему? так же не хвататет фолбэка для шрифта

Comment thread index.css
main
{
margin-left: 60px;
font: 1em zelek;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Где-то em где-то pt, приведи к одному виду

Comment thread index.css Outdated

h1
{
margin-left: 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.

От чего считается процент? точно он здесь нужен?

Comment thread index.css
.block2
{
font-size: 1em;
border: 7px dotted #000;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Если делаешь разный бордер, то сделай что бы на стыке блоков был только один бордер, а не как сейчас 2(не очень красиво)

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@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

@trixartem trixartem left a comment

Choose a reason for hiding this comment

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

Есть пара комментариев 🍅

Comment thread index.css
border: 2px dashed #000;
}

.block2
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

.c2:checked ~ main
{
font-family: Arial;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Нужен фолбэк для шрифта

@trixartem
Copy link
Copy Markdown

@samstikhin Сделай, пожалуйста, правки

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.

5 participants