Алексей Николаев,
Ну да.
1 - Замените ...extends Component... на ...extends PureComponent...
2 - Используюте redux для храниния состояния модальных окон.
3 - Использюте combineReducersдабы постоянный вызов mapStateToPropsне приводил к тормозам.
... Еще куча сомнительных штук вместо какого-нибудь, по непонятным причинам отсутствующего, getComponent('modal').showWithProps(props);
Вобщем пора react-ам, redux-ам и т.д. делать ребрендинг и менять логитип на:
Роман Александрович,
Ну оно только на словах выглядит нормально ибо:
- придумывать ключи для каждого всплывающего окна, так себе идея.
- предположим, что каждый popup знает о состоянии остальных и при открытии закрывает их, вызывая setState открытых popup-ов. И тут у нас появляется Dropdown, который сам по себе должен иметь стейт, так как без него мы не узнаем открывать или закрывать меню (дальше и так понятно что произойдет).
В общем пример бы где-нибудь посмотреть.
Ну да.
1 - Замените
...extends Component...на...extends PureComponent...2 - Используюте redux для храниния состояния модальных окон.
3 - Использюте
combineReducersдабы постоянный вызовmapStateToPropsне приводил к тормозам.... Еще куча сомнительных штук вместо какого-нибудь, по непонятным причинам отсутствующего,
getComponent('modal').showWithProps(props);Вобщем пора react-ам, redux-ам и т.д. делать ребрендинг и менять логитип на: