Skip to content

Денисов Андрей#127

Open
AndyDen98 wants to merge 1 commit intourfu-2018:masterfrom
AndyDen98:master
Open

Денисов Андрей#127
AndyDen98 wants to merge 1 commit intourfu-2018:masterfrom
AndyDen98:master

Conversation

@AndyDen98
Copy link
Copy Markdown

@AndyDen98 AndyDen98 commented Oct 25, 2018

@honest-hrundel honest-hrundel changed the title done styles Денисов Андрей Oct 25, 2018
@honest-hrundel
Copy link
Copy Markdown

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

Comment thread tasks/1.html
background-color: #6698ff;
}

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/4.html
border-top: 1px solid rgb(0, 0, 0);
}

article:nth-child(4),
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Можно проще, используя odd или even

Comment thread tasks/7.html
<main class="global-results">
<style>
/* your styles here */
div:nth-of-type(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.

Прозрачные картинки от непрозрачных отличаются атрибутом alt, можно это использовать

Comment thread tasks/9.html
color: #ffb366;
}

[href^='https']:hover
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

не обязательно тут проверять на https, у псевдоссылок вообще нет атрибута href. Можно использовать например :not для них

Comment thread tasks/12.html
div:nth-of-type(1),
:nth-of-type(3),
:nth-of-type(7),
:nth-of-type(9)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Так работает, но можно было посмотреть внимательней на html ) Оранжевые и желтые квадраты отличаются там

Comment thread tasks/13.html
/* your styles here */
div:nth-child(1) > div:nth-child(1),
div:nth-child(5) > div:nth-child(1),
div:nth-child(9) > div:nth-child(1)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Здесь тоже можно было посмотреть на html, и придумать более простой селектор: div:only-of-type

Comment thread tasks/16.html

#id1
{
background-color: #ffb366 !important;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Зачем везде !important? Его нужно использовать только в крайнем случае, когда других способов нет. Здесь хватило бы одного important в нужном селекторе

Comment thread tasks/18.html
opacity: .5;
}

div:hover
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Работает не совсем так. При наведении на один блок все остальные блоки должны становиться еще более прозрачными

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