/todomvc/todomvc-toolkit

googlearchive / projects

Projects that use, enhance, or demonstrate Polymer's technologies

BSD 3-Clause "New" or "Revised" License

62 stars 17 forks source link

Open sorvell opened 11 years ago

sorvell commented 11 years ago

make sure app satisfies the todomvc specification.
blurring the editing item should commit its value
Change name to just "todomvc."
Remove empty js folder.
td-model: is the map needed? Consider the td-model in the todomvc-model-as-component version
td-app: consider making a model method to 'markAllCompleted' to call in toggleAllCompletedAction.
td-list: do we need base.css here?

morethanreal commented 11 years ago

I think calling it markAllCompleted would be incorrect since you can click it to mark all incomplete.

morethanreal commented 11 years ago

re: spec

The spec says we should change the item state by toggling the class, but it is awkward to do for us and I implemented it with an attribute instead.