Skip to content
This repository has been archived by the owner on Sep 11, 2018. It is now read-only.

Container / smart / dumb components #26

Closed
ioRekz opened this issue Aug 17, 2015 · 2 comments
Closed

Container / smart / dumb components #26

ioRekz opened this issue Aug 17, 2015 · 2 comments

Comments

@ioRekz
Copy link

ioRekz commented Aug 17, 2015

The project contains a containers directory. Just to be clear, we speak about that kind of "containers".
https://medium.com/@learnreact/container-components-c0e67432e005

Also https://medium.com/@dan_abramov/smart-and-dumb-components-7ca2f9a7c7d0

If yes, it could make sense to add it to the doc/readme. It would also helps understanding why there are different types of components/directories: views, containers and components.
That would also explains why it is nice to have styles and tests close to the component that is now dumb a easily reusable.

@ioRekz
Copy link
Author

ioRekz commented Aug 18, 2015

I can give it a try in a PR if that sounds good.

@dvdzkwsk
Copy link
Owner

dvdzkwsk commented Nov 9, 2015

Not sure how this got buried, my bad dude :(

The views/layouts/components directories have now been documented. As far as the tests being close to the components, this has now changed so it's no longer valid unfortunately.

@dvdzkwsk dvdzkwsk closed this as completed Nov 9, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants