Skip to content

Муравьев Артем#73

Open
dreamzoul wants to merge 9 commits intourfu-2017:masterfrom
dreamzoul:master
Open

Муравьев Артем#73
dreamzoul wants to merge 9 commits intourfu-2017:masterfrom
dreamzoul:master

Conversation

@dreamzoul
Copy link
Copy Markdown

@dreamzoul dreamzoul commented Oct 15, 2017

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

Copy link
Copy Markdown
Member

@evilj0e evilj0e left a comment

Choose a reason for hiding this comment

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

🍅

Comment thread tasks/1.html
@@ -1,38 +1,59 @@
<!DOCTYPE html>
<html>
<head>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Верни форматирование во всех файлах

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

Copy link
Copy Markdown
Member

@evilj0e evilj0e left a comment

Choose a reason for hiding this comment

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

🍅

Comment thread tasks/1.html
@@ -1,59 +1,51 @@
<!DOCTYPE html>
<html>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Верни форматирование

@evilj0e evilj0e dismissed their stale review October 24, 2017 06:47

По ошибке посмотрел сабсет

Copy link
Copy Markdown
Member

@evilj0e evilj0e left a comment

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
/* your styles here */
/* your styles here -*/

input[value*='@yandex.ru']
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Есть :valid и :not. Здесь бы они подошли лучше, но ты молодец

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Я с ними попробую реализацию сделать. Спасибо за совет)

Comment thread tasks/15.html Outdated
/* your styles here */
/* your styles here -*/

a[href$='.zip']::after
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Обрати внимание на тег download. Именно он отвечает за скачивание файла.

Comment thread tasks/2.html Outdated
div.c1.c2
{
background-color: #6698ff !important;
border: 2px solid #000;
Copy link
Copy Markdown
Member

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
/* your styles here */
/* your styles here -*/

A[target='_blank']
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

target никакого отношения не имеет является ли ссылка внешней или нет

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

@honest-hrundel
Copy link
Copy Markdown

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

Comment thread tasks/15.html Outdated
/* your styles here -*/

a[href$='.zip']::after
a[href$='.zip']::after,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ты неправильно понял. Ссылка с хреф, заканчивающаяся на зип и с даунлоад.

Comment thread tasks/15.html Outdated
/* your styles here -*/

a[href$='.zip']::after,
[download]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ты неправильно понял. Ссылка с хреф, заканчивающаяся на зип и с даунлоад.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Пытался что-то подобное сделать не вышло. Спасибо за наставление поработаю над этим

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Там кстати действительно в файле был лишний блок. Спасибо что заметил

@honest-hrundel
Copy link
Copy Markdown

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

@evilj0e
Copy link
Copy Markdown
Member

evilj0e commented Nov 10, 2017

Задание 2 и 11 работает не так, как ожидеается

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