Skip to content

Commit 1d202c6

Browse files
author
timmywil
committed
Move the value attrHook to the main attrHooks object to save bytes
1 parent 9d4033d commit 1d202c6

File tree

1 file changed

+19
-20
lines changed

1 file changed

+19
-20
lines changed

src/attributes.js

Lines changed: 19 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -414,6 +414,25 @@ jQuery.extend({
414414
0 :
415415
undefined;
416416
}
417+
},
418+
// Use the value property for back compat
419+
// Use the formHook for button elements in IE6/7 (#1954)
420+
value: {
421+
get: function( elem, name ) {
422+
if ( formHook && jQuery.nodeName( elem, "button" ) ) {
423+
return formHook.get( elem, name );
424+
}
425+
return name in elem ?
426+
elem.value :
427+
null;
428+
},
429+
set: function( elem, value, name ) {
430+
if ( formHook && jQuery.nodeName( elem, "button" ) ) {
431+
return formHook.set( elem, value, name );
432+
}
433+
// Does not return so that setAttribute is also used
434+
elem.value = value;
435+
}
417436
}
418437
},
419438

@@ -497,26 +516,6 @@ boolHook = {
497516
}
498517
};
499518

500-
// Use the value property for back compat
501-
// Use the formHook for button elements in IE6/7 (#1954)
502-
jQuery.attrHooks.value = {
503-
get: function( elem, name ) {
504-
if ( formHook && jQuery.nodeName( elem, "button" ) ) {
505-
return formHook.get( elem, name );
506-
}
507-
return name in elem ?
508-
elem.value :
509-
null;
510-
},
511-
set: function( elem, value, name ) {
512-
if ( formHook && jQuery.nodeName( elem, "button" ) ) {
513-
return formHook.set( elem, value, name );
514-
}
515-
// Does not return so that setAttribute is also used
516-
elem.value = value;
517-
}
518-
};
519-
520519
// IE6/7 do not support getting/setting some attributes with get/setAttribute
521520
if ( !jQuery.support.getSetAttribute ) {
522521

0 commit comments

Comments
 (0)