#12858 closed bug (fixed)
Sizzle, expando, $.clone(), ie6-8
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | undecided | Milestone: | None |
Component: | selector | Version: | 1.8.0 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Sizzle will add a variable: expando = ( "sizcache" + Math.random() ).replace( ".", "" ).
When i use $.clone(), the 'sizcache' would record in html tags, i need to clear them by RegExp.
Sorry for my poor english.
Change History (7)
comment:1 Changed 10 years ago by
Component: | unfiled → selector |
---|---|
Owner: | set to [email protected]… |
Status: | new → pending |
comment:2 Changed 10 years ago by
Status: | pending → open |
---|
I don't know if we'll want to fix this, but it is real. It affects .html()
as well.
comment:4 Changed 10 years ago by
It's been like this forever though, so although it might affect serialized html it doesn't have any really bad effects. Trying to remove those properties would make the slowest browsers even slower.
comment:5 Changed 10 years ago by
True, but negligibly so. It's just more removeAttribute
ing in cloneFixAttributes and a tweak to rinlinejQuery.
But it would add some size that might be better relegated to a plugin.
comment:6 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | open → closed |
Fix #12858: don't expose jQuery/Sizzle expandos
Changeset: e78cfb1a3756773aec24e83e358e1e4b40f2a116
comment:7 Changed 10 years ago by
So I think I've got this fixed, but the unit test was a bit evasive. Can you confirm against the new edge build?
Thanks for taking the time to contribute to the jQuery project! Please provide a complete reduced test case on jsFiddle to help us assess your ticket.
Additionally, be sure to test against the jQuery Edge version to ensure the issue still exists. To get you started, use this boilerplate: http://jsfiddle.net/FrKyN/. Open the link and click to "Fork" (in the top menu) to get started.