From: Jonathan Sampson Date: Tue, 11 Mar 2014 07:44:34 +0000 (-0400) Subject: Manipulation: Change support test to be WWA-friendly X-Git-Tag: 2.1.1-beta1~6 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=85af4e6412e49c2e6a872feef00718a46c2fa2ce;p=jquery.git Manipulation: Change support test to be WWA-friendly Setting the innerHTML property in an unsafe manner raises issues in Windows Web Applications. Strings being passed into innerHTML cannot include the name attribute. Closes gh-1537 --- diff --git a/src/manipulation/support.js b/src/manipulation/support.js index e2ba066ec..fb1e85567 100644 --- a/src/manipulation/support.js +++ b/src/manipulation/support.js @@ -4,10 +4,17 @@ define([ (function() { var fragment = document.createDocumentFragment(), - div = fragment.appendChild( document.createElement( "div" ) ); + div = fragment.appendChild( document.createElement( "div" ) ), + input = document.createElement( "input" ); // #11217 - WebKit loses check when the name is after the checked attribute - div.innerHTML = ""; + // Support: Windows Web Apps (WWA) + // `name` and `type` need .setAttribute for WWA + input.setAttribute( "type", "radio" ); + input.setAttribute( "checked", "checked" ); + input.setAttribute( "name", "t" ); + + div.appendChild( input ); // Support: Safari 5.1, iOS 5.1, Android 4.x, Android 2.3 // old WebKit doesn't clone checked state correctly in fragments