aboutsummaryrefslogtreecommitdiffstats
path: root/src/attributes/prop.js
diff options
context:
space:
mode:
Diffstat (limited to 'src/attributes/prop.js')
-rw-r--r--src/attributes/prop.js20
1 files changed, 15 insertions, 5 deletions
diff --git a/src/attributes/prop.js b/src/attributes/prop.js
index 06528b0a5..da7bc1e86 100644
--- a/src/attributes/prop.js
+++ b/src/attributes/prop.js
@@ -5,7 +5,8 @@ define( [
"../selector"
], function( jQuery, access, support ) {
-var rfocusable = /^(?:input|select|textarea|button)$/i;
+var rfocusable = /^(?:input|select|textarea|button)$/i,
+ rclickable = /^(?:a|area)$/i;
jQuery.fn.extend( {
prop: function( name, value ) {
@@ -55,10 +56,19 @@ jQuery.extend( {
propHooks: {
tabIndex: {
get: function( elem ) {
- return elem.hasAttribute( "tabindex" ) ||
- rfocusable.test( elem.nodeName ) || elem.href ?
- elem.tabIndex :
- -1;
+
+ // elem.tabIndex doesn't always return the
+ // correct value when it hasn't been explicitly set
+ // http://fluidproject.org/blog/2008/01/09/getting-setting-and-removing-tabindex-values-with-javascript/
+ // Use proper attribute retrieval(#12072)
+ var tabindex = jQuery.find.attr( elem, "tabindex" );
+
+ return tabindex ?
+ parseInt( tabindex, 10 ) :
+ rfocusable.test( elem.nodeName ) ||
+ rclickable.test( elem.nodeName ) && elem.href ?
+ 0 :
+ -1;
}
}
},