Skip to content

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

Merged
merged 20 commits into from
Mar 22, 2022
Merged

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

merged 20 commits into from
Mar 22, 2022

Conversation

lskramarov
Copy link
Contributor

No description provided.

@PositiveJS
Copy link
Contributor

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

@PositiveJS
Copy link
Contributor

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

@PositiveJS
Copy link
Contributor

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

@PositiveJS
Copy link
Contributor

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

@PositiveJS
Copy link
Contributor

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

@lskramarov lskramarov changed the title feat(input): input password (#UIM-716) WIP feat(input): input password (#UIM-716) Mar 15, 2022
Copy link

@rmnturov rmnturov left a comment

Choose a reason for hiding this comment

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

  • Неправильный стиль кнопки Показать/скрыть пароль (цвета) в светлой теме
  • Добавить тултип не кнопку Показать/скрыть пароль
  • в примере. ввел 4 цифры, а система мне уже говорит, что соблюдено требования от 8 до 16 символов
  • Было бы супер не убирать рамку фокуса у инпута в момент нажатия мышкой на переключатель Показать/скрыть пароль
  • Давай сделаем возможность задать разную ширину для пароля и списка требований (например, имеем широкий контейнер, в который ставим короткое поле пароля (фикс. ширина под 8 символов), а подсказки тянутся на всю ширину контейнера)

@lskramarov
Copy link
Contributor Author

  • Неправильный стиль кнопки Показать/скрыть пароль (цвета) в светлой теме
  • Добавить тултип не кнопку Показать/скрыть пароль
  • в примере. ввел 4 цифры, а система мне уже говорит, что соблюдено требования от 8 до 16 символов
  • Было бы супер не убирать рамку фокуса у инпута в момент нажатия мышкой на переключатель Показать/скрыть пароль
  • Давай сделаем возможность задать разную ширину для пароля и списка требований (например, имеем широкий контейнер, в который ставим короткое поле пароля (фикс. ширина под 8 символов), а подсказки тянутся на всю ширину контейнера)

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

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

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

image

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

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

@PositiveJS
Copy link
Contributor

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

@rmnturov
Copy link

  • Неправильный стиль кнопки Показать/скрыть пароль (цвета) в светлой теме
  • Добавить тултип не кнопку Показать/скрыть пароль
  • в примере. ввел 4 цифры, а система мне уже говорит, что соблюдено требования от 8 до 16 символов
  • Было бы супер не убирать рамку фокуса у инпута в момент нажатия мышкой на переключатель Показать/скрыть пароль
  • Давай сделаем возможность задать разную ширину для пароля и списка требований (например, имеем широкий контейнер, в который ставим короткое поле пароля (фикс. ширина под 8 символов), а подсказки тянутся на всю ширину контейнера)

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

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

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

image

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

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

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

@rmnturov rmnturov closed this Mar 22, 2022
Copy link

@rmnturov rmnturov left a comment

Choose a reason for hiding this comment

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

всё круто

@rmnturov rmnturov reopened this Mar 22, 2022
@rmnturov
Copy link

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

@PositiveJS
Copy link
Contributor

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

@sonarcloud
Copy link

sonarcloud bot commented Mar 22, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot E 3 Security Hotspots
Code Smell A 26 Code Smells

0.0% 0.0% Coverage
12.5% 12.5% Duplication

@PositiveJS
Copy link
Contributor

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

@lskramarov lskramarov merged commit 694fb6f into master Mar 22, 2022
@lskramarov lskramarov deleted the feature/UIM-716 branch March 22, 2022 11:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants