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)
Tracking
()
RESOLVED
DUPLICATE
of bug 242829
People
(Reporter: george.wang, Unassigned)
Details
Attachments
(1 file)
13.78 KB,
text/html
|
Details |
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.
Comment 1•7 years ago
|
||
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
Comment 2•7 years ago
|
||
(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)
Comment 3•7 years ago
|
||
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)
Comment 4•7 years ago
|
||
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.
Description
•