Dont merge - Dustin's refactor part 1

[dustingetz]

I screwed up and did this off of master instead of networked-pong, so i will need to redo it.

In the meantime, what do you think of these changes?

• busy work (splitting out files and cleaning up variables)
• interesting changes are in index.js
• I made everything into stateless function components to help indicate that we will never ever use a react lifecycle method - e.g. componentDidMount is not useful in aframe. shouldComponentUpdate is, maybe, the only exception to this. So it seems reasonable to prefer the function syntax rather than class syntax.
• I export an entrypoint fn, rather than calling ReactDOM.render inline, because source maps need to finish loading before we execute anything, otherwise debugging initialization is hard.

To understand the changes i would just start with index.js and explore the repo from there.

The next thing to refactor would be gameTick.

[meta-meta]

This is really great! I like the clean boundaries this sets up.

componentDidMount is not useful in aframe

it is when we want to get at the underlying THREE object, but in general, I agree... I guess the ref callback eliminates the need though https://facebook.github.io/react/docs/more-about-refs.html#the-ref-callback-attribute

I export an entrypoint fn, rather than calling ReactDOM.render inline, because source maps need to

finish loading before we execute anything, otherwise debugging initialization is hard. I still don't totally get how source maps work. Could you explain next time we pair?

The next thing to refactor would be gameTick.

Yes, this code is kind of gross, probably due to the dissonance of attempting to stick with setState() when it wasn't really applicable.

[dustingetz]

hi paul!

[meta-meta]

Hey Dustin, what's up? Time to port Aztec Pong to Hyperfiddle?