Skip to content

Refactor button contract#685

Merged
TorinAsakura merged 3 commits into
masterfrom
refactor/button-contract
Jun 1, 2026
Merged

Refactor button contract#685
TorinAsakura merged 3 commits into
masterfrom
refactor/button-contract

Conversation

@comfrt1k
Copy link
Copy Markdown
Contributor

@comfrt1k comfrt1k commented May 29, 2026

Таска

Как проверять

  1. Контекст: Storybook ui-parts/design
    Действие: открыть Components/Button, переключить appearance между blue, lightBlue, ghost и shape между доступными вариантами
    Ожидаемый результат: кнопка меняет внешний вид и размер без wrapper-компонента, состояния hover/pressed/disabled работают корректно

  2. Контекст: Storybook ui-admin/design
    Действие: открыть Admin/Button и проверить переключение appearance и shape
    Ожидаемый результат: admin-кнопка использует тот же контракт appearance/shape и рендерится без локальной реализации стилей

  3. Контекст: компонент Button в коде
    Действие: передать в Button собственный className вместе с appearance и shape
    Ожидаемый результат: className попадает на корневой button и мержится с внутренними классами, не ломая базовые стили и состояния

Пруфы

Details
2026-05-29.14.28.40.mov

@comfrt1k comfrt1k self-assigned this May 29, 2026
@comfrt1k comfrt1k requested a review from TorinAsakura May 29, 2026 11:33
Copy link
Copy Markdown
Member

@TorinAsakura TorinAsakura left a comment

Choose a reason for hiding this comment

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

PR связан с #684, но описание и проверка покрывают только Button. В задаче отдельным критерием указано, что остальные компоненты с тем же блокером по стилям должны быть проверены и приведены к этому подходу либо явно оставлены без изменения с причиной. Нужно синхронизировать scope PR с задачей: либо закрыть этот критерий в реализации и доказательствах, либо явно оформить PR как частичный и не привязывать его к полной задаче как источник готовности.

@comfrt1k
Copy link
Copy Markdown
Contributor Author

PR связан с #684, но описание и проверка покрывают только Button. В задаче отдельным критерием указано, что остальные компоненты с тем же блокером по стилям должны быть проверены и приведены к этому подходу либо явно оставлены без изменения с причиной. Нужно синхронизировать scope PR с задачей: либо закрыть этот критерий в реализации и доказательствах, либо явно оформить PR как частичный и не привязывать его к полной задаче как источник готовности.

в ПРе не Close, а Ref

@comfrt1k
Copy link
Copy Markdown
Contributor Author

PR связан с #684, но описание и проверка покрывают только Button. В задаче отдельным критерием указано, что остальные компоненты с тем же блокером по стилям должны быть проверены и приведены к этому подходу либо явно оставлены без изменения с причиной. Нужно синхронизировать scope PR с задачей: либо закрыть этот критерий в реализации и доказательствах, либо явно оформить PR как частичный и не привязывать его к полной задаче как источник готовности.

поправила

Comment thread ui-admin/button/src/button.component.tsx
@comfrt1k comfrt1k requested a review from TorinAsakura June 1, 2026 15:26
@TorinAsakura TorinAsakura merged commit d9da81c into master Jun 1, 2026
6 checks passed
@TorinAsakura TorinAsakura deleted the refactor/button-contract branch June 1, 2026 15:36
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.

[Feature] Прокидывание пользовательских стилей в Button без враппера

2 participants