Skip to content

Textarea component - #180

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

tolms wants to merge 9 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)

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

Comment thread components/textarea/src/TextArea.tsx Outdated
Comment thread components/textarea/src/TextArea.types.ts Outdated
@sadcitizen

Copy link
Copy Markdown
Member

Скормил код GPT-5.6 Terra. Вот её ответ:

Проверил ветку [feature/textarea](https://github.com/Byndyusoft/ui/tree/feature/textarea) пакета @byndyusoft-ui/textarea. Замечания:

Существующие тесты пакета проходят: 12 из 12. Они не покрывают перечисленные сценарии. Исходный проект не изменял.

@tolms

tolms commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author
  1. Решено
  2. Решено
  3. Решено. Сделал ESM-only. Для Vite SSR и Next.js это подходит
  4. Решено
  5. Решено
  6. Решено

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