Skip to content
Permalink
Browse files

Fix #12915, add propHook for scrip src property on oldIE. Close gh-1035.

  • Loading branch information
aFarkas authored and dmethvin committed Nov 19, 2012
1 parent 20be650 commit ca5e06a15c99d5c181aff6e08804ca803424648d
Showing with 12 additions and 7 deletions.
  1. +8 −6 src/attributes.js
  2. +4 −1 test/unit/attributes.js
@@ -568,12 +568,14 @@ if ( !jQuery.support.hrefNormalized ) {
});
});

// link's href property should get the full normalized URL (#10299)
jQuery.propHooks.href = {
get: function( elem, name ) {
return elem.getAttribute( name, 4 );
}
};
// href/src property should get the full normalized URL (#10299/#12915)
jQuery.each([ "href", "src" ], function( i, name ) {
jQuery.propHooks[ name ] = {
get: function( elem ) {
return elem.getAttribute( name, 4 );
}
};
});
}

if ( !jQuery.support.style ) {
@@ -54,7 +54,7 @@ test( "jQuery.propFix integrity test", function() {
});

test( "attr(String)", function() {
expect( 47 );
expect( 48 );

equal( jQuery("#text1").attr("type"), "text", "Check for type attribute" );
equal( jQuery("#radio1").attr("type"), "radio", "Check for type attribute" );
@@ -94,6 +94,9 @@ test( "attr(String)", function() {
equal( jQuery("#tAnchor5").attr("href"), "#5", "Check for non-absolute href (an anchor)" );
jQuery("<a id='tAnchor6' href='#5' />").appendTo("#qunit-fixture");
equal( jQuery("#tAnchor5").prop("href"), jQuery("#tAnchor6").prop("href"), "Check for absolute href prop on an anchor" );

$("<script type='jquery/test' src='#5' id='scriptSrc'></script>").appendTo("#qunit-fixture");
equal( jQuery("#tAnchor5").prop("href"), jQuery("#scriptSrc").prop("src"), "Check for absolute src prop on a script" );

// list attribute is readonly by default in browsers that support it
jQuery("#list-test").attr( "list", "datalist" );

0 comments on commit ca5e06a

Please sign in to comment.
You can’t perform that action at this time.