ErikSchierboom / knockout-pre-rendered

94%

master: 95%

LAST BUILD ON BRANCH 0.9.2

branch: 0.9.2

CHANGE BRANCH

x

Reset

• 0.9.2
• 0.10.0
• 0.10.1
• 0.11.0
• 0.7.0
• 0.7.1
• 0.7.2
• 0.8.0
• 0.9.0
• 0.9.1
• 0.9.1-beta
• appveyor
• coverage
• data-change-callback
• examples
• fix-foreach-virtual-elements-bug
• gulp-v4
• master
• tr-tags-script
• unknown-attribute-bug
• update-dependencies

Build # 88

Build Type

push

travis-ci

Committed by ErikSchierboom

Commit Message

Nodes per element (#51)

* Fixed some browser compatibility issues with older versions of IE.

   * Changed how `<tr>` elements are read from a named template (original code did not work in IE<10).
   * Do not use cloneNode or delete script tag based templates. This breaks the templates in IE<9.
   * Used `ko.utils.arrayMap` instead of `.map` since the latter is unsupported in IE<9.
   * Removed hanging commas that cause syntax errors in IE<8.

These changes allow the ko-pre-rendered bindings to support the same browser versions as knockout itself.

* Added "nodesPerElement" option to indicate how many adjacent HTML nodes should be associated with a single element in the array.

   * If unspecified, the default nodesPerElement is 1
   * Modified "makeTemplateNode" to support multiple nodes in the template.
   * Modified the "existing" method so it correctly applies bindings when nodesPerElement > 1.
   * Modified the "createElements" method so it creates the correct number of array elements for the number of existing HTML nodes.

* Added some tests for the "nodesPerElement" option.

Tested:
   * Initialization of the observable array based on existing markup.
   * Proper rendering of multi-node templates when adding new items to the observable array.
   * Proper removal of markup when removing items from the observable array.

* Updated package version and distributable files.

* Added nodesPerElement exmaples.

* Updated version info in README.

* Added test for script tag template with tr elements.

* Fixed code styling and indentation issues

* Couple more styling fixes.

Run Details

192 of 205 relevant lines covered (93.66%)

42.74 hits per line