ErikSchierboom / knockout-pre-rendered / 87

95%

push

travis-ci

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.

25 of 25 new or added lines in 1 file covered. (100.0%)

192 of 205 relevant lines covered (93.66%)

42.74 hits per line