Comments (28)
Other deficiencies:
- Clicking on the task content should not mark the task as completed.
- Double click should allow editing.
- "Mark all complete" checkbox should be hidden when there are no tasks.
- Footer should be hidden when there are no tasks.
- Need to add delete ability on hover over tasks.
from todomvc.
The Ember+Require app just landed in 1b65322
@stas Would you be interested in getting the non-require version completed? I think there could be a good amount of reuse.
//cc @addyosmani
from todomvc.
I could take a look if there's nobody else interested in picking it up.
Also maybe we should ask @tomdale
He is the original author and It would be nice to ask him first.
from todomvc.
👍
from todomvc.
I think the non-AMD/Require version wouldn't be too difficult to complete. Always happy for @tomdale to review anything Ember related going into this project, just to be on the safe side :)
from todomvc.
@stas Still interested? We would really like to get an up to date version into our 1.0 release 😃
from todomvc.
Hey, sorry guys I'm a bit busy lately.
I was following ember devlog and was thinking to update both examples with the new routing support.
When you were planning to release 1.0, lets see if we can sync.
from todomvc.
Np, I absolutely understand ;)
We've set the release date to june 20th, which means we'll need it some days before that.
Do you think you'll be able to have it done by that?
from todomvc.
20th looks great, my holidays start early june, so it seems doable.
Sure this should not stop people jumping in :)
from todomvc.
@stas Sounds good! :D
from todomvc.
I've got bandwidth to work on this so I'm going to wrap it up this evening and hopefully have @sindresorhus review :)
from todomvc.
If you dare :p
from todomvc.
I plan on submitting this while you're asleep.
It's the safest way to sneak in a merge ;D
from todomvc.
Sounds cool :)
from todomvc.
I never sleep!
from todomvc.
Of course you don't :p
from todomvc.
@sindresorhus could you review https://github.com/addyosmani/todomvc/tree/emberjs?
If you're happy with it, let me know and I'll pull it into master, otherwise address any issues you notice :)
from todomvc.
Prepare... Ya, there are a few comments :p
- Trim input on edit and remove if empty
Make sure to .trim() the input and then check that it's not empty. If it's empty the todo should instead be destroyed.
- Remove app.css and the reference in the HTML
- app.js needs an IIFE and 'use strict'
- app.js: Missing some statement curly braces
- Split app.js up in appropriate files?
from todomvc.
@sindresorhus thanks for the review. I'll address those!
On the last point, I think we need to decide whether we want to enforce splitting up of applications or keeping it optional. I was looking at the Backbone app last night and remember that it currently also keeps all of its logic in a single file (but of course can be split out). Thoughts?
from todomvc.
Even though not necessary in our small Todo app, I think we should enforce some good practices.
from todomvc.
+1 on splitting into multiple files. I think that making these as "realistic" as possible is important. Every web app developer I know is using a tool for file management these days.
from todomvc.
Updated in https://github.com/addyosmani/todomvc/tree/emberjs:
- IIFE + use strict added
- Split app.js into its component parts (models, controllers, main)
- Missing curly braces added
- Wasn't sure about the app.css comment - it's used in the Spine demo app and is needed for overrides
Missing: Trim input on edit and remove if empty
@tomdale there's currently one issue with the app that I'm trying to wrap my head around. We're working under the same namespaces as the RequireJS version of the app, however, now that IIFEs are being used the binding context for content seems to be a little messed up. e.g the remaining and completed item counts now only seem to update correctly on page refresh but not when the app is interacted with.
Do you think you might be able to take a look to see what I'm doing wrong? :)
from todomvc.
There's a new pull request #202 to include routing support.
@addyosmani please check if there are any other issues like the one with counts.
from todomvc.
@sindresorhus I've just patched the trim on edit behaviour and the remove if empty should be working fine too. Could you do a final review on the emberjs branch so I can merge into master if we're happy with it? :)
from todomvc.
@addyosmani Almost, only thing left is that the filters are not selected when clicked. All should be selected by default. The others should be selected when the hash changes and on load.
from todomvc.
ping
from todomvc.
@sindresorhus Tried fixing this, but ran into some roadblocks. I've pinged a few Ember devs including @tomdale but I think they might be swamped. If no one is able to help resolve, I'll give this another shot tomorrow morning.
from todomvc.
Closing as routing was fixed in #231
from todomvc.
Related Issues (20)
- Change footer's todo item filter anchor tag components to button tag
- [New App] Bonsai Framework
- Yew framework TODO request HOT 1
- Upstream Speedometer 3.0 TodoMVC apps
- More developer-friendly Ember builds HOT 3
- bug: Found this on https://todomvc.com/
- Azure cloud example for Todo
- Title HOT 1
- Thoughts on contributing an updated version of the React example
- Running tests with node 20.10 or 20.11 raises "ReferenceError: primordials is not defined" HOT 1
- Ember example link broken HOT 6
- Ember example link broken in index.html page
- Native app development
- Flutter
- React Native
- Astrojs SSR
- update Speedometer link HOT 1
- Add sidebar to Ember example HOT 2
- https://github.com/tastejs/todomvc.wiki.git
- [Angular] Expression Changed After Checked Error
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from todomvc.