Skip to content

wrap/wrapAll + tests #167

merged 3 commits into from May 23, 2011

2 participants


I was doing a lot of DOM manipulation and needed wrapAll so ended up implementing it, if it's something you'd like feel free to take... or if you know a native way to do it so that's it's not needed please share as well ;) Tky Thomas!


There was a problem with replaceWith, it was inserting the new content after the previousSibling of the old content, this failed when the old content was the firstChild. Switched to just using node.replaceChild, updated the tests and probably saved some bytes.

@madrobby madrobby merged commit 65afc38 into madrobby:master May 23, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.