common-mapping-client icon indicating copy to clipboard operation
common-mapping-client copied to clipboard

Break all components into HOC wrappers and "simple" React components

Open flynnplatt opened this issue 8 years ago • 0 comments

Inheriting from and sharing many of our components is made difficult by the fact that the default export is the result of the redux connect() function that makes them an HOC. Additionally, it means that to use those components without relying on state requires more or less duplicating the component.

If we instead created standard react components and then created wrapper components that used the connect() HOC, we could have the best of all the worlds. At the cost of inflating the number of files we have significantly.

flynnplatt avatar Mar 16 '18 01:03 flynnplatt