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)
Change History (5)
Changed November 25, 2008 11:35PM UTC by comment:1
| cc: | → jollytoad |
|---|---|
| owner: | → flesler |
| status: | new → assigned |
Changed December 03, 2008 03:40PM UTC by 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 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 comment:4
New patch adds comments and fixes selector tests.
Changed January 05, 2009 05:34PM UTC by comment:5
| resolution: | → fixed |
|---|---|
| status: | assigned → closed |
Fixed in SVN rev [6043].
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 ?