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

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

<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

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.

объединить с первым

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

👍

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.

цвета неправильные
можно гораздо проще
обрати внимание на то, чем отличаются в верстке дивы

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

можно гораздо проще, обрати внимание что этот див единственный такого типа в своем родителе

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.

у первого не должно быть бордера

<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 между ними, вдруг мы его уберем или добавим еще один?

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.

должно зависеть от адреса сайта, на вк - синий, на яндекс - красный

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 и тогда одного селектора хватит

<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