-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
orderBy doesn't seems to work #12
Comments
@diego-vieira I don't support you have come across the solution to this problem? Getting the same error here. What's odd is it still seems to work. Just errors out a lot and makes it super slow. |
@wedgybo because I had to move on the next steps, I used ng-repeat instead till I hear from the dev. |
No problem, I'll have a look into it the noo. Cheers for the fast reply! |
@diego-vieira @wedgybo |
Hi @allaud, thanks for having a look. Can you paste any errors you had with the plunker code please? This is my output. https://www.dropbox.com/s/ibnpj2yw103vd4o/Screenshot%202013-12-19%2008.41.59.png |
@wedgybo I have the following error: http://cropme.ru/e2d82c7191848a7dd4f51a19756bf809 |
@allaud Thanks. Can you try again, I must have had the wrong version set or something as there's a bunch of stuff missing from the markup there. |
@wedgybo just tested your latest plunker on IE to be on the safe side. @allaud I'll try to follow the trail and find out why. Maybe I can find something. |
Cool. I had a look last night but my understanding of the internals of angular and ng-repeat are still lacking so I'll need to get up to speed with how all that works before I can help out. |
Line of interest:
Error message
ng-repeat work just fine.
Thanks
The text was updated successfully, but these errors were encountered: