Skip to content
Snippets Groups Projects
Commit f8971ca1 authored by Ben Wheeler's avatar Ben Wheeler
Browse files

added comments about compose function

parent 339e1caf
No related branches found
No related tags found
No related merge requests found
......@@ -124,6 +124,9 @@ const ConnectedGUI = connect(
mapDispatchToProps,
)(GUI);
// note that redux's 'compose' function is just being used as a general utility to make
// the hierarchy of HOC constructor calls clearer here; it has nothing to do with redux's
// ability to compose reducers.
const WrappedGui = compose(
ErrorBoundaryHOC('Top Level App'),
ProjectFetcherHOC,
......
......@@ -50,7 +50,14 @@ const mapDispatchToProps = dispatch => ({
onSeeInside: () => dispatch(setPlayer(false))
});
const ConnectedPlayer = connect(mapStateToProps, mapDispatchToProps)(Player);
const ConnectedPlayer = connect(
mapStateToProps,
mapDispatchToProps
)(Player);
// note that redux's 'compose' function is just being used as a general utility to make
// the hierarchy of HOC constructor calls clearer here; it has nothing to do with redux's
// ability to compose reducers.
const WrappedPlayer = compose(
HashParserHOC,
AppStateHOC,
......
......@@ -15,6 +15,10 @@ import ProjectMetaDataHOC from '../lib/project-metadata-hoc.jsx';
*/
export default appTarget => {
GUI.setAppElement(appTarget);
// note that redux's 'compose' function is just being used as a general utility to make
// the hierarchy of HOC constructor calls clearer here; it has nothing to do with redux's
// ability to compose reducers.
const WrappedGui = compose(
AppStateHOC,
HashParserHOC, // NOTE: IMPORTANT: ok that i've reordered this?
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment