feat(input): input password (#UIM-716)

PositiveJS commented 2 years ago

Preview docs changes for 0ada4d8 at https://positive-js.github.io/mosaic-previews/pr821-0ada4d8/

PositiveJS commented 2 years ago

Preview docs changes for 3acbbbd at https://positive-js.github.io/mosaic-previews/pr821-3acbbbd/

PositiveJS commented 2 years ago

Preview docs changes for bdeeef3 at https://positive-js.github.io/mosaic-previews/pr821-bdeeef3/

PositiveJS commented 2 years ago

Preview docs changes for 7b3c1cc at https://positive-js.github.io/mosaic-previews/pr821-7b3c1cc/

PositiveJS commented 2 years ago

Preview docs changes for b664935 at https://positive-js.github.io/mosaic-previews/pr821-b664935/

lskramarov commented 2 years ago

[x] Неправильный стиль кнопки Показать/скрыть пароль (цвета) в светлой теме

[x] Добавить тултип не кнопку Показать/скрыть пароль

[x] в примере. ввел 4 цифры, а система мне уже говорит, что соблюдено требования от 8 до 16 символов

[x] Было бы супер не убирать рамку фокуса у инпута в момент нажатия мышкой на переключатель Показать/скрыть пароль

[ ] Давай сделаем возможность задать разную ширину для пароля и списка требований (например, имеем широкий контейнер, в который ставим короткое поле пароля (фикс. ширина под 8 символов), а подсказки тянутся на всю ширину контейнера)

@rmnturov поправил все кроме последнего пункта, можно смотреть.

Насчет последнего пункта:

Мы везде придерживались правила, что hint не может быть шире самого поля, у нас так в формах и везде на макетах:

по каким причинам здесь вдруг решили изменить подход ?

Если такой функционал действительно нужен, то нужно его описать в гайде и добавить макет с примерами (возможно макет конкретного кейса)

PositiveJS commented 2 years ago

Preview docs changes for 037780d at https://positive-js.github.io/mosaic-previews/pr821-037780d/

rmnturov commented 2 years ago

[x] Неправильный стиль кнопки Показать/скрыть пароль (цвета) в светлой теме

[x] Добавить тултип не кнопку Показать/скрыть пароль

[x] в примере. ввел 4 цифры, а система мне уже говорит, что соблюдено требования от 8 до 16 символов

[x] Было бы супер не убирать рамку фокуса у инпута в момент нажатия мышкой на переключатель Показать/скрыть пароль

[ ] Давай сделаем возможность задать разную ширину для пароля и списка требований (например, имеем широкий контейнер, в который ставим короткое поле пароля (фикс. ширина под 8 символов), а подсказки тянутся на всю ширину контейнера)

@rmnturov поправил все кроме последнего пункта, можно смотреть.

Насчет последнего пункта:

Мы везде придерживались правила, что hint не может быть шире самого поля, у нас так в формах и везде на макетах:

по каким причинам здесь вдруг решили изменить подход ?

Если такой функционал действительно нужен, то нужно его описать в гайде и добавить макет с примерами (возможно макет конкретного кейса)

Спасибо за доработки. Договоренности про ширину, как оказалось, не существует. Возможность сделать поле, примечание, лейбл разной ширины нужна. Сделаем это в отдельной задаче.

rmnturov commented 2 years ago

закрыл по ошибке. открываю

PositiveJS commented 2 years ago

Preview docs changes for 6a0c254 at https://positive-js.github.io/mosaic-previews/pr821-6a0c254/