Side navigation
#5022 closed enhancement (duplicate)
Opened August 07, 2009 02:06AM UTC
Closed November 11, 2010 03:46AM UTC
Last modified November 11, 2010 03:46AM UTC
strings of markup and application/xml
| Reported by: | jablko | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | 1.4 |
| Component: | core | Version: | 1.3.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Attachments (1)
Change History (5)
Changed August 07, 2009 02:42AM UTC by comment:1
| component: | unfilled → core |
|---|
Changed April 09, 2010 11:43PM UTC by comment:2
I committed this updated change to github, http://github.com/jablko/jquery/commit/2fea569dce44e5f6df8d67fef277db4599d4aab2
Changed April 09, 2010 11:49PM UTC by comment:3
.innerHTML doesn't work in non-HTML documents, I think handling the exception with DOMParser() allows some current tests which fail with test/xhtml.php (Content-Type: application/xhtml+xml) to pass, e.g. test/unit/manipulation.js "html(String)"
Here's a new test, http://www.sfu.ca/~jdbates/tmp/jquery/201004090/patch
Without this change it passes with test/index.html but fails with test/xhtml.php - with this change it passes with both
Changed November 11, 2010 03:46AM UTC by comment:4
| resolution: | → duplicate |
|---|---|
| status: | new → closed |
This patch is too magical for my taste, it's very difficult to tell xml from html and wrapping a try/catch around it is going to cause trouble no doubt.
I'm closing this as a dup of #4617 which asks for xml support but takes a different approach.
Perhaps you'd want to see if the context, rather than the div, is xml?