feat: [EV-8779] add alwaysShowMask to InputMask - #104
Conversation
| // @ts-ignore | ||
| mask={mask} | ||
| maskChar={maskChar} | ||
| alwaysShowMask={alwaysShowMask} |
There was a problem hiding this comment.
не хватает примера в сторибуке
| className, | ||
| mask, | ||
| maskChar, | ||
| alwaysShowMask, |
There was a problem hiding this comment.
надо указать дефолтное значение, а также у нас есть TextFieldV2, там тоже надо
There was a problem hiding this comment.
в компоненте TextFieldV2 не используется InputMask и не передаются никакие пропсы для маски, добавить все пропсы для маски и InputMask?
There was a problem hiding this comment.
добавила и для TextFieldV2 + поправила стили
| variant, | ||
| isFocused, | ||
| value, | ||
| isLabelFloated, |
There was a problem hiding this comment.
а чем отличается value от isLabelFloated?
There was a problem hiding this comment.
в хелпере потом value использовалось вместе с isFocused isFocused || !!value для определения стилей, к этому условию добавилось const isLabelFloated = isFocused || !!inputProps.value || alwaysShowMask; для отображения состояния поднятого лейбла и это условие вынесла в сам компонент чтобы было единообразно с компонентом TextField
3 participants
No description provided.