Skip to content

Use PureRenderMixin #10

Description

@mlsteele

Almost all of the components in this app are pure.
We should use PureRenderMixin

Also, it would be interesting to benchmark various UI changes to see whether this makes a difference. It still feels a bit slow on mobile.

Maybe doing "onChange -> rerender app" is slow.
Try using setprops on the outer component (forcing the update against PureRenderMixin) instead.
"Calling React.renderComponent() again on the same node is the preferred way to update a root-level component."

http://facebook.github.io/react/docs/component-api.html#forceupdate

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions