diff options
| author | Keyamoon | 2012-12-20 03:18:47 +0330 |
|---|---|---|
| committer | Igor Minar | 2013-01-08 14:54:15 -0800 |
| commit | 76a6047af690781b8238ba7924279470ba76d081 (patch) | |
| tree | 1fc16cac57f8fac4ce9f6b84a7dacd0f76d30544 /src/jqLite.js | |
| parent | b6b7c5a1d66073937709158da8c2d688cb45c9f6 (diff) | |
| download | angular.js-76a6047af690781b8238ba7924279470ba76d081.tar.bz2 | |
fix(jqLite): make next() ignore non-element nodes
next() is supposed to return the next sibling *element* so it
should ignore text nodes. To achieve this, nextElementSibling()
should be used instead of nextSibling().
Diffstat (limited to 'src/jqLite.js')
| -rw-r--r-- | src/jqLite.js | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/src/jqLite.js b/src/jqLite.js index 8305f0c3..d156ae6d 100644 --- a/src/jqLite.js +++ b/src/jqLite.js @@ -718,7 +718,16 @@ forEach({ }, next: function(element) { - return element.nextSibling; + if (element.nextElementSibling) { + return element.nextElementSibling; + } + + // IE8 doesn't have nextElementSibling + var elm = element.nextSibling; + while (elm != null && elm.nodeType !== 1) { + elm = elm.nextSibling; + } + return elm; }, find: function(element, selector) { |
