Created
August 16, 2016 23:35
-
-
Save steida/637c748c796327f1e96a87476195a506 to your computer and use it in GitHub Desktop.
React stateless functional component with higher order component wrappers.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| import Helmet from 'react-helmet'; | |
| import React from 'react'; | |
| import linksMessages from '../../common/app/linksMessages'; | |
| import { FormattedMessage } from 'react-intl'; | |
| import { connect } from 'react-redux'; | |
| import { fields } from '../../common/lib/redux-fields'; | |
| let OfflinePage = ({ fields, online }) => ( | |
| <div className="offline-page"> | |
| <FormattedMessage {...linksMessages.offline}> | |
| {message => | |
| <Helmet title={message} /> | |
| } | |
| </FormattedMessage> | |
| <h2> | |
| Offline | |
| </h2> | |
| <p> | |
| <code>state.app.online: <b>{online.toString()}</b></code> | |
| </p> | |
| <p> | |
| Editable fields are persisted in the local storage by default. But | |
| syncing is hard, so that's why the next field is disabled on offline. | |
| </p> | |
| <input | |
| {...fields.somePersistedText} | |
| disabled={!online} | |
| type="text" | |
| /> | |
| <p> | |
| The user state is also persisted in the local storage. Check{' '} | |
| <code>configureStorage.js</code>. | |
| </p> | |
| <p>TODO: Explain easy immutable offline syncing.</p> | |
| </div> | |
| ); | |
| OfflinePage.propTypes = { | |
| fields: React.PropTypes.object.isRequired, | |
| online: React.PropTypes.bool.isRequired, | |
| }; | |
| OfflinePage = fields(OfflinePage, { | |
| path: 'offline', | |
| fields: [ | |
| 'somePersistedText', | |
| ], | |
| }); | |
| export default connect(state => ({ | |
| online: state.app.online, | |
| }))(OfflinePage); |
I know. But hoc is even more confusing than let and it's another dependency. let is explicit.
Maybe the best syntax will be decorators when reborn.
https://gist.github.com/steida/440609d8bd51a04080854c9a5ec9b35e
Yes, I would like to see decorators for const! Is there any initiative?
There is, but we are not there yet. Until then, use let is the best.
Or you can use Ramda. Actually you should, because your hoc is too much specific. https://medium.com/@mirkomariani/functional-components-with-react-stateless-functions-and-ramda-e83e54fcd86b#.cv5t8uz86
Thanks for the link, will look at it 👍
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The use of
letcould be confusing. I would prefer to useconst OfflinePageWithFields = fields(OfflinePage, …. Or I would like to see the following pattern for applying HOC: https://github.com/robinpokorny/transform-props-with#multiple-transformationsThen it could look like this: