Skip to content

Стольников Константин#101

Open
Gyke wants to merge 5 commits intourfu-2018:masterfrom
Gyke:master
Open

Стольников Константин#101
Gyke wants to merge 5 commits intourfu-2018:masterfrom
Gyke:master

Conversation

@Gyke
Copy link
Copy Markdown

@Gyke Gyke commented Oct 18, 2018

Первый коммит, первые тесты
Посмотреть решение

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

jdsfgsd
@honest-hrundel
Copy link
Copy Markdown

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

some changes with whortin words
@honest-hrundel
Copy link
Copy Markdown

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

Copy link
Copy Markdown

@SemyonMakhaev SemyonMakhaev left a comment

Choose a reason for hiding this comment

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

🍅

Привет! Давай будем следить за тем, чтобы инструкция !important использовалась только в крайних случаях 😅

В 20-ом задании попробуй подумать на тему псевдокласса :nth-last-child


input:focus
{
border: 3px solid #6698ff;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

В двух селекторах с псевдоклассами полностью переопределяется правило, задающее стили для границы в селекторе тега input, хотя разница только в цвете этой границы. Можно в селекторах с псевдоклассами :first-child и :focus переназначать только border-color

background-color: #ff6;
}

div:nth-child(4n-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.

Попробуй использовать псевдокласс :empty


div:first-child
{
background-color: #ffb366;
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 */
div:nth-child(4n + 1) > 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.

Обрати внимание на псевдокласс only-of-type


ul
{
list-style-type: square !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 здесь? Подсказка: можно стилизовать псевдоэлементы, например, before

<main class="global-results">
<style>
/* your styles here */
input#checkbox:checked ~ 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.

Зачем столько селекторов? Не достаточно ли указать только идентификатор или только тег?


article > div:last-child
{
opacity: 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.

Здесь можно просты выбрать все картинки, у которых нет атрибута alt

<main class="global-results">
<style>
/* your styles here */
[href*='//']
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.ru

color: #ff6698;
}

[href*='my-site.ru']
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 давай не будем переназначать стили

/* your styles here */
[href]:visited
{
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)
Повысить специфичность можно, указав в селекторе помимо атрибута ещё и класс

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