Hacker News new | ask | show | jobs
by STRML 3994 days ago
We have a few rather complex mixins in React-Router-Component and react-mixin's decorators worked just fine [1]. The only limitation I see now is a mixin that defines more mixins; this currently doesn't work, but there's no reason why it couldn't. I am considering a patch.

1. https://github.com/STRML/react-router-component/blob/master/...
1 comments
egeozcan 3994 days ago
Thank you for the heads up and the example.

> The only limitation I see now is a mixin that defines more mixins; this currently doesn't work

Why and how would you do this?

link
STRML 3994 days ago
It's just a nice way to split up concerns. In React-Router-Component, the RouterMixin [1] mixes in the Environment Mixin [2] which allows the Environment to address its own concerns in a separate file and to be included in other, custom components.

1. https://github.com/STRML/react-router-component/blob/master/... 2. https://github.com/STRML/react-router-component/blob/master/...

link
egeozcan 3994 days ago
Ah, now I get it, you meant nested mixins. Great, thanks for the pull, hope it gets merged.
link
STRML 3994 days ago
I've submitted a fix for this in react-mixin here. https://github.com/brigand/react-mixin/pull/22
link