Skip to content

Refactor/upload style contract#712

Open
comfrt1k wants to merge 5 commits into
masterfrom
refactor/upload-style-contract
Open

Refactor/upload style contract#712
comfrt1k wants to merge 5 commits into
masterfrom
refactor/upload-style-contract

Conversation

@comfrt1k

Copy link
Copy Markdown
Contributor

Таска

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

  1. Контекст: открыт Admin/Upload в Storybook
    Действие: выбрать story Кастомный placeholder
    Ожидаемый результат: 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

@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

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