Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(Modal.Header, Modal.Footer): passing common props #3187

Merged
merged 1 commit into from
Jun 23, 2023

Conversation

JackUait
Copy link
Contributor

@JackUait JackUait commented Jun 16, 2023

Проблема

Перестали работать пропы передаваемые из <CommonWrapper /> на корень <Modal.Header />/<Modal.Footer />, из-за того, что корень это React.Fragment

Решение

Заменил React.Fragment на <div />

Альтернативное решение: прокидывать пропы на элементы с data-tid *.root ручками. Таким образом мы на 100% сохраним обратную совместимость и решим проблему без создания "лишних" обёрток. Не уверен, что в текущих реалиях такой способ правильный, но если мы по-прежнему планируем отказаться от <CommonWrapper /> я бы пошёл именно этим путём

Ссылки

IF-1271

Чек-лист перед запросом ревью

  1. Добавлены тесты на все изменения
    ✅ unit-тесты для логики
    ⬜ скриншоты для верстки и кросс-браузерности
    ⬜ нерелевантно

  2. Добавлена (обновлена) документация
    ⬜ styleguidist для пропов и примеров использования компонентов
    ⬜ jsdoc для утилит и хелперов
    ⬜ комментарии для неочевидных мест в коде
    ⬜ прочие инструкции (README.md, contributing.md и др.)
    ✅ нерелевантно

  3. Изменения корректно типизированы
    ⬜ без использования any (см. PR 2856)
    ✅ нерелевантно

  4. Прочее
    ✅ все тесты и линтеры на CI проходят
    ✅ в коде нет лишних изменений
    ✅ заголовок PR кратко и доступно отражает суть изменений (он попадет в changelog)

@JackUait JackUait changed the base branch from next to master June 16, 2023 10:59
@JackUait JackUait changed the title fix(theme): add missing units to values (#3184) fix(Modal.Header, Modal.Footer): passing common props Jun 16, 2023
@JackUait JackUait requested a review from HelenaIsh June 20, 2023 15:23
@zhzz zhzz merged commit 4aa327b into master Jun 23, 2023
6 checks passed
@zhzz zhzz deleted the fix/pass-prop-style-modal branch June 23, 2023 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants