Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

virtual element binding fails in re-written template #494

Closed
mbest opened this issue May 18, 2012 · 2 comments
Closed

virtual element binding fails in re-written template #494

mbest opened this issue May 18, 2012 · 2 comments
Milestone
2.2.0

Comments

@mbest
Copy link
Member

mbest commented May 18, 2012

Example: http://jsfiddle.net/mbest/r9Mtg/

Error and stack trace (from Chrome):

Uncaught Error: HIERARCHY_REQUEST_ERR: DOM Exception 3
ko.utils.setDomNodeChildren         knockout-latest.debug.js:180
ko.virtualElements.setDomNodeChildren       knockout-latest.debug.js:1711
executeTemplate     knockout-latest.debug.js:2917
ko.renderTemplate.ko.dependentObservable.disposeWhen        knockout-latest.debug.js:2960
evaluateImmediate       knockout-latest.debug.js:1194
ko.dependentObservable      knockout-latest.debug.js:1248
ko.renderTemplate       knockout-latest.debug.js:2950
ko.bindingHandlers.template.update      knockout-latest.debug.js:3062
ko.dependentObservable.disposeWhenNodeIsRemoved     knockout-latest.debug.js:1997
evaluateImmediate       knockout-latest.debug.js:1194
ko.dependentObservable      knockout-latest.debug.js:1248
applyBindingsToNodeInternal     knockout-latest.debug.js:1949
ko.applyBindingsToNode      knockout-latest.debug.js:2023
ko.templateRewriting.applyMemoizedBindingsToNextSibling     knockout-latest.debug.js:2757
ko.memoization.unmemoize        knockout-latest.debug.js:748
ko.memoization.unmemoizeDomNodeAndDescendants       knockout-latest.debug.js:762
options     knockout-latest.debug.js:2891
invokeForEachNodeOrCommentInContinuousRange     knockout-latest.debug.js:2871
activateBindingsOnContinuousNodeArray       knockout-latest.debug.js:2890
ko.renderTemplateForEach.activateBindingsCallback       knockout-latest.debug.js:2993
ko.utils.setDomNodeChildrenFromArrayMapping     knockout-latest.debug.js:3305
ko.renderTemplateForEach.ko.dependentObservable.disposeWhenNodeIsRemoved        knockout-latest.debug.js:3008
evaluateImmediate       knockout-latest.debug.js:1194
ko.dependentObservable      knockout-latest.debug.js:1248
ko.renderTemplateForEach        knockout-latest.debug.js:2998
ko.bindingHandlers.template.update      knockout-latest.debug.js:3055
ko.dependentObservable.disposeWhenNodeIsRemoved     knockout-latest.debug.js:1997
evaluateImmediate       knockout-latest.debug.js:1194
ko.dependentObservable      knockout-latest.debug.js:1248
applyBindingsToNodeInternal     knockout-latest.debug.js:1949
applyBindingsToNodeAndDescendantsInternal       knockout-latest.debug.js:1917
applyBindingsToDescendantsInternal      knockout-latest.debug.js:1899
applyBindingsToNodeAndDescendantsInternal       knockout-latest.debug.js:1926
ko.applyBindings        knockout-latest.debug.js:2036
(anonymous function)        fiddle.jshell.net:45
@mbest mbest mentioned this issue May 18, 2012
Merged
mbest added a commit that referenced this issue Aug 17, 2012
@mbest
Fix new spec so it works in IE. Ensure virtual elements with newlines…
a6377f1 
… work in rewritten templates: requires fix for #494 also.
@mbest
Copy link
Member Author

mbest commented Aug 17, 2012

only the last commit counts.

@mbest
Copy link
Member Author

mbest commented Aug 21, 2012

fixed with #279

@mbest mbest closed this as completed Aug 21, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.2.0
Development

No branches or pull requests
1 participant
@mbest