Skip to content

Зориков Кирилл#110

Open
agrossers wants to merge 48 commits intourfu-2018:masterfrom
agrossers:master
Open

Зориков Кирилл#110
agrossers wants to merge 48 commits intourfu-2018:masterfrom
agrossers:master

Conversation

@agrossers
Copy link
Copy Markdown

@agrossers agrossers commented Oct 18, 2018

@honest-hrundel honest-hrundel changed the title Зориков Кирилл Владимирович Зориков Кирилл Oct 18, 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

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

1 similar comment
@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

1 similar comment
@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

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

1 similar comment
@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

1 similar comment
@honest-hrundel
Copy link
Copy Markdown

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

Comment thread tasks/1.html
<main class="global-results">
<style>
/* your styles here */
article div:first-child
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

можно без first-child

Comment thread tasks/1.html
background: #85f68f;
}

aside
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 tasks/11.html
.global-scope input:focus
{
border: 3px solid blue;
}
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 tasks/12.html
article div:last-child
{
background: orange;
}
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 tasks/13.html
.global-scope div:nth-child(5) div
{
background: #f7549e;
}
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 tasks/4.html
background: yellow;
}

.global-scope article
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 tasks/5.html
<main class="global-results">
<style>
/* your styles here */
input[type='checkbox']:checked + br + div
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

плохо, что зависит от br между ними, вдруг мы его уберем или добавим еще один?

Comment thread tasks/6.html
article a:first-child
{
color: blue;
}
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 tasks/7.html
opacity: .3;
}

.global-scope img[alt^='css']
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

можно просто смотреть на наличие alt и тогда одного селектора хватит

Comment thread tasks/8.html
<main class="global-results">
<style>
/* your styles here */
a[target='_blank']
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

это не показатель того, что внешний по отношению к my-site

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