ssorallen / react-todos

Backbone's example TODO app with React Components for views

React Todos

Backbone's example TODO app with React JS Components for views in place of Backbone.View.

Demo

https://ssorallen.github.io/react-todos/

The demo uses local storage to save todos in the browser. The views are all rendered with React, which means there is no dependency on another DOM manipulation library like jQuery.

Running Locally

1. Clone this repository

2. Install dependencies

    $ yarn install
   

3. Run the local server

    $ yarn start
   

4. See the running app at http://localhost:3000

Favicon

The great little todo list favicon comes from Typicons.

react-todos's Issues

Leave public API to object, not BackboneMixin

Mixins should avoid modifying the public API of objects, which means BackboneMixin should let objects that use it implement the equivalent of getBackboneObject.

I'd like to go with something like getResource, which should return an object that extends Backbone.Events, e.g. an instance or Backbone.Model or Backbone.Collection.

As mentioned in #1.

A few ideas for tweaks

This is great! Had a few ideas for improvement.

• TodoListItemComponent could have editing be on props instead of state
• Then in componentDidUpdate() you can only focus if (this.props.editing && !prevProps.editing)
• In general I don't like it when mixins read from props since they can manipulate the public API of a component. So I would move getBackboneObject() to the user of the mixin rather than the mixin itself.

Just a few ideas... this looks great!