andrewbeng89/is429angulartest

is429angulartest's Issues

Issue about test expression of 'ng-hide' in app/partials/todos.html

Hello Andrew. In app/partials/todos.html, I found that the test expression of 'ng-hide' is not compatible with both Firebase and Mongolab implementations.

During debugging process, I found that the <p> element which should show up when there is no todo item is always hidden in Firebase implementation. Using Batarang I found that the todos in Firebase implementation is a JS Object specified by Angularfire, so length is undefined, and I found 'todos.$getIndex().length' will be a work around. In Mongolab implementation, todos is an Array which is fine.

Thus, the test expression of 'ng-hide' could be adjusted to "todos.length || todos.$getIndex().length || !loaded" to cover both implementations.

• Shen Rui