Closed Bug 1356216 Opened 7 years ago Closed 7 years ago

it doesn't work, when there is tabindex attribute in html anchor tag within table caption tag.

Categories

(Core :: DOM: Core & HTML, defect, P3)

52 Branch
defect

Tracking

()

RESOLVED DUPLICATE of bug 242829

People

(Reporter: george.wang, Unassigned)

Details

Attachments

(1 file)

Attached file assighmentlist.html
User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36

Steps to reproduce:

1.create a html table with caption.
2.add html anchor tag within caption with attribute tabindex="0"
3.using tab to tab through the html 


Actual results:

fireforx won't stop at html anchor within caption tag.


Expected results:

Both IE and Chrome will honor this tabindex.
Firefox should stop at html anchor tag within caption tag when user use tab to tab through html.
Component: Untriaged → DOM: Core & HTML
Product: Firefox → Core
Summary: it doesn't wokr, when there is tabindex attribute in html anchor tag within table capton tag. → it doesn't work, when there is tabindex attribute in html anchor tag within table caption tag.
By simplifying the test case, I encounter a problem that <a> with href attribute and tabindex=0 cannot be focused via tab key.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P3
(In reply to Hsin-Yi Tsai [:hsinyi] from comment #1)
> By simplifying the test case, I encounter a problem that <a> with href
> attribute and tabindex=0 cannot be focused via tab key.
Per my best understanding of the spec https://html.spec.whatwg.org/multipage/interaction.html#attr-tabindex there's something wrong in our code. Neil, maybe you have some ideas?
Flags: needinfo?(enndeakin)
Looks like caption frames aren't being included in the frame traversal (nsFrameIterator). It looks like they are stored in the non-principal child list but this is not checked.

Need to ask someone who knows a little more about tables.
Flags: needinfo?(enndeakin) → needinfo?(mats)
I have a patch + tests in bug 242829 which likely needs some unbitrotting and addressing
review comments.  Feel free to steal it.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(mats)
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: