Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

ng-class-even and ng-class-odd do not properly update with filter/orderby #1076

Closed
andyneville opened this issue Jun 18, 2012 · 4 comments
Closed

Comments

@andyneville
Copy link

There is a Google Groups discussion at https://groups.google.com/forum/#!topic/angular/V6JfL75_pPI, Aleksandr Petrov put together a jsFiddle that demonstrates this bug with filtering. http://jsfiddle.net/alexpetrov/CLrWd/

I see the same issue when orderby changes dynamically, here's a jsFiddle to demonstrate that: http://jsfiddle.net/AndyNeville/m9xHA/

@IgorMinar
Copy link
Contributor

confirmed. this is definitely a bug. thanks for the report. we are going to take care of this in one of the 1.0.x bug fix releases

@NathanWalker
Copy link

+1 on fixing this bug.

@tion8888
Copy link

this fix would be much appreciated!

@rubycut
Copy link

rubycut commented Aug 20, 2012

Is this going to be merged any time soon?

@mhevery mhevery closed this as completed in 6f207a2 Sep 4, 2012
mhevery pushed a commit to mhevery/angular.js that referenced this issue Sep 7, 2012
rgaskill pushed a commit to rgaskill/angular.js that referenced this issue Nov 13, 2012
…ven is not updated.

Issue angular#1563 fix. Put back in place the original fix issue angular#1076
IgorMinar added a commit to IgorMinar/angular.js that referenced this issue Nov 23, 2012
We need to watch $index in addition to cssClasses because only then
we can correctly respond to shrinking or reordered repeaters.

Closes angular#1076
IgorMinar added a commit that referenced this issue Nov 26, 2012
We need to watch $index in addition to cssClasses because only then
we can correctly respond to shrinking or reordered repeaters.

Closes #1076
IgorMinar added a commit that referenced this issue Nov 26, 2012
We need to watch $index in addition to cssClasses because only then
we can correctly respond to shrinking or reordered repeaters.

Closes #1076
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants