Сидоров Артем by art-ylem · Pull Request #57 · urfu-2017/markup-task-2

art-ylem · 2017-10-14T16:55:24Z

Посмотреть решение

honest-hrundel · 2017-10-14T16:55:37Z

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

honest-hrundel · 2017-10-14T17:33:32Z

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

honest-hrundel · 2017-10-15T15:12:07Z

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

savichev-igor · 2017-10-16T04:46:21Z

Сразу же на будущее - называй коммиты так, чтобы они несли хоть какую-то смысловую нагрузку и кратко отражали то, что изменилось

art-ylem · 2017-10-16T04:50:29Z

ок

savichev-igor

🍅

savichev-igor · 2017-10-16T04:52:06Z

+                    background: #ffb366;
+                }

+                .c2 :first-child


Попробуй сделать без first-child

savichev-igor · 2017-10-16T04:55:04Z

+                    border: 2px solid #000;
+                }
+
+                div:empty


Можно сделать вложенность обычную div div

savichev-igor · 2017-10-16T05:14:25Z

            <style>
                /* your styles here */
+
+                .global-scope article:nth-of-type(n+2)


Сложно

Можно решить с использованием двух тэгов

savichev-igor · 2017-10-16T05:23:25Z

            <style>
                /* your styles here */
+
+                #checkbox:checked ~ .tab


Я бы не стал полагаться на дефолтные стили для анчекнутого состояния, вдруг их кто-то переопределит в будущем

savichev-igor · 2017-10-16T05:25:23Z

            <style>
                /* your styles here */
+
+                .global-scope a :first-child


Слишком сложно и работает неправильно

savichev-igor · 2017-10-16T06:12:30Z

            <style>
                /* your styles here */
+
+                input:checked ~ input


Можно решить задание за один селектор

savichev-igor · 2017-10-16T06:12:59Z

+                    opacity: .5;
+                }
+
+                section article div:hover


section можно убрать

savichev-igor · 2017-10-16T06:30:51Z

+                    background: #fff;
+                }
+
+                div:first-child > div:nth-child(4)


А почему не сделать last-child вместо nth-child?

savichev-igor · 2017-10-16T06:31:05Z

+                    background: #6698ff;
+                }
+
+                div:last-child > div:nth-child(1)


Аналогично

savichev-igor · 2017-10-16T06:32:09Z

            <style>
                /* your styles here */
+
+                div:nth-child(3) > div,


Использовал бы класс вместо div

honest-hrundel · 2017-10-16T08:22:33Z

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

honest-hrundel · 2017-10-16T08:23:37Z

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

savichev-igor · 2017-10-16T08:35:14Z

bags fix
bags fix 2

почти также информативно как
коммит 1
коммит 2

art-ylem · 2017-10-16T10:20:57Z

Но там ведь много ошибок исправил. Надо по каждую писать? Или надо было что-то типа "Исправление замечаний от ментора"?

art-ylem · 2017-10-16T10:21:45Z

И что насчет самого исправленного кода, норм?

savichev-igor

Где-то стало лучше 🍅

savichev-igor · 2017-10-16T08:36:24Z

                }

-                input:last-child
+                input[value='email@yandex.ru']


То есть если я введу igor@savichev.me, то это значение будет невалидно?

savichev-igor · 2017-10-16T08:37:02Z

                /* your styles here */

-                a[href$='.zip']::after
+                li:last-child a::after,


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

savichev-igor · 2017-10-16T17:49:44Z

            <section class="global-result">
                <style>
-                    /* your styles here */
+


Лишняя строка

savichev-igor · 2017-10-16T17:50:19Z

@@ -12,6 +12,16 @@ <h1 class="global-heading">&nbsp;</h1>
        <main class="global-results">
            <style>
                /* your styles here */


А почему вверху удалил комментарий, а тут оставил)?

savichev-igor · 2017-10-16T17:52:12Z

            <style>
                /* your styles here */
+
+                input[value='invalid-email']


Не нужно цепляться к значению и хардкодить его

type="email" сразу знает, как выглядит нормальный электронный адрес и здесь нужно использовать valid и invalid

savichev-igor · 2017-10-16T17:52:52Z

+                    background: #ffb366;
+                }
+
+                div:nth-child(even),


Можно сильно проще..

savichev-igor · 2017-10-16T17:53:01Z

            <style>
                /* your styles here */
+
+                li:last-child a::after,


Можно сильно проще..

savichev-igor · 2017-10-16T17:56:08Z

            <style>
                /* your styles here */
+
+                a:first-child


Здесь нужно цепляться к содержимому href

savichev-igor · 2017-10-16T17:56:48Z

            <style>
                /* your styles here */
+
+                a.link[href]


Эффект при нажатии не работает

savichev-igor · 2017-10-16T18:03:50Z

"Исправил замечания" вполне понятный коммит =)

honest-hrundel changed the title ~~try1~~ Сидоров Артем Oct 14, 2017

honest-hrundel assigned savichev-igor Oct 14, 2017

art-ylem added 2 commits October 14, 2017 21:56

try1

2eb20e6

try2

32b6bb6

try3

92751fd

savichev-igor requested changes Oct 16, 2017

View reviewed changes

art-ylem added 2 commits October 16, 2017 13:24

bags fix

1246ccc

bags fix 2

48e3b49

savichev-igor requested changes Oct 16, 2017

View reviewed changes

savichev-igor closed this Mar 11, 2018

honest-hrundel added closed-after-deadline half-accepted labels Mar 11, 2018

savichev-igor reopened this Mar 11, 2018

honest-hrundel removed closed-after-deadline half-accepted labels Mar 11, 2018

Conversation

art-ylem commented Oct 14, 2017 • edited by honest-hrundel Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

honest-hrundel commented Oct 14, 2017

Uh oh!

honest-hrundel commented Oct 14, 2017

Uh oh!

honest-hrundel commented Oct 15, 2017

Uh oh!

savichev-igor commented Oct 16, 2017

Uh oh!

art-ylem commented Oct 16, 2017

Uh oh!

savichev-igor left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

honest-hrundel commented Oct 16, 2017

Uh oh!

honest-hrundel commented Oct 16, 2017

Uh oh!

savichev-igor commented Oct 16, 2017

Uh oh!

art-ylem commented Oct 16, 2017

Uh oh!

art-ylem commented Oct 16, 2017

Uh oh!

savichev-igor left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

savichev-igor Oct 16, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

savichev-igor commented Oct 16, 2017

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

3 participants

art-ylem commented Oct 14, 2017 •

edited by honest-hrundel

Loading

savichev-igor left a comment •

edited

Loading

savichev-igor Oct 16, 2017 •

edited

Loading