36217 - Tabindex attribute not working with A, BUTTON, or INPUT elements.

Status: VERIFIED FIXED

(Core :: Layout, defect, P3)

Description

[Chris Petersen]

Build:2000041806
Expect Results: Pressing the tab key should apply focus to the element specified 
by the tabindex value assigned.
What I got: Pressing the tab key doesn't apply focus on the next element in the 
tab order.

Steps to reproduce:
Anchor problem:
1) Open the url in NS6.
2) The file contains five anchor each that have a tabindex value assigned.
3) Attempt to place focus on the 3rd link (the first tabindex value).
4) The link never gets focus so you can press the tab key.

Button problem:
1) Open the attached button test case in NS6.
2) The file contains four buttons each that have a tabindex value assigned.
3) Place focus on the first button (the first tabindex value).
4) After this is done, press the tab key.
5) Focus is not moved to the next element in the tab order. 

Input problem:
1) Open the attached input test case in NS6.
2) The file contains four input fields each that have a tabindex value assigned.
3) Place the cursor in the fisrt field.
4) Press the tab key.
5) Focus is not moved to the next input element (field 4) in the tab order.

Comment 1

[Chris Petersen]

Attachment: A series of buttons that use tabindex.

Comment 2

[Chris Petersen]

Attachment: A series of input fields that use tabindex.

Comment 3

[rickg]

I think this goes to rods for fixing.

Comment 4

[rods (gone)]

Tom, I have tried to decypher the tabbing code and I am not having any luck, I 
think you or saari are much better prepared for figuring this out than I am.

Comment 5

[Chris Petersen]

Tabindex should be working since users will commonly tab throught the order of 
form elements. Marking nsbeta2.

Comment 6

[joki (gone)]

This is a regression, this used to work.  cc'ing hyatt and saari as likely 
candidates.

Comment 7

[joki (gone)]

*** Bug 2642 has been marked as a duplicate of this bug. ***

Comment 8

[joki (gone)]

Added TEXTAREA to the list of broken tabindexs

Comment 9

[Hixie (not reading bugmail)]

[moving regression from summary to keywords]

Comment 10

[leger]

Putting on [nsbeta2-] radar. Not critical to beta2.

Comment 11

[joki (gone)]

Nominating for nsbeta3.  Fix in hand.

Comment 12

[Peter 'Rufus' Nelson]

*** Bug 46139 has been marked as a duplicate of this bug. ***

Comment 13

[Nisheeth Ranjan]

Marking nsbeta3+...

Comment 14

[Nisheeth Ranjan]

The two attachments work, but the url test case (the one with links on it) does 
not show focus rectangle around the links, so I can't tell whether this bug is 
fixed for links.  We are not going to spend more time on this bug pre Netscape 6 
ship.

This bug has been marked "future" because the original netscape engineer working 
on this is over-burdened. If you feel this is an error, that you or another
known resource will be working on this bug,or if it blocks your work in some way 
-- please attach your concern to the bug for reconsideration, but do not clear 
the nsbeta3- nomination.

Comment 15

[Stephen Walker]

*** Bug 49892 has been marked as a duplicate of this bug. ***

Comment 16

[Stephen Walker]

*** Bug 49967 has been marked as a duplicate of this bug. ***

Comment 17

[Stephen Walker]

*** Bug 49982 has been marked as a duplicate of this bug. ***

Comment 18

[Stephen Walker]

*** Bug 50089 has been marked as a duplicate of this bug. ***

Comment 19

[Stephen Walker]

*** Bug 50045 has been marked as a duplicate of this bug. ***

Comment 20

[Stephen Walker]

*** Bug 50216 has been marked as a duplicate of this bug. ***

Comment 21

[Loco]

Testing build 2000-09-04-08 on MacOS 9.0.4.

http://slip/projects/marvin/html/button_tabindex.html

Visiting this link it seems that it is working partially.  Tabbing normally (i.e. 
from URL field) doesn't take the focus to the buttons, but once you click on the 
frame containing the buttons the Tab key indeed takes you through the right 
order.  Can't go back, since Shift-Tab isn't implemented, but at least it works 
in one direction.  Just some food for thought.

Comment 22

[Loco]

FYI-
http://slip/projects/marvin/html/a_tabindex.html is working in Linux 6.2 2000-09-
27-10.

Comment 23

[HJ]

When will this work? And how long do kids, older people and disabled people have
to wait for this kind of support?If this is labeled 'future', then for a lot of
human mankind this browser is HISTORY!

Comment 24

[Alec Flett]

*** Bug 67404 has been marked as a duplicate of this bug. ***

Comment 25

[Jesse Ruderman]

Be sure to check bug 67404 and its testcase after fixing this for shift-tab...

The "button" testcase on this bug doesn't work for me for tab *or* shift-tab, 
btw.

Comment 26

[sairuh (rarely reading bugmail)]

clearing milestone, nominating for beta1. what the status on this --ie, why not
check it in now? thx!

Comment 27

[sairuh (rarely reading bugmail)]

joki, i assume your fixinhand is in your local tree --could you pls attach it
here? thx!

Comment 28

[Blake Ross]

taking...

Comment 29

[Blake Ross]

Attachment: [patch] button tabindex

Comment 30

[Ben Goodger (use ben at mozilla dot org for email)]

a=[e-mail]

Comment 31

[Blake Ross]

Fix checked in.  Links and textfields should be working already.  You still 
have to click for this to work, and shift-tab still isn't reliable, but those 
are separate bugs.

Comment 32

[HJ]

Where should we dup shift-tab bugs to?? Bug 67404 isn't fixed with this patch
but is a dup of this bug.

Comment 33

[Chris Petersen]

Marking verified in the March 23rd build.

Comment 34

[Heikki Toivonen (remove -bugzilla when emailing directly)]

SPAM. HTML Element component deprecated, changing component to Layout. See bug
88132 for details.

Comment 35

[[e-mail]]

When anchor tag is within a table caption. The tabindex won't work for firefox.
Chrome and Ie both have worked.