angular.js/test/ng/directive, branch v1.0.5 fix(ngClass): keep track of old ngClass value manually 2013-02-19T04:35:40+00:00 Per Rovegård 2013-02-15T23:32:38+00:00 2aa212b19c71df82287b4b074da3ab14cbf37348 ngClassWatchAction, when called as a $watch function, gets the wrong old value after it has been invoked previously due to observation of the interpolated class attribute. As a result it doesn't remove classes properly. Keeping track of the old value manually seems to fix this. Closes #1637 
ngClassWatchAction, when called as a $watch function, gets the wrong old
value after it has been invoked previously due to observation of the
interpolated class attribute. As a result it doesn't remove classes
properly. Keeping track of the old value manually seems to fix this.

Closes #1637
fix(compile): should not leak memory when there are top level empty text nodes 2013-02-18T13:04:24+00:00 Pete Bacon Darwin 2013-02-18T12:05:16+00:00 1f23cfe9c751f84d9fc996df6d54961b2e46e868 The change to prevent <span> elements being wrapped around empty text nodes caused these empty text nodes to have scopes and controllers attached, through jqLite.data() calls, which led to memory leaks and errors in IE8. Now we exclude all but document nodes and elements from having jqLite.data() set both in the compiler and in ng-view. Fixes: #1968 and #1876 
The change to prevent <span> elements being wrapped around empty text nodes caused these empty text nodes to have scopes and controllers attached, through jqLite.data() calls, which led to memory leaks and errors in IE8.
Now we exclude all but document nodes and elements from having jqLite.data() set both in the compiler and in ng-view.

Fixes: #1968 and #1876
fix(a): workaround IE bug affecting mailto urls 2013-02-15T00:43:28+00:00 Igor Minar 2013-02-14T23:13:50+00:00 8043784fd73a4768f4e904fd4121b21cd1fb49b4 Apparently there is a really weird bug in IE6-8 that causes anchor textContent to be reset with href content when both contain @ symbol. Inserting a bogus comment node into all anchor elements in IE works around this browser bug. I'm fixing the issue via directive because that way we'll fix it for jQuery as well. I fixed an e2e test too because it was incorrect. Closes #1949 
Apparently there is a really weird bug in IE6-8 that causes anchor textContent
to be reset with href content when both contain @ symbol.

Inserting a bogus comment node into all anchor elements in IE works around this
browser bug.

I'm fixing the issue via directive because that way we'll fix it for jQuery as
well.

I fixed an e2e test too because it was incorrect.

Closes #1949
fix(ngSwitch): don't leak when destroyed while not attached 2013-01-18T08:03:42+00:00 danilsomsikov 2013-01-11T17:21:13+00:00 89303fd2dccb780bea594b16b8ec85d9f3bd8f92 The leak can occur when ngSwich is used inside ngRepeat or any other directive which is destroyed while its transcluded content (which includes ngSwitch) is not attached to the DOM. Refactor ngSwitch to use controller instead of storing data on compile node. This means that we don't need to clean up the jq data cache. Controller reference is released when the linking fn is released. Closes #1621 
The leak can occur when ngSwich is used inside ngRepeat or any other
directive which is destroyed while its transcluded content (which
includes ngSwitch) is not attached to the DOM.

Refactor ngSwitch to use controller instead of storing data on compile
node. This means that we don't need to clean up the jq data cache.
Controller reference is released when the linking fn is released.

Closes #1621
fix(ngRepeat): correctly apply $last if repeating over object 2013-01-17T08:25:07+00:00 Pete Bacon Darwin 2013-01-11T12:47:38+00:00 ed2fd2d0caad4e4b45f79bb39fd2a8b03fea8faa If the $last property is calculated from the original collectionLength on an object and properties starting with $ were filtered out, then $last is never applied and $middle is applied erroniously. Closes #1789 
If the $last property is calculated from the original collectionLength
on an object and properties starting with $ were filtered out, then $last
is never applied and $middle is applied erroniously.

Closes #1789
fix(select): support optgroup + select[multiple] combo 2012-12-05T01:21:31+00:00 _pants 2012-11-29T16:46:51+00:00 54c0d464b0f34f9bfd076883a6a78cf5925bd975 Closes #1553 
Closes #1553
fix(ngRepeat): support mostly-stable repeating for primitives 2012-11-26T20:33:45+00:00 Igor Minar 2012-11-23T21:43:30+00:00 cde2f1a868ffacdc592803358130587fd73aca7b I'm reverting changes that were originally done to ngRepeat to fix #933, because these are now not necessary after the previous changes to keep ngModel always synced with the DOM. 
I'm reverting changes that were originally done to ngRepeat to fix #933,
because these are now not necessary after the previous changes to keep
ngModel always synced with the DOM.
test(ngRepeat): clean up and improve tests 2012-11-26T19:39:10+00:00 Igor Minar 2012-11-23T15:08:47+00:00 6e2c38f54dc012dc919cdd7f7ab9b73dc1e99c66 






test(ngRepeat): add test for issue #1076
2012-11-26T19:38:51+00:00

Igor Minar

2012-11-26T19:17:11+00:00

ff4b3e20c1bcf8f4c76114f5537c8857393c9b2b












fix(select): select option with a label of 0 is not shown
2012-10-31T22:03:48+00:00

Sudhir Jonathan

2012-10-21T07:27:27+00:00

d981c2a3ecfaf572a836c7533ee3a9553aae327d

Bug caused by the use of the `||` operator to replace all non-truthy
values with an empty string. Changed to replace only `undefined` values.

Closes #1401





Bug caused by the use of the `||` operator to replace all non-truthy
values with an empty string. Changed to replace only `undefined` values.

Closes #1401