Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

AndyDen98
Copy link

@AndyDen98 AndyDen98 commented Oct 25, 2018

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

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

background-color: #6698ff;
}

aside

Choose a reason for hiding this comment

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

Можно объединить в один селектор с предыдущим

border-top: 1px solid rgb(0, 0, 0);
}

article:nth-child(4),

Choose a reason for hiding this comment

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

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

@@ -12,7 +12,11 @@ <h1 class="global-heading">&nbsp;</h1>

<main class="global-results">
<style>
/* your styles here */
div:nth-of-type(2),

Choose a reason for hiding this comment

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

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

color: #ffb366;
}

[href^='https']:hover

Choose a reason for hiding this comment

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

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

div:nth-of-type(1),
:nth-of-type(3),
:nth-of-type(7),
:nth-of-type(9)

Choose a reason for hiding this comment

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

Так работает, но можно было посмотреть внимательней на 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)

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


#id1
{
background-color: #ffb366 !important;

Choose a reason for hiding this comment

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

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

opacity: .5;
}

div:hover

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