Skip to content

Refactor/upload style contract#712

Merged
TorinAsakura merged 6 commits into
masterfrom
refactor/upload-style-contract
Jun 19, 2026
Merged

Refactor/upload style contract#712
TorinAsakura merged 6 commits into
masterfrom
refactor/upload-style-contract

Conversation

@comfrt1k

@comfrt1k comfrt1k commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Таска

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

  1. Контекст: открыт Admin/Upload в Storybook
    Действие: открыть story Базовый и включить control customPlaceholder
    Ожидаемый результат: placeholder отображается с кастомными стилями root, icon, accept и link

  2. Контекст: используется DropzonePlaceholder из @atls-ui-admin/upload
    Действие: передать className в DropzonePlaceholder
    Ожидаемый результат: пользовательский класс применяется к root-элементу placeholder

  3. Контекст: используется DropzonePlaceholder из @atls-ui-admin/upload
    Действие: передать slotProps.icon.className, slotProps.accept.className и slotProps.link.className
    Ожидаемый результат: стили применяются к icon, accept и link без замены placeholder-разметки

  4. Контекст: пакет @atls-ui-admin/upload
    Действие: выполнить yarn workspace @atls-ui-admin/upload build
    Ожидаемый результат: сборка проходит без ошибок

Пруфы

Details
2026-06-16.23.45.42.mov
image

@comfrt1k comfrt1k self-assigned this Jun 16, 2026

@TorinAsakura TorinAsakura left a comment

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.

В PR body и #691 заявлена проверка story Кастомный placeholder, где должны быть видны root/icon/accept/link стили. В финальном диффе эта story удалена, а оставшийся BaseUpload не передаёт ни className, ни slotProps, поэтому Как проверять и Пруфы не подтверждают заявленный контракт. Нужно вернуть story/пруф для кастомизации placeholder или привести body к фактическому объёму PR

@comfrt1k

comfrt1k commented Jun 18, 2026

Copy link
Copy Markdown
Contributor Author

В PR body и #691 заявлена проверка story Кастомный placeholder, где должны быть видны root/icon/accept/link стили. В финальном диффе эта story удалена, а оставшийся BaseUpload не передаёт ни className, ни slotProps, поэтому Как проверять и Пруфы не подтверждают заявленный контракт. Нужно вернуть story/пруф для кастомизации placeholder или привести body к фактическому объёму PR

45145cd

пруф добавила тоже

@comfrt1k comfrt1k requested a review from TorinAsakura June 18, 2026 17:21
@TorinAsakura TorinAsakura merged commit f62b48b into master Jun 19, 2026
6 checks passed
@TorinAsakura TorinAsakura deleted the refactor/upload-style-contract branch June 19, 2026 00:03
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] Прокидывание пользовательских стилей в Upload placeholder без враппера

2 participants