#10667 closed bug (fixed)
HTML5 Support in .wrapAll() does add a ":" to element
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | blocker | Milestone: | 1.7.1 |
Component: | manipulation | Version: | 1.7 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Hi there - im testing the HTML5 support for adding some content in IE 8
$('#content section.content').clone()
.find('h2') .prependTo('#content section.content') .wrapAll('<aside class="sidebar"></aside>');
leads to <:aside class="sidebar"><h2>...</h2>
jsFiddle:take a look at it on IE 8
Change History (11)
comment:1 Changed 12 years ago by
Component: | unfiled → manipulation |
---|---|
Milestone: | None → 1.7.1 |
Owner: | set to Rick Waldron |
Priority: | undecided → blocker |
Status: | new → assigned |
comment:2 Changed 12 years ago by
Owner: | changed from Rick Waldron to t.fellinger@… |
---|---|
Status: | assigned → pending |
comment:3 Changed 12 years ago by
Status: | pending → open |
---|
I can see it in the html view from F12 Developer Tools.
comment:8 Changed 12 years ago by
So, I propose we do something similar to the newCloneNode function in http://www.iecss.com/shimprove/javascript/shimprove.1-0-1.js, but with the html5-safe fragment that we have and only for html5 elements. elem.cloneNode( true ) is not sufficient for initial creation.
There are a couple different codepaths I'm envisioning. We should keep in mind that using cloneNode and then attaching it to the safeFragment also fixes it.
comment:9 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | open → closed |
Fix busted rnoshimcache. Correctly clone detached unknown elems. Fixes #10667, #10670.
- \s => |, Removes 4 bytes from gzipped build
- Adds tests for clone attributes, children and events Changeset: 66e65c81684e314448620822c0ba93d9d8c523cd
comment:11 Changed 12 years ago by
@tom - the patch has landed and will be included in the 1.7.1 bug-fix release
I'm not able to reproduce this - how are you testing and/or observing this issue?