Switch to using classes
arthurvr opened this issue · 5 comments
See original issue, tastejs/todomvc#348. We want to switch all our ids over to classes. Than we should release v2 and start porting all of our apps over.
We'll also need to update the https://github.com/tastejs/todomvc/blob/master/app-spec.md and the test suite.
We'll also need to update the https://github.com/tastejs/todomvc/blob/master/app-spec.md and the test suite.
Yup. What's the preferred way to handle the tests? I guess that if we switch them all over to classes all tests will fail. I don't think that's wanted as it might take a while for all apps to get updated.
Opened a PR for the template. Next up the tests and then I can start working on updating the apps.
Regarding the tests, it looks like some changes were made a while back which allow them to switch between the use of classes or IDs to identify elements: