Dumbs folder seems non idiomatic

PolymerX / polymer-skeleton

:skull: Skeleton for Polymer 3 app with Webpack, PostCSS and Service Workers ready.

https://polymer-skeleton.herokuapp.com

MIT License

177 stars 23 forks source link

Dumbs folder seems non idiomatic #67

Closed richardanaya closed 7 years ago

LasaleFamine commented 7 years ago

Hi @richardanaya, can you please argument a bit your title? I think I'm not getting it 😅

equinusocio commented 7 years ago

Hi @richardanaya, we are take as references how other frameworks handle the components:

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

You’ll find your components much easier to reuse and reason about if you divide them into two categories. I call them Container and Presentational components* but I also heard Fat and Skinny, Smart and Dumb, Stateful and Pure, Screens and Components, etc. These all are not exactly the same, but the core idea is similar.

LasaleFamine commented 7 years ago

Hey @richardanaya, do you think we can close this one?

richardanaya commented 7 years ago

Go for it!

On Nov 5, 2017 5:04 AM, "Alessio Occhipinti" notifications@github.com wrote:

Hey @richardanaya https://github.com/richardanaya, do you think we can close this one?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PolymerX/polymer-skeleton/issues/67#issuecomment-341971713, or mute the thread https://github.com/notifications/unsubscribe-auth/AAR8mgULiOakdHxGSgtfWJMpmATJtkWCks5szbJ4gaJpZM4P_jly .