Failing tests for auto-binded class methods defined via arrow functions.

[nikolaymatrosov]

I scetched some tests that follow pattern that I use to define auto-binded method in ES6 classes. I tried to highlight the problem I have encountered using previous version of React Hot Loader. The tests could be wrong, I just wanted to add some illustration to my feature request. It would be awesome if the new version of Hot Reloader could handle this case.

[gaearon]

👍

These are currently really problematic (I don't know how to handle them).
My best bet is to create a separate Babel plugin that will help React Proxy realize what's going on there.

[gaearon]

By the way (don't mean to be pedantic) this is class property syntax proposed for ES7.
It's not part of ES6.

[nikolaymatrosov]

I know. But it compiles with --stage 0, so it is good enough for me;)

On Saturday, August 22, 2015, Dan Abramov notifications@github.com wrote:

By the way (don't mean to be pedantic) this is class property syntax
proposed for ES7. It's not part of ES6.

—
Reply to this email directly or view it on GitHub
#8 (comment).

[jbrantly]

I ran into this as well and was going to make a PR with failing tests but you did the work for me. Thanks! Hope something can be figured out for these since they're so darn useful with React.

[gaearon]

I want to avoid spending significant effort here until we're at least sure this is the way properties will end up. So far the news aren't very encouraging:

https://twitter.com/sebmarkbage/status/646473000749928448
https://twitter.com/sebmarkbage/status/646486868888911872
https://twitter.com/sebmarkbage/status/646489649972580353

[ramivalta]

Might it be time to revisit this now that the proposal is at stage 2 and new react docs refer to this feature (albeit with an experimental feature caveat)?

[selbekk]

+1 - is this something we can start looking into?

[wkwiatek]

I guess this PR is not valid for v3-alpha, am I right @nikolaymatrosov ?

[gajus]

Related bugs:

• Bind ES6 class methods in constructor #69
• does not respect public field binding react-hot-loader#554

[noinkling]

Proposal is now stage 3.

[rjhilgefort]

Any news on supporting this now that it's at stage 3?

[Fr33maan]

@gaearon Hi, I've read your thoughts on this feature from 2015/2016. Now it's in a good way of being part of one of the norm. Do you think the effort worth the benefits ?
Thanks for your job and your projects ! All contributors of redux are life saviors !