Size: a a a

Atomic Design for React

2020 July 31

BA

Bogdan Aleksandrovic... in Atomic Design for React
Dmitry Zherebko
Тебе прийдется в тесте с модалками постоянно резетить стейт открытых модалок
там одна строчка, камон
источник

DZ

Dmitry Zherebko in Atomic Design for React
беда в том что тебе про это надо помнить
источник

BA

Bogdan Aleksandrovic... in Atomic Design for React
https://codesandbox.io/s/react-modal-effector-x9xkf?file=/src/lib/modal.js


вот я пробовал как он хочет, ну вполне нормально, id только нужно или типизировать или следить как то
источник

OR

Oleg Rizhkov in Atomic Design for React
что запомнить? максимум "тест упал, ой, hide()"
источник

Е

Едуард in Atomic Design for React
Dmitry Zherebko
Как ты это гарантируешь?
Symbol для ключа
источник

DZ

Dmitry Zherebko in Atomic Design for React
Едуард
Symbol для ключа
тогда надо ещё файл с константами модалок
источник

🦜

🦜 in Atomic Design for React
Едуард
обьясни
* Модалки не про запихивание целых страниц с логикой
* Доступность
* Нельзя открыть по ссылке
* Как много страниц может дать линк на эту модалку
* Неудобно на мобильном дейвайсе
* Страница дешевле по коду, чем обслуживание модалки
* Без JS ты никак не войдешь
* Ты хочешь чтобы юзер зарегался?
* Конверсия
*
источник

Е

Едуард in Atomic Design for React
Dmitry Zherebko
тогда надо ещё файл с константами модалок
и что, это проблема ?
источник

BA

Bogdan Aleksandrovic... in Atomic Design for React
Oleg Rizhkov
что запомнить? максимум "тест упал, ой, hide()"
10 модалок - 10 id , запомнить, что бы не пересекались , если не ты писал, проверить, что такого id нет
источник

DZ

Dmitry Zherebko in Atomic Design for React
сова и я предлали когда-то намного лучше апи
источник

DZ

Dmitry Zherebko in Atomic Design for React
без всех этих проблем
источник

DZ

Dmitry Zherebko in Atomic Design for React
Переслано от Dmitry Zherebko
источник

DZ

Dmitry Zherebko in Atomic Design for React
const [Modal, { open, close }] = useModal()
источник

DZ

Dmitry Zherebko in Atomic Design for React
<Modal onOpen={ƒ}>Content</Modal>
источник

Е

Едуард in Atomic Design for React
🦜
* Модалки не про запихивание целых страниц с логикой
* Доступность
* Нельзя открыть по ссылке
* Как много страниц может дать линк на эту модалку
* Неудобно на мобильном дейвайсе
* Страница дешевле по коду, чем обслуживание модалки
* Без JS ты никак не войдешь
* Ты хочешь чтобы юзер зарегался?
* Конверсия
*
спасибо, я соглашусь с этим)
источник

BA

Bogdan Aleksandrovic... in Atomic Design for React
Dmitry Zherebko
Переслано от Dmitry Zherebko
и как передать в модалку ребенка другого
источник

DZ

Dmitry Zherebko in Atomic Design for React
Переслано от Dmitry Zherebko
сама имплементация там тривиальная
источник

DZ

Dmitry Zherebko in Atomic Design for React
Bogdan Aleksandrovich
и как передать в модалку ребенка другого
это специфичная модалка под конкретный кейс
источник

DZ

Dmitry Zherebko in Atomic Design for React
Dmitry Zherebko
Переслано от Dmitry Zherebko
сама имплементация там тривиальная
вот базовый вариант
источник

BA

Bogdan Aleksandrovic... in Atomic Design for React
сделать одну модалку обертку, передавать ей ребенка, все, вообще гемороя нет, только за id  следить
источник