Skip to content

Textarea component - #180

Open
tolms wants to merge 2 commits into
masterfrom
feature/textarea
Open

Textarea component#180
tolms wants to merge 2 commits into
masterfrom
feature/textarea

Conversation

@tolms

@tolms tolms commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Суть изменений

Кратко, что изменено и почему:

  • Example: "Fix: align icon in Button for mobile"

Чек-лист перед созданием PR (обязательно)

  • Код написан в соответствии с гайдлайнами (дока в CODE_OF_CONDUCT.md) (npm run lint проходит)
  • Все новые и существующие тесты проходят (npm run test)
  • Локально прошёл билд изменённых пакетов (npm run build)
  • Создан changeset для всех изменённых пакетов (npm run set-changes)
    • При создании changeset выбери изменённые пакеты, укажи тип изменения (patch/minor/major) и кратко опиши, что поменялось
  • Обновлён README/Storybook docs/др. документация локальная/глобальная, если API/публичное поведение изменилось
  • Обновлена/добавлена типизация (если применимо)
  • Экраны добавлены/обновлены в storybook

Что было сделано (описание)

Опиши ключевые изменения, алгоритм, влияние на API и migration notes (если нужно).

Как тестировать

  1. Как запустить сборку/тесты.
  2. Примеры использования (snippets).
  3. Особые шаги (например, сброс кэша).

Скриншоты / GIF (если с UI)

Прикрепи изображения/анимацию

return (
<textarea
{...rest}
className={cn(withAutoHeight && styles.withAutoHeight, className)}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Надо как-то избавиться от стилей в пакете, чтобы пакет отдавал чистый ts

isDisabled?: boolean;
minHeight?: number;
withAutoHeight?: boolean;
onChange?: ChangeEventHandler<HTMLTextAreaElement>;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

onChange чем отличается от нативного?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants