diff options
| author | Igor Minar | 2011-03-28 17:51:51 -0700 | 
|---|---|---|
| committer | Igor Minar | 2011-03-30 15:24:03 -0700 | 
| commit | 15ec78f5eff3f8fa74714fe10986be094915c800 (patch) | |
| tree | fe527b3cfe1bec20c371fae2113f62500823ad14 /src/widgets.js | |
| parent | a4863d52448170a8b94fa1fd2df79af1b66b6ad1 (diff) | |
| download | angular.js-15ec78f5eff3f8fa74714fe10986be094915c800.tar.bz2 | |
use document fragments to grow repeaters
- unless we are repeating OPTION elements, buffer new nodes in document
  fragment and append them to the DOM in one go at the end
- for OPTION elements we have to keep on using the old way
  because of how option widget communicates with select widget
  this should be change, but that change is out of scope of this CL
- modify jqLite to support wrapping of document fragments
- fix jqLite documentation typo
This change unintentionally avoids the following webkit bug that
that affects repeater growth:
https://bugs.webkit.org/show_bug.cgi?id=57059
However the following bug affecting shrining of repeaters is still
unresolved https://bugs.webkit.org/show_bug.cgi?id=57061
Diffstat (limited to 'src/widgets.js')
| -rw-r--r-- | src/widgets.js | 19 | 
1 files changed, 17 insertions, 2 deletions
| diff --git a/src/widgets.js b/src/widgets.js index a7d59289..2b44fccc 100644 --- a/src/widgets.js +++ b/src/widgets.js @@ -914,6 +914,8 @@ angularWidget('@ng:repeat', function(expression, element){            lastIterElement = iterStartElement,            collection = this.$tryEval(rhs, iterStartElement),            collectionLength = size(collection, true), +          fragment = (element[0].nodeName != 'OPTION') ? document.createDocumentFragment() : null, +          addFragment,            childScope,            key; @@ -938,13 +940,26 @@ angularWidget('@ng:repeat', function(expression, element){              children.push(childScope);              linker(childScope, function(clone){                clone.attr('ng:repeat-index', index); -              lastIterElement.after(clone); -              lastIterElement = clone; + +              if (fragment) { +                fragment.appendChild(clone[0]); +                addFragment = true; +              } else { +                //temporarily preserve old way for option element +                lastIterElement.after(clone); +                lastIterElement = clone; +              }              });            }            index ++;          }        } + +      //attach new nodes buffered in doc fragment +      if (addFragment) { +        lastIterElement.after(jqLite(fragment)); +      } +        // shrink children        while(children.length > index) {          children.pop().$element.remove(); | 
