Closed dangreen closed 4 years ago
@dangreen тут ветвиться от component/Schedule-container
?
Да
вт, 5 нояб. 2019 г. в 11:10, Rustam notifications@github.com:
@dangreen https://github.com/dangreen тут ветвиться от component/Schedule-container?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/dangreen/DevFest-Siberia/issues/200?email_source=notifications&email_token=AAHQ5BHKARUTZCFLO6HSIPLQSDW2FA5CNFSM4JIXRBM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDBREDA#issuecomment-549655052, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHQ5BECTIQZ5HVZDSH3K4DQSDW2FANCNFSM4JIXRBMQ .
@dangreen есть пару вопросов. 1) description обязательный проп? 2) Должен ли быть ScheduleItem быть обернут в withRouter? Внутри него в ссылках мы будем юзать addSearchparams, значит должен(так как нам нужен доступ к search)? Если да, то локейшн проп надо переименовать иначе ругается в интерфейсе на location так как в RouteComponentProps тоже есть location 3) ScheduleItemModal тоже должен быть подключен к withRouter? Так как внутри надо чекать на серч параметр title? Если есть то актив, нету то скрывать.
@Rustamhabibullin 1) пусть будет нет, если его нет, то и модалку не вызывать 2) Выходит что дожен. Переименуй location в place. 3) Не должен, Item и так подключен, пусть он и управляет состоянием модалки
Сделать подкомпонент ScheduleItemModal + добавить проп description в ScheduleItem. ScheduleModal должна быть внутри ScheduleItem. В ScheduleItem название доклада сделать ссылкой для открытия модалки.
Макет: https://app.zeplin.io/project/5d4aaf915cc9e39b8a55e9d9/screen/5dbdf6ec38b01b6cb14d2386
но не делать блок с инфой о спикере
Ветка:
component/ScheduleItemModal