Skip to content

Гафурова Софья#124

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

Гафурова Софья#124
sofelvpopel wants to merge 1 commit intourfu-2018:masterfrom
sofelvpopel:master

Conversation

@sofelvpopel
Copy link
Copy Markdown

@sofelvpopel sofelvpopel commented Oct 21, 2018

@honest-hrundel honest-hrundel changed the title fhdsjfk Гафурова Софья Oct 21, 2018
@honest-hrundel
Copy link
Copy Markdown

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

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

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
article > input:last-child
{
border: 3px solid #98ff66;
}
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
border: 3px solid #98ff66;
}

article > input:first-child:focus
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
background: #ffb366;
}

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

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

Comment thread tasks/13.html
background: #ff6698;
}

article > div:nth-child(9) > 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.

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

Comment thread tasks/6.html
color: #ff6698;
}

[href='https://market.yandex.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.

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

Comment thread tasks/7.html
article > div:nth-child(3)
{
opacity: .5;
}
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/8.html
[href*='ya.ru']
{
color: #ff6698;
}
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, лучше сделать универсально, проверяя на то, что это не ссылка на my-site

Comment thread tasks/9.html
}

.link:nth-child(2):hover,
.link:nth-child(2):visited: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.

черносерыми должны быть именно псевдоссылки (без урла), а не вторая в списке
а все остальные - цветные

Comment thread tasks/9.html
<main class="global-results">
<style>
/* your styles here */
[href*='ya.ru']: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