Conversation
Ramazanmak
force-pushed
the
feat/states
branch
from
99d4bb2 to
5116aa7
Compare
There was a problem hiding this comment.
На первом шаге слишком много стилей. Должен быть минимум - смотри предыдущую демку
There was a problem hiding this comment.
Я намеренно оставил столько стилей в начале. Подумал, что в первом уроке какие-то основы простого синтаксиса усвоили, а в этом уроке нужно сосредоточиться именно на states. А если минимум стилей оставить, то придётся объяснять, как делать простую вёрстку => сдвинется акцент урока.
There was a problem hiding this comment.
Как раз наоборот: надо, чтобы немного повторили базу. Совсем голое делать не нужно, но можно так, чтобы элементы со states надо было сначала чуть стилизовать. Надо так рассчитать, что шагов на 10 набралось материала
states/0/index.html
Outdated
There was a problem hiding this comment.
Зачем -ShortTsd200ms? Используй дефолтный
states/0/index.html
Outdated
There was a problem hiding this comment.
Вернее, раз он тут чисто одноразовый для подложки, можно использовать дефолтный цвет из браузера. Там же их много, типа grey и тд. Просто hex в утилитах указывать - особый случай, который в образцовой демке не стоит без надобности юзать
states/0/index.html
Outdated
There was a problem hiding this comment.
Flg0.9 - это что-то странное. Нельзя проще?
There was a problem hiding this comment.
Fns5.5u - зачем дробное? У нас же не pixel perfect верстка
states/0/index.html
Outdated
There was a problem hiding this comment.
Это для того, чтобы слово "Save" было в кнопке по центру, а не чуть ниже центра. Там просто по умолчанию высота строки чуть больше, чем размер шрифта, поэтому сам находится в середине по вертикали, а вот текст прижат к низу этого блока, поэтому в итоге текст чуть ниже середины.
There was a problem hiding this comment.
Убрал Lnh100p в девтулзах обоих браузеров - ничего не поменялось
There was a problem hiding this comment.
Вот с Lnh100p:
А вот без него:
Мне кажется, что разница есть - без него надпись ниже)
states/0/index.html
Outdated
There was a problem hiding this comment.
Если input внутри label, то связывать через for не надо
states/0/index.html
Outdated
There was a problem hiding this comment.
Если placeholder пустой, то и указывать его не надо
states/0/index.html
Outdated
There was a problem hiding this comment.
Ol-n раньше времени. Надо убирать только при назначении states
states/-1/index.html
Outdated
There was a problem hiding this comment.
На названия тегов завязываться не надо. В крайнем случае, допустимо на input для css only интерактивности
states/-1/index.html
Outdated
There was a problem hiding this comment.
Ps-r - можно использовать дефолтное значение утилиты, оно такое же
states/-1/index.html
Outdated
There was a problem hiding this comment.
Bd1;s;$accent600 - вижу, что меняется только цвет. Есть же Bdc
states/0/index.html
Outdated
There was a problem hiding this comment.
hover - не самое подходящее название для цвета
states/0/index.html
Outdated
There was a problem hiding this comment.
Чтобы каждый раз не писать у svg Fi-cc, надо вставить в самом спрайте у иконок fill="currentColor"
There was a problem hiding this comment.
И зачем P0.5u, если padding у родителя есть?
states/0/index.html
Outdated
There was a problem hiding this comment.
Bgc#000*0.5 - прозрачность лучше через 50p указывать
2 participants
No description provided.