-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(plasma-b2c, plasma-web): autoResize for TextArea #614
Conversation
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-614/ |
6196c5b
to
60dc603
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
60dc603
to
5c69b3d
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-614/ |
5c69b3d
to
eaef928
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-614/ |
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-614/ |
c88264d
to
4e6a3be
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
4e6a3be
to
eaef928
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-614/ |
@kayman233 Добавь пожалуйста короткое описание для PR? |
eaef928
to
4886021
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-614/ |
@@ -0,0 +1,31 @@ | |||
import { useEffect, useRef } from 'react'; | |||
|
|||
const ROOT_FONT_SIZE = 16; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
предлагаю экспортнуть эту переменную из TextArea.tsx, чтобы не следить за этими переменными в двух местах
4886021
to
c3c455d
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
const inputRef = useRef<HTMLTextAreaElement | null>(null); | ||
const forkRef = useForkRef(inputRef, outerRef); | ||
|
||
useAutoResize(autoResize, inputRef, value, minAuto, maxAuto); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
а что думаешь на счёт выноса этого хука на уровень выше в TextArea.tsx с прокидыванием всех свойств сюда? Чтобы не дублировать эту логику в оба View, т.к. функционал по сути тот же самый
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
да, ты прав. Теперь в одном месте всё
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-614/ |
c3c455d
to
39f3528
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
39f3528
to
f4a1134
Compare
Theme Builder app deployed! http://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-614/ |
Documentation preview deployed! website: http://plasma.sberdevices.ru/pr/pr-614/ |
🚀 This PR is included in version: @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected], @salutejs/[email protected] 🚀 |
Добавлена возможность автоматического регулирования высоты поля ввода по длине контента внутри (параметра
value
) с помощью свойстваautoResize
.В этом режиме можно указать крайние значения высоты поля ввода, используя свойства
minAuto
,maxAuto
,указав их в
rem
.🐤 Download canary assets:
📦 Published PR as canary version:
Canary Versions
✨ Test out this PR locally via: