Ticket #3649 (closed bug: fixed)
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 |
| Blocking: | Blocked by: |
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
-
tabindex.html
(464 bytes) -
added by jollytoad 5 years ago.
- Test page
-
tabindex-jquery.patch
(323 bytes) -
added by jollytoad 5 years ago.
- Patch for core,js against SVN rev 5949
-
tabindex.patch
(4.4 KB) -
added by scott.gonzalez 4 years ago.
-
tabindex.2.patch
(5.8 KB) -
added by scott.gonzalez 4 years ago.
Change History
Changed 5 years ago by jollytoad
-
attachment
tabindex.html
added
Changed 5 years ago by jollytoad
-
attachment
tabindex-jquery.patch
added
Patch for core,js against SVN rev 5949
comment:1 Changed 4 years ago by flesler
- Cc jollytoad added
- Owner set to flesler
- Status changed from new to assigned
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 ?
comment:2 Changed 4 years ago by scott.gonzal
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.
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Test page