Skip to main content

Bug Tracker

Side navigation

#3649 closed bug (fixed)

Opened November 25, 2008 11:46AM UTC

Closed January 05, 2009 05:34PM UTC

Last modified March 14, 2012 11:51PM UTC

Can't set tabindex using attr() in IE - patch supplied

Reported by: jollytoad Owned by: flesler
Priority: minor Milestone: 1.3
Component: core Version: 1.2.6
Keywords: tabindex attr Cc: jollytoad
Blocked by: Blocking:
Description

IE only allows the tabindex to be set with the tabIndex property,

so .attr('tabindex', ...) does not work.

I've added it to jQuery.props list and it works in IE, FF, Opera, and Chrome. Safari doesn't seems to respond to tabindex like any other browser anyway!

Anyway, test case and patch attached.

Attachments (4)
  • tabindex-jquery.patch (0.3 KB) - added by jollytoad November 25, 2008 11:48AM UTC.

    Patch for core,js against SVN rev 5949

  • tabindex.2.patch (5.8 KB) - added by scottgonzalez January 05, 2009 05:25PM UTC.
  • tabindex.html (0.5 KB) - added by jollytoad November 25, 2008 11:46AM UTC.

    Test page

  • tabindex.patch (4.4 KB) - added by scottgonzalez January 05, 2009 03:59AM UTC.
Change History (5)

Changed November 25, 2008 11:35PM UTC by flesler comment:1

cc: → jollytoad
owner: → flesler
status: newassigned

That's odd. I thought I added tabIndex to $.props.

I'll add it asap.

It's odd that it still doesn't work on Safari. Does it work properly if you use the expando directly ?

Changed December 03, 2008 03:40PM UTC by scott.gonzal comment:2

The tabindex problem is actually larger than just capitalization. There's a related ticket in jQuery UI that covers all of the problems. There are plenty of links to additional resources in that ticket as well (some of the topics aren't relevant to core). You can see the current progress in the tabbable branch. We'll update this ticket when we finalize our proxy of $.attr.

Changed January 05, 2009 04:02AM UTC by scott.gonzal comment:3

Added a patch with tests that makes getting and setting tabindex work in all browsers.

Changed January 05, 2009 05:25PM UTC by scott.gonzal comment:4

New patch adds comments and fixes selector tests.

Changed January 05, 2009 05:34PM UTC by john comment:5

resolution: → fixed
status: assignedclosed

Fixed in SVN rev [6043].